-
Notifications
You must be signed in to change notification settings - Fork 196
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
MGMT-3353 RHCOS per version #869
MGMT-3353 RHCOS per version #869
Conversation
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: YuviGold The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
ebd1ae3
to
c7b532f
Compare
cmd/main.go
Outdated
}), "Failed uploading boot files") | ||
for version := range openshiftVersionsMap { | ||
go func(version string) { | ||
failOnError(baseISOUploadLeader.RunWithLeader(context.Background(), func() error { |
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 would be nice to have a leader per version and parallelize the uploads, but that's for another PR
DoAllBootFilesExist(ctx context.Context, isoObjectName string) (bool, error) | ||
DownloadBootFile(ctx context.Context, isoObjectName, fileType string) (io.ReadCloser, string, int64, error) | ||
GetS3BootFileURL(isoObjectName, fileType string) 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.
Maybe it would be nicer to have openshiftVersion as the parameter instead of isoObjectName?
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 it's much flexible to have the ISO received as a parameter, as been done for the rest of the boot functions.
"4.6": { | ||
"release_image": "quay.io/openshift-release-dev/ocp-release:4.6.8-x86_64" | ||
"display_name": "4.6.8", | ||
"release_image": "quay.io/openshift-release-dev/ocp-release:4.6.8-x86_64", | ||
"rhcos_image": "https://mirror.openshift.com/pub/openshift-v4/dependencies/rhcos/4.6/4.6.8/rhcos-4.6.8-x86_64-live.x86_64.iso", | ||
"support_level": "production" | ||
}, | ||
"4.7": { | ||
"release_image": "quay.io/openshift-release-dev/ocp-release@sha256:2419f9cd3ea9bd114764855653012e305ade2527210d332bfdd6dbdae538bd66" | ||
"display_name": "4.7-pre-release", | ||
"release_image": "quay.io/openshift-release-dev/ocp-release@sha256:2419f9cd3ea9bd114764855653012e305ade2527210d332bfdd6dbdae538bd66", | ||
"rhcos_image": "https://mirror.openshift.com/pub/openshift-v4/dependencies/rhcos/4.6/4.6.8/rhcos-4.6.8-x86_64-live.x86_64.iso", | ||
"support_level": "beta" | ||
} |
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.
@ErezAlster We are all set on metrics right? (We should always have the openshift version)
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.
yes, @oshercc took care for it :)
Enhancing the OPENSHIFT_VERSIONS hashmap to hold rhcos image, display name and support level. Updating everywhere in the code to use RHCOS image matching the cluster OCP version. Including - DownloadBootFiles and CreateISOAndUploadToS3 operations. The name of the baseIso is received from new function s3wrapper.GetBaseIsoObject. For k8s it would be rhcos-<ocp_version>.iso and for onprem it will be for now livecd.iso until updating onprem livecd volume to support a directory of livecds.
c7b532f
to
31056e6
Compare
/lgtm |
Enhancing the OPENSHIFT_VERSIONS hashmap to hold rhcos image, display name and support level. Updating everywhere in the code to use RHCOS image matching the cluster OCP version. Including - DownloadBootFiles and CreateISOAndUploadToS3 operations. The name of the baseIso is received from new function s3wrapper.GetBaseIsoObject. For k8s it would be rhcos-<ocp_version>.iso and for onprem it will be for now livecd.iso until updating onprem livecd volume to support a directory of livecds.
Enhancing the
OPENSHIFT_VERSIONS
hashmap to hold rhcos image,display name and support level.
i.e.
Updating everywhere in the code to use RHCOS image matching the cluster OCP version.
Including -
DownloadBootFiles
andCreateISOAndUploadToS3
operations.The name of the baseIso is received from new function
s3wrapper.GetBaseIsoObject
.For k8s it would be
rhcos-<ocp_version>.iso
and for onprem it will be for nowlivecd.iso
until updating onprem livecd volume to support a directory of livecds.