-
Notifications
You must be signed in to change notification settings - Fork 17
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
Add new variables in advance to work with both versions #579
Conversation
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.
Lgtm !
.gitlab-ci.yml
Outdated
:git: Branch <$BRANCH_URL|$CI_COMMIT_BRANCH> for commit \`$CI_COMMIT_TITLE\` (<$COMMIT_URL|$CI_COMMIT_SHORT_SHA>) | ||
:idea: You can test them in the datadog-agent repository by running: | ||
\`\`\`inv pipeline.update-buildimages -i v$CI_PIPELINE_ID-${CI_COMMIT_SHORT_SHA} [--no-test-version] [--branch-name <your_branch>]\`\`\` | ||
\`\`\`inv buildimages.update --v ${IMAGE_VERSION} [--no-test]\`\`\` |
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.
Should this be either :
\`\`\`inv buildimages.update --version ${IMAGE_VERSION} [--no-test]\`\`\`
or
\`\`\`inv buildimages.update -v ${IMAGE_VERSION} [--no-test]\`\`\`
?
/ping |
🚂 Devflow: Pong 🏓 |
/status |
🚂 Devflow: This pull request is not in the merge queue, type MergeQueue status for DataDog/datadog-agent-buildimages branch main There is currently no merge request being processed. There is no merge request queued There is no merge request waiting |
Related to DataDog/datadog-agent#24638 and #578
Objective is to be able to build only modified images on buildimage PR to have a faster feedback when images are modified