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
remove flatpak_base_image #808
Conversation
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.
I have concerns about changing the API for Arrangements 5 and earlier.
@@ -126,14 +124,10 @@ def set_params(self, | |||
self.koji_parent_build.value = koji_parent_build | |||
self.koji_upload_dir.value = koji_upload_dir | |||
self.flatpak.value = flatpak | |||
self.flatpak_base_image.value = flatpak_base_image |
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.
You'll need to change atomic_reactor/schemas/user_params.json in atomic-reactor
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.
ok will make that change
@@ -189,7 +189,6 @@ def __init__(self): | |||
self.koji_kerberos_principal = BuildParam("koji_kerberos_principal", allow_none=True) | |||
self.koji_kerberos_keytab = BuildParam("koji_kerberos_keytab", allow_none=True) | |||
self.flatpak = BuildParam("flatpak", default=False) | |||
self.flatpak_base_image = BuildParam("flatpak_base_image", allow_none=True) |
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.
I don't think you want to change build_request v1 here - that changes the behavior of Arrangement 5- systems.
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.
I think we're fine with making changes that only affect flatpak builds, as they are not working prior to arrangement 6 in any case.
490998b
to
e57f2bd
Compare
Signed-off-by: Robert Cerven <rcerven@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.
LGTM
Release notes:
|
Signed-off-by: Robert Cerven rcerven@redhat.com