Skip to content

Improve Rancher create and edit#64

Merged
amstuta merged 2 commits intoovh:mainfrom
thcdrt:dev/thcdrt/rancher-improve-create-and-edit
Oct 14, 2025
Merged

Improve Rancher create and edit#64
amstuta merged 2 commits intoovh:mainfrom
thcdrt:dev/thcdrt/rancher-improve-create-and-edit

Conversation

@thcdrt
Copy link
Contributor

@thcdrt thcdrt commented Oct 10, 2025

Description

Remove iprestrictions from edition as it's not supported by backend for now.
Add iam-auth-enabled parameter to edit.

Type of change

Please delete options that are not relevant.

  • Improvement (improvement of existing commands)
  • Documentation update

Checklist:

  • My code follows the style guidelines of this project
  • I have commented my code
  • I updated the documentation by running make doc
  • I ran go mod tidy
  • I have added tests that prove my fix is effective or that my feature works

@thcdrt thcdrt requested a review from a team as a code owner October 10, 2025 14:28
@thcdrt thcdrt force-pushed the dev/thcdrt/rancher-improve-create-and-edit branch from 246a829 to d39fd72 Compare October 13, 2025 14:16
@thcdrt thcdrt force-pushed the dev/thcdrt/rancher-improve-create-and-edit branch from d39fd72 to 040a17d Compare October 14, 2025 12:19
amstuta
amstuta previously approved these changes Oct 14, 2025
@amstuta
Copy link
Collaborator

amstuta commented Oct 14, 2025

needs to be rebased, otherwise all good :)

Signed-off-by: Thomas Coudert <thomas.coudert@ovhcloud.com>
Signed-off-by: Thomas Coudert <thomas.coudert@ovhcloud.com>
@thcdrt thcdrt force-pushed the dev/thcdrt/rancher-improve-create-and-edit branch from 040a17d to d981c56 Compare October 14, 2025 13:40
@amstuta amstuta merged commit 8d341f8 into ovh:main Oct 14, 2025
3 checks passed
fluatovh pushed a commit to fluatovh/ovhcloud-cli that referenced this pull request Nov 17, 2025
…te-and-edit

Improve Rancher create and edit
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