-
Notifications
You must be signed in to change notification settings - Fork 800
[release/v1.28.x-0.49bx] Prepare release 1.28.2/0.49b2 #3014
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.28.x-0.49bx] Prepare release 1.28.2/0.49b2 #3014
Conversation
|
This took longer than expected. Going to update the changelog date and finish release on Monday 🏖️ |
| # limitations under the License. | ||
|
|
||
| __version__ = "2.0.0.dev" | ||
| __version__ = "0.49b2" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks wrong
| # limitations under the License. | ||
|
|
||
| __version__ = "2.1.0.dev" | ||
| __version__ = "1.28.2" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Wrong aswell
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think we should skip packages that are released individually no?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yep, looks like the script updating this is also forgetting about exporter/, propagator/and processor/ and maybe other dirs?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think we should skip packages that are released individually no?
I think @lzchen mentioned this, I will make a note of it. IIUC it should be inconsequential because the independent release packages won't be uploaded during the release workflow. WDYT?
looks like the script updating this is also forgetting about
exporter/,propagator/andprocessor/and maybe other dirs?
Ahh that's more concerning, I'll take a look thanks!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks like the script updating this is also forgetting about
exporter/,propagator/andprocessor/and maybe other dirs?Ahh that's more concerning, I'll take a look thanks!
I guess we're missing a ton of entries for these in eachdist.ini?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I figured it out, #3016
Running workflow in that PR generated the below diff stat vs this PR
Details
_template/version.py | 2 +-
exporter/opentelemetry-exporter-prometheus-remote-write/src/opentelemetry/exporter/prometheus_remote_write/version.py | 2 +-
exporter/opentelemetry-exporter-richconsole/src/opentelemetry/exporter/richconsole/version.py | 2 +-
instrumentation-genai/opentelemetry-instrumentation-openai-v2/src/opentelemetry/instrumentation/openai_v2/version.py | 2 +-
instrumentation/opentelemetry-instrumentation-aiohttp-client/pyproject.toml | 2 +-
instrumentation/opentelemetry-instrumentation-aiohttp-server/pyproject.toml | 2 +-
instrumentation/opentelemetry-instrumentation-asgi/pyproject.toml | 2 +-
instrumentation/opentelemetry-instrumentation-django/pyproject.toml | 2 +-
instrumentation/opentelemetry-instrumentation-falcon/pyproject.toml | 2 +-
instrumentation/opentelemetry-instrumentation-fastapi/pyproject.toml | 2 +-
instrumentation/opentelemetry-instrumentation-flask/pyproject.toml | 2 +-
instrumentation/opentelemetry-instrumentation-httpx/pyproject.toml | 2 +-
instrumentation/opentelemetry-instrumentation-pyramid/pyproject.toml | 2 +-
instrumentation/opentelemetry-instrumentation-requests/pyproject.toml | 2 +-
instrumentation/opentelemetry-instrumentation-starlette/pyproject.toml | 2 +-
instrumentation/opentelemetry-instrumentation-tornado/pyproject.toml | 2 +-
instrumentation/opentelemetry-instrumentation-urllib/pyproject.toml | 2 +-
instrumentation/opentelemetry-instrumentation-urllib3/pyproject.toml | 2 +-
instrumentation/opentelemetry-instrumentation-wsgi/pyproject.toml | 2 +-
opentelemetry-instrumentation/src/opentelemetry/instrumentation/bootstrap_gen.py | 102 +++++++++++++++++++++++++++++++++++++++++++++++++++---------------------------------------------------
processor/opentelemetry-processor-baggage/src/opentelemetry/processor/baggage/version.py | 2 +-
propagator/opentelemetry-propagator-ot-trace/src/opentelemetry/propagators/ot_trace/version.py | 2 +-
scripts/eachdist.py | 13 +++++++++++++
sdk-extension/opentelemetry-sdk-extension-aws/src/opentelemetry/sdk/extension/aws/version.py | 2 +-
util/opentelemetry-util-http/src/opentelemetry/util/http/version.py
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
PTAL at the Prepare release PR: #3017
|
Will run the release workflow again and generate a new PR |
Prepare release 1.28.2/0.49b2.