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

[chore] Build and publish application hosted in the repository for the Python instrumentation E2E test #2095

Merged
merged 21 commits into from
Oct 4, 2023

Conversation

iblancasa
Copy link
Contributor

Related to #2073.

The test migration will be done in a follow up PR after publishing the image.

Signed-off-by: Israel Blancas <[email protected]>
@iblancasa iblancasa marked this pull request as ready for review September 14, 2023 16:32
@iblancasa iblancasa requested a review from a team September 14, 2023 16:32
@pavolloffay
Copy link
Member

#2097 was merged please rebase and resue the GHA config to publish the image

Copy link
Contributor

@jaronoff97 jaronoff97 left a comment

Choose a reason for hiding this comment

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

LGTM, im rerunning the tests before i merge.

@jaronoff97 jaronoff97 merged commit 423ff4c into open-telemetry:main Oct 4, 2023
27 checks passed
@iblancasa iblancasa deleted the task/2073-python branch October 5, 2023 08:09
ItielOlenick pushed a commit to ItielOlenick/opentelemetry-operator that referenced this pull request May 1, 2024
…e Python instrumentation E2E test (open-telemetry#2095)

* Build and publish the Python E2E image

Signed-off-by: Israel Blancas <[email protected]>

* Fix archs

Signed-off-by: Israel Blancas <[email protected]>

* Fix the tags

Signed-off-by: Israel Blancas <[email protected]>

* Fix the tags

Signed-off-by: Israel Blancas <[email protected]>

* Use reusable workflow

Signed-off-by: Israel Blancas <[email protected]>

* Fix workflow name

Signed-off-by: Israel Blancas <[email protected]>

---------

Signed-off-by: Israel Blancas <[email protected]>
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.

4 participants