Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Release v1.7.0 #1926

Merged
merged 8 commits into from
Sep 30, 2021
Merged

Release v1.7.0 #1926

merged 8 commits into from
Sep 30, 2021

Conversation

carlosalberto
Copy link
Contributor

September release, following our monthly cadence.

@reyang Are we ready to go, metrics-wise?

@bogdandrutu @dyladan Should we include #1923 in this release?

cc @tigrannajaryan

@tigrannajaryan
Copy link
Member

Looking at the changelog I don't see any changes to the schema so the schema file for 1.7.0 needs just an empty section for 1.7.0 version. @carlosalberto this should do it (put it in schemas/1.7.0):

file_format: 1.0.0
schema_url: https://opentelemetry.io/schemas/1.7.0
versions:
  1.7.0
  1.6.1:
  1.5.0:
  1.4.0:

@anuraaga
Copy link
Contributor

Need action on #1658 before releasing I think @Oberon00 @bogdandrutu

@Oberon00
Copy link
Member

Yes, please can we resolve #1658 now? It has been hanging around for quite a while and would be cumbersome to implement it if we don't release it in the same release that added the InstrumentationLibrary to the sampler.

@tigrannajaryan
Copy link
Member

tigrannajaryan commented Sep 17, 2021

Yes, please can we resolve #1658 now? It has been hanging around for quite a while and would be cumbersome to implement it if we don't release it in the same release that added the InstrumentationLibrary to the sampler.

I believe we are using the catch-the-train release model so #1658 is better merged soon or it is not going to make it (or do we delay the release since it is critical?)

Copy link
Member

@reyang reyang left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks @carlosalberto!

@carlosalberto
Copy link
Contributor Author

Hey, was wondering if we should postpone this release to the first days of October, so related issues are properly solved? Or would that hurt anybody?

@github-actions
Copy link

This PR was marked stale due to lack of activity. It will be closed in 7 days.

@github-actions github-actions bot added the Stale label Sep 25, 2021
@tigrannajaryan
Copy link
Member

#1658 is closed. Are we waiting for something else?

CHANGELOG.md Outdated Show resolved Hide resolved
@carlosalberto
Copy link
Contributor Author

Let's try to wait for #1970 by EOD, and then we finally do our release ;)

@tigrannajaryan
Copy link
Member

Let's try to wait for #1970 by EOD, and then we finally do our release ;)

@carlosalberto I will be travelling and may not be able to merge later today, please do it if you can.

@carlosalberto carlosalberto merged commit 1335892 into open-telemetry:main Sep 30, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants