Skip to content
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

Issue #136 Stop generating virtualbox bundle #138

Merged
merged 1 commit into from
Dec 10, 2019
Merged

Conversation

praveenkumar
Copy link
Member

As part of CRC we disable some of operators like (monitoring/machine-config ..etc.) but
the images are always present on the node since there is no knob on installer side right now
to disable those operators from starting (iirc.) Now overall disk size is increases,
till 4.2 our final disk size was around 2GB but with 4.3 it is increasing around 3GB
and that is because all the other images are added as part of CVO payload.

crictl version used in the RHCOS is 0.1.0 which doesn't have fix
for kubernetes-sigs/cri-tools#399 one yet so
using kubernetes-sigs/cri-tools#399 (comment) as workaround.

Copy link
Contributor

@cfergeau cfergeau left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you mention in the commit log how much space is gained after this change?
Overall looks good!

createdisk.sh Outdated
@@ -247,6 +247,9 @@ ${OC} --config $1/auth/kubeconfig create clusterrolebinding developer --clusterr
certImage=$(${OC} --config $1/auth/kubeconfig adm release info --image-for=cluster-kube-apiserver-operator)
${SSH} core@api.${CRC_VM_NAME}.${BASE_DOMAIN} -- sudo podman tag $certImage openshift/cert-recovery

# Remove unused images from container stroage
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

'storage'

createdisk.sh Outdated Show resolved Hide resolved
@praveenkumar
Copy link
Member Author

@cfergeau since after removing the images doesn't make a difference on final tarball so I am not going to add this patch and rebase it with only virtualbox removal code.

@praveenkumar praveenkumar changed the title Issue #137 Remove unused images from container storage Issue #136 Stop generating virtualbox bundle Dec 10, 2019
@praveenkumar praveenkumar merged commit 9a361a0 into master Dec 10, 2019
@praveenkumar praveenkumar deleted the issue_137 branch January 17, 2020 14:16
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.

2 participants