Skip to content

Conversation

@rohanpm
Copy link
Contributor

@rohanpm rohanpm commented Jun 17, 2021

This commit introduces support for two new push item types which can now
be located by errata or koji source:

  • ContainerImagePushItem: a container image

  • OperatorManifestPushItem: operator manifests bundle typically found
    alongside container images in koji

This is referred to as "preliminary support" because these classes
haven't defined any attributes yet (i.e. they only have the inherited
attributes from the base PushItem class). This is done to avoid the
change becoming too large and complicated.

In this commit, we focus only on how we locate these push items when
given an advisory; filling out the model with all needed attributes
will come later, and it's expected to be rather complicated to figure
out what's needed there.

@rohanpm rohanpm force-pushed the rohan/errata-containers-prelim branch from f7cef46 to 8291b7f Compare June 17, 2021 05:29
This commit introduces support for two new push item types which can now
be located by errata or koji source:

- ContainerImagePushItem: a container image

- OperatorManifestPushItem: operator manifests bundle typically found
  alongside container images in koji

This is referred to as "preliminary support" because these classes
haven't defined any attributes yet (i.e. they only have the inherited
attributes from the base PushItem class). This is done to avoid the
change becoming too large and complicated.

In this commit, we focus only on how we locate these push items when
given an advisory; filling out the model with all needed attributes
will come later, and it's expected to be rather complicated to figure
out what's needed there.
@rohanpm rohanpm force-pushed the rohan/errata-containers-prelim branch from 8291b7f to 289b2ae Compare June 17, 2021 05:53
@rohanpm rohanpm marked this pull request as ready for review June 17, 2021 06:06
@rohanpm rohanpm changed the base branch from rohan/module-source-ftp-paths to master June 21, 2021 20:51
@rohanpm rohanpm merged commit dbd115c into master Jun 21, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant