8000 GA for Azure exporter v1.0.0, minor Releases for certain packages by lzchen · Pull Request #792 · census-instrumentation/opencensus-python · GitHub
[go: up one dir, main page]

Skip to content
This repository was archived by the owner on Sep 17, 2025. It is now read-only.

GA for Azure exporter v1.0.0, minor Releases for certain packages#792

Merged
lzchen merged 5 commits intocensus-instrumentation:v0.7.xfrom
lzchen:release
Oct 1, 2019
Merged

GA for Azure exporter v1.0.0, minor Releases for certain packages#792
lzchen merged 5 commits intocensus-instrumentation:v0.7.xfrom
lzchen:release

Conversation

@lzchen
Copy link
Contributor
@lzchen lzchen commented Sep 30, 2019

Includes minor release for the following packages:
core - 0.7.4
azure - 1.0.0
django - 0.7.2

We are bumping the version for opencensus-ext-azure to 1.0.0 for our GA release.

EDIT:
It seems that [#781] is breaking some system tests if included in the release which includes changes to opencensus-ext-flask.
I will exclude these changes for this release since the GA needs to be published today.

@lzchen lzchen requested review from a team, c24t, reyang and songy23 as code owners September 30, 2019 22:58
@lzchen lzchen changed the title Minor Releases for certain packages GA for Azure exporter v1.0.0, minor Releases for certain packages Sep 30, 2019
@reyang
Copy link
Contributor
reyang commented Sep 30, 2019

Copy link
Contributor
@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 once we have the setup.py file updated.

- Standard Metrics - Incoming Requests Execution Time
([#773](https://github.com/census-instrumentation/opencensus-python/pull/773))
- Implement connection strings
([#767](https://github.com/census-instrumentation/opencensus-python/pull/767))
Copy link
Member

Choose a reason for hiding this comment

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

Ideally these would already be in the changelog on master at the time of the release.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Good point. I will update the changelog on master ASAP.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

[#794]

@c24t
Copy link
Member
c24t commented Oct 1, 2019

Are you going to cut a separate release for #781?

@lzchen
Copy link
Contributor Author
lzchen commented Oct 1, 2019

@c24t
Yes, I will try to fix the breaking system tests and cut a separate release for [#781] after this one.

@lzchen lzchen merged commit 5b064c9 into census-instrumentation:v0.7.x Oct 1, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet
< 48C9 /div>

Development

Successfully merging this pull request may close these issues.

4 participants

0