-
Notifications
You must be signed in to change notification settings - Fork 60
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
Add the option to hide os from admin settings #6070
Add the option to hide os from admin settings #6070
Conversation
11bd61a
to
d0600cc
Compare
Can you update the release note to something like:
|
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.
The API part looks fine. @Waseem826 can review the UI parts for this one.
LGTM label has been added. Git tree hash: 7c7be2e62c0434b6514125d5f452d54794123289
|
/approve |
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.
Sorry I accidentally clicked the Add Single Comment
button but I have some more comments. PTAL.
d0600cc
to
37bd562
Compare
37bd562
to
f1fbade
Compare
/retest |
f1fbade
to
6b9d9ff
Compare
/retest |
97256d0
to
4a102fd
Compare
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: ahmadhamzh, ahmedwaleedmalik 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 |
/retest |
4a102fd
to
1b4a26e
Compare
/retest |
LGTM label has been added. Git tree hash: 078edd0c6980dd18ee0745b6d2bf4cc672587652
|
/unhold |
What this PR does / why we need it:
Add an option in the admin setting to hide/show operating system when create machine deployment
admin settings
initial Node in wizard
MD dialogs
Which issue(s) this PR fixes:
Fixes #4105
What type of PR is this?
/kind feature