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

Deprecate KubeVirt compute resource #3619

Merged
merged 1 commit into from
Jan 30, 2025

Conversation

Lennonka
Copy link
Contributor

What changes are you introducing?

Adding deprecation notice for KubeVirt compute resource

Why are you introducing these changes? (Explanation, links to references, issues, etc.)

SAT-29985 (private)

Anything else to add? (Considerations, potential downsides, alternative solutions you have explored, etc.)

Checklists

  • I am okay with my commits getting squashed when you merge this PR.
  • I am familiar with the contributing guidelines.

Please cherry-pick my commits into: F3.14

@Lennonka Lennonka marked this pull request as draft January 27, 2025 17:03
Copy link
Contributor

@maximiliankolb maximiliankolb left a comment

Choose a reason for hiding this comment

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

diff looks very similar to #3616. So you'll probably want to rebase after merging 3616. Also, I am kinda surprised: https://www.redhat.com/en/blog/introducing-red-hat-openshift-virtualization-engine

@Lennonka
Copy link
Contributor Author

Waiting for #3616 to be merged.

@Lennonka Lennonka marked this pull request as ready for review January 30, 2025 13:38
@Lennonka Lennonka added Needs tech review Requires a review from the technical perspective Needs style review Requires a review from docs style/grammar perspective labels Jan 30, 2025
@ares
Copy link
Member

ares commented Jan 30, 2025

ACK

@Lennonka Lennonka added tech review done No issues from the technical perspective and removed Needs tech review Requires a review from the technical perspective labels Jan 30, 2025
Copy link
Contributor

@maximiliankolb maximiliankolb left a comment

Choose a reason for hiding this comment

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

style-wise LGTM

@maximiliankolb maximiliankolb added style review done No issues from docs style/grammar perspective and removed Needs style review Requires a review from docs style/grammar perspective labels Jan 30, 2025
@Lennonka Lennonka merged commit d6f7dfb into theforeman:master Jan 30, 2025
8 of 9 checks passed
@Lennonka Lennonka deleted the deprecate-kubevirt branch January 30, 2025 15:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
style review done No issues from docs style/grammar perspective tech review done No issues from the technical perspective
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants