-
Notifications
You must be signed in to change notification settings - Fork 8
feat!: update to general availability versions of sandboxed containers and trustee #54
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
Conversation
…s and trustee BREAKING CHANGE: pre GA releases will no longer work due to introduction of compressed initdata Signed-off-by: Chris Butler <chris.butler@redhat.com>
Signed-off-by: Chris Butler <chris.butler@redhat.com>
Signed-off-by: Chris Butler <chris.butler@redhat.com>
Signed-off-by: Chris Butler <chris.butler@redhat.com>
Signed-off-by: Chris Butler <chris.butler@redhat.com>
ansible/gzipper.py
Outdated
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.
@mhjacks - As discussed - this is what I did (with gemini) as there no gzip. Adding gzip would be great and mandatory as native ansible can only '.tar.gz' not gzip an arbitrary file / string.
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 have merged PRs to both Utility and Imperative containers to include UBI-9 gzip.
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.
Great. I'll get that fixed merged then before we release.
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's no time like the present to break backward compatibility, especially for the benefit of GA
ansible/gzipper.py
Outdated
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 have merged PRs to both Utility and Imperative containers to include UBI-9 gzip.
Signed-off-by: Chris Butler <chris.butler@redhat.com>
Signed-off-by: Chris Butler <chris.butler@redhat.com>
Signed-off-by: Chris Butler <chris.butler@redhat.com>
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.
Great work @butler54! Everything came up and was running.
Only a small question regarding the origination of one of the images.
Signed-off-by: Chris Butler <chris.butler@redhat.com>
BREAKING CHANGE: pre GA releases will no longer work due to introduction of compressed initdata