-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
[olm] Store multiple versions of olm manifests #4043
[olm] Store multiple versions of olm manifests #4043
Conversation
@estroz @camilamacedo86 @joelanford - How many versions of OLM do you think we could support for a particular release. In the current PR, the thought is to have a sliding window of 5 versions. When we have a new SDK release, we will remove the oldest OLM version, and add a newer one. |
Hi @varshaprasad96, could you please rebase this one with the master and address the nits for we are able to move forward with? |
e9a09aa
to
528bdf2
Compare
528bdf2
to
32291c5
Compare
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.
It shows fine for me 👍
IMO it needs @estroz ok's as well since he is who is most involved.
This PR modifies the existing script to download multiple versions of olm manifests and generate bindata for all of them during release.
d8f028f
to
a3d2759
Compare
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
a3d2759
to
e59bb1f
Compare
New changes are detected. LGTM label has been removed. |
* [olm] Store multiple versions of olm manifests This PR modifies the existing script to download multiple versions of olm manifests and generate bindata for all of them during release. * Modify changes to samples * change the number stored olm versions Signed-off-by: reinvantveer <rein.van.t.veer@geodan.nl>
* [olm] Store multiple versions of olm manifests This PR modifies the existing script to download multiple versions of olm manifests and generate bindata for all of them during release. * Modify changes to samples * change the number stored olm versions Signed-off-by: rearl <rearl@secureworks.com>
Description of the change:
This PR modifies the existing script to download multiple versions
of olm manifests and generate bindata for all of them during release.
Motivation for the change:
Follow up from #3906
Checklist
If the pull request includes user-facing changes, extra documentation is required:
changelog/fragments
(seechangelog/fragments/00-template.yaml
)website/content/en/docs