-
Notifications
You must be signed in to change notification settings - Fork 183
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
SUMO-117261: Skip using TRAVIS_COMMIT_RANGE for yaml detection #168
Conversation
2c08dcb
to
72b5318
Compare
72b5318
to
031566e
Compare
ci/build.sh
Outdated
git checkout $TRAVIS_BRANCH | ||
fi | ||
|
||
# Check for invalid changes to overrides yaml files |
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 check be at the top like it was before? The reason we put it at the top is because we didn't want to build the gems or the image if it would fail here
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.
If we're okay with moving the # Set up Github
part to the top too, I'm fine with having it fail faster. In my testing it wasn't a huge deal that it built the gems/docker image before failing.
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.
Thanks for fixing this Ryan
echo "No changes in the generated overrides files." | ||
fi | ||
# Generate override yaml files for chart dependencies to determine if changes are made to overrides yaml files | ||
echo "Generating overrides files..." |
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.
Do we always generate override files now?
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.
We do. We will only commit them if there is a difference detected, however.
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.
gtk
Description
Recommend using "Hide whitespace changes" since most of this is just indent change.
Rather than using
TRAVIS_COMMIT_RANGE
which is empty on initial commit, we cangit diff master...$TRAVIS_BRANCH
to detect manual changes to the yaml files.Rather than checking for changes made to
values.yaml
, we can always generate the new files and commit them if there is a difference.Testing performed