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

Change default root disk size to 100Gi #74

Merged
merged 1 commit into from
Jan 9, 2024

Conversation

DebakelOrakel
Copy link
Contributor

@DebakelOrakel DebakelOrakel commented Jan 9, 2024

RedHat lowered requirements for OpenShift node root disk size to 100Gi.
This change also makes the default root disk size configurable.

Checklist

  • Keep pull requests small so they can be easily reviewed.
  • Categorize the PR by setting a good title and adding one of the labels:
    bug, enhancement, documentation, change, breaking, dependency
    as they show up in the changelog
  • Link this PR to related issues.

@DebakelOrakel DebakelOrakel added enhancement New feature or request breaking and removed enhancement New feature or request labels Jan 9, 2024
@DebakelOrakel DebakelOrakel requested a review from simu January 9, 2024 12:27
@DebakelOrakel DebakelOrakel force-pushed the feat/change-root-size branch 2 times, most recently from af41ff8 to ed1e9db Compare January 9, 2024 12:32
@DebakelOrakel DebakelOrakel requested review from a team and removed request for simu January 9, 2024 12:42
Copy link
Member

@simu simu left a comment

Choose a reason for hiding this comment

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

IIRC, you need to add volume_size_gb in

lifecycle {
ignore_changes = [
skip_waiting_for_ssh_host_keys,
image_slug,
user_data,
]
}
to ensure that this doesn't try to recreate VMs for existing clusters or make sure that existing clusters have overrides, since we probably don't want to ignore all changes.

@DebakelOrakel
Copy link
Contributor Author

DebakelOrakel commented Jan 9, 2024

IIRC, you need to add volume_size_gb in

lifecycle {
ignore_changes = [
skip_waiting_for_ssh_host_keys,
image_slug,
user_data,
]
}

to ensure that this doesn't try to recreate VMs for existing clusters or make sure that existing clusters have overrides, since we probably don't want to ignore all changes.

The Plan is to define the default_volume_size_gb in tenant-repos, but I suppose putting in lifecycle is more flexible 👍
This also means, the change is non-breaking.

RedHar lowered requirements for OpenShift node root disk size to 100Gi.
This change also makes the default root disk size configurable.
@DebakelOrakel DebakelOrakel force-pushed the feat/change-root-size branch from ed1e9db to e3e7780 Compare January 9, 2024 12:59
@DebakelOrakel DebakelOrakel added enhancement New feature or request and removed breaking labels Jan 9, 2024
@simu
Copy link
Member

simu commented Jan 9, 2024

IIRC, you need to add volume_size_gb in

lifecycle {
ignore_changes = [
skip_waiting_for_ssh_host_keys,
image_slug,
user_data,
]
}

to ensure that this doesn't try to recreate VMs for existing clusters or make sure that existing clusters have overrides, since we probably don't want to ignore all changes.

The Plan is to define the default_volume_size_gb in tenant-repos, but I suppose putting in lifecycle is more flexible 👍 This also means, the change is non-breaking.

Yeah, I guess we'd have to do volume size changes manually anyway, even for changes that are possible (i.e. make the volume bigger).

@DebakelOrakel DebakelOrakel merged commit 3332396 into master Jan 9, 2024
1 check passed
@DebakelOrakel DebakelOrakel deleted the feat/change-root-size branch January 9, 2024 14:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants