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

Move init container release from lambda to GHA #2179

Merged
merged 8 commits into from
Jan 30, 2025

Conversation

hmstepanek
Copy link
Contributor

@hmstepanek hmstepanek commented Dec 18, 2024

Overview

We are getting rid of the internal auto-layer-releases repo and moving creating of the release tags into each agent's GHA deploy flow.

@hmstepanek hmstepanek marked this pull request as ready for review January 15, 2025 00:31
Copy link
Contributor

@kanderson250 kanderson250 left a 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 move this into its own workflow, to be run at the end of our release process.

I'm not super familiar with the init container process, but it looks like these changes will cut the init container release immediately. publish_release is the start of our release process. The agent jar generally isn't available on the downloads site, where the init container is looking for it, until a few days after this action is run.

.github/workflows/Create-Release-Tags.yml Outdated Show resolved Hide resolved
.github/workflows/Create-Release-Tags.yml Outdated Show resolved Hide resolved
.github/workflows/Create-Release-Tags.yml Outdated Show resolved Hide resolved
Copy link
Contributor

@kanderson250 kanderson250 left a comment

Choose a reason for hiding this comment

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

Looks good, thank you! Will merge myself once release activities conclude this week.

@jtduffy
Copy link
Contributor

jtduffy commented Jan 30, 2025

Merging with AIT failures (token issue). New workflow action that AITs don't cover.

@jtduffy jtduffy merged commit 3b2fcd9 into newrelic:main Jan 30, 2025
29 of 30 checks passed
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.

3 participants