-
Notifications
You must be signed in to change notification settings - Fork 486
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 build validation when variant packages aren't in bootstrap #1318
Conversation
This repo has @mesosphere-mergebot integration. You can interact with the following commands.
|
Here's proof that the |
This prevents an exception from being raised when a bootstrap package list doesn't include a package listed in a treeinfo's package variants. A treeinfo with bootstrap_package_list is expected to produce a bootstrap tarball containing a subset of all cluster packages, so it's not necessarily an error if a package variant isn't present in it.
30408d5
to
ec228bc
Compare
@BenWhitehead @orsenthil Ready for review. |
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.
🚢
High Level Description
This prevents an exception from being raised when a treeinfo's bootstrap package list doesn't pull in a package listed in the treeinfo's package variants. A treeinfo with
bootstrap_package_list
is expected to produce a bootstrap tarball containing a subset of its complete set of packages, so it's not necessarily an error if a package variant isn't present within it.Related Issues
Checklist for all PR's
Checklist for component/package updates:
If you are changing components or packages in DC/OS (e.g. you are bumping the sha or ref of anything underneath
packages
), then in addition to the above please also include: