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

Changes to OS and package updates as a follow-up on the discussion in PR #169 #209

Merged

Conversation

georgievaVMW
Copy link
Contributor

@georgievaVMW georgievaVMW commented Nov 5, 2021

Removed the step that changes the OS auto-upgrading settings as this is work of the site operator and that step is not needed.
Changed the step that installs external packages such as kubeadm, kubelet and so on to disable auto-upgrading after installing the package.
Updated the tests which are checking for number of completed steps with the new values.
Link to PR #169

Removed the step that changes the OS auto-upgrading settings.
Changed the step that installs outside packages such as kubeadm, kubelet and so on to disable auto-upgrading after installing the package.
Updated the tests which are checking for number of completed steps with the new values.
Copy link
Contributor

@ratanasovvmw ratanasovvmw left a comment

Choose a reason for hiding this comment

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

LGTM. Let's see if Yixing's ok

Copy link
Contributor

@yixingjia yixingjia left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@jeuxdemains jeuxdemains left a comment

Choose a reason for hiding this comment

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

2nd approval

@anusha94 anusha94 self-requested a review November 17, 2021 10:13
@anusha94 anusha94 merged commit 615352e into vmware-tanzu:main Nov 17, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants