-
Notifications
You must be signed in to change notification settings - Fork 31
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
build(base): inline base img into build #982
Conversation
Hi @andrewazores! Add at least one of the required labels to this PR Required labels are : chore,ci,cleanup,docs,feat,fix,perf,refactor,style,test |
2 similar comments
Hi @andrewazores! Add at least one of the required labels to this PR Required labels are : chore,ci,cleanup,docs,feat,fix,perf,refactor,style,test |
Hi @andrewazores! Add at least one of the required labels to this PR Required labels are : chore,ci,cleanup,docs,feat,fix,perf,refactor,style,test |
@ebaron should we edit that GitHub Workflow config file to add a "build" semantic label? Assuming this will play nice with the automatic release drafter. |
Ah, that new That plugin is doing two things:
Worth dropping the |
Absolutely. I like using |
|
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.
Hey Andrew, looks good overall. Would you also be able to update the assembly descriptor under /src/assembly/dist.xml
? We need to remove cryostat.jfc
from the src/main/extras
fileSet and probably just add another fileSet to include the whole src/container/include
directory.
allows UBI image stream and tag to be defined in pom
How should I verify the results of updating the |
Just run |
Here are the results of producing and then extracting the tarball. Before PR:
After PR:
Looks like the |
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.
Looks good, thanks!
Fixes cryostatio/cryostat-base#14