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.13.0 #2724

Merged
merged 13 commits into from
Sep 19, 2022
Merged

Conversation

arminru
Copy link
Member

@arminru arminru commented Aug 12, 2022

This PR prepares the changelog for this month's spec release and adds some missing entries.

@arminru arminru requested review from a team August 12, 2022 13:17
Copy link
Member

@bogdandrutu bogdandrutu left a comment

Choose a reason for hiding this comment

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

Please resolve #2726 before, to not release the new direction changes without a clear guidance.

@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 Aug 20, 2022
@arminru arminru removed the Stale label Aug 22, 2022
CHANGELOG.md Outdated Show resolved Hide resolved
@tigrannajaryan
Copy link
Member

Please resolve #2726 before, to not release the new direction changes without a clear guidance.

@bogdandrutu do you want full resolution of #2726 or it is enough to if we merge #2748 which reverts #2617 and we can work on resolving #2726 fully after the release (including adding better guidance about usage of attributes vs separate metrics)?

@reyang
Copy link
Member

reyang commented Sep 2, 2022

Please resolve #2726 before, to not release the new direction changes without a clear guidance.

@bogdandrutu this should have been resolved by #2748, PTAL.

@tigrannajaryan
Copy link
Member

Note that #2762 is a release blocker.

@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 14, 2022
@carlosalberto
Copy link
Contributor

@open-telemetry/specs-approvers The last blocking item has been merged - I suggest we release later today. Please take a look, etc.

@tigrannajaryan
Copy link
Member

I think we need to make sure #2789 is seen by as many maintainers as possible to avoid surprises (some languages already started implementing the Scope attribute in the wrong way and have to revert it). The PR was created only 2 days, so I am not sure people are aware of this change.

I posted in Slack to raise awareness. I suggest that we give at least 2 days before we make 1.13 release.

@carlosalberto
Copy link
Contributor

@tigrannajaryan Makes sense - early Friday as release date (as long as no complains happen) sounds good enough?

@tigrannajaryan
Copy link
Member

Sounds good. In the meantime please circulate #2789 everywhere with everyone you think may be impacted or have an opinion.

CHANGELOG.md Outdated Show resolved Hide resolved
@arminru arminru merged commit 59073d8 into open-telemetry:main Sep 19, 2022
@arminru arminru deleted the prep-release-1.13 branch September 19, 2022 16:37
carlosalberto pushed a commit to carlosalberto/opentelemetry-specification that referenced this pull request Oct 31, 2024
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