-
Notifications
You must be signed in to change notification settings - Fork 805
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
Fix for make check does not respect the env-images.yaml #3216
Conversation
Signed-off-by: Daivik <daivikdave229@gmail.com>
[REVIEW NOTIFICATION] This pull request has been approved by:
To complete the pull request process, please ask the reviewers in the list to review by filling The full list of commands accepted by this bot can be found here. Reviewer can indicate their review by submitting an approval review. |
Codecov Report
@@ Coverage Diff @@
## master #3216 +/- ##
==========================================
- Coverage 41.26% 41.11% -0.16%
==========================================
Files 165 165
Lines 13819 13819
==========================================
- Hits 5703 5681 -22
- Misses 7691 7707 +16
- Partials 425 431 +6
Continue to review full report at Codecov.
|
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.
Hi @DaivikDave! Thanks for your contribution!
Please update the CHANGELOG.md, rest LGTM!
after exporting enough of ENV, maybe BASIC_IMAGE_ENV could be removed in the future?
/cc @YangKeao
e4635c8
to
78aca42
Compare
Signed-off-by: Daivik <daivikdave229@gmail.com>
78aca42
to
4dd1faf
Compare
Thanks @STRRL , Updated the changelog! |
Hi @DaivikDave , you should update the section under |
Hi @DaivikDave, do you need some help? :) |
Hey @STRRL , I was unable to work on it due to some urgency. I'll get the changes pushed. |
…into makefile_env_fix
Signed-off-by: Daivik <daivikdave229@gmail.com>
Hey @STRRL , Sorry for the delay, I made the changes. |
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!
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
BTW: export
doesn't work for the shell
command in the Makefile
. It's a known but will not be fixed issue (of the GNU make) 😢 .
/merge |
This pull request has been accepted and is ready to merge. Commit hash: 1ed522c
|
cherry pick to release-2.1 failed |
cherry pick to release-2.2 failed |
Signed-off-by: Daivik daivikdave229@gmail.com
What problem does this PR solve?
#3210
What's changed and how it works?
IMAGE_DEV_ENV_TAG and IMAGE_DEV_ENV_TAG were not accessible in build/build_images.py. It resulted in it taking the default value defined by export IMAGE_TAG ?= latest
Related changes
chaos-mesh/website
Dashboard UI
Checklist
CHANGELOG
CHANGELOG.md
Tests
Side effects
Release note
DCO
If you find the DCO check fails, please run commands like below (Depends on the actual situations. For example, if the failed commit isn't the most recent) to fix it: