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

[BUG] RKE2 created with incorrect volumes/networks amount #3786

Closed
lanfon72 opened this issue Apr 18, 2023 · 2 comments
Closed

[BUG] RKE2 created with incorrect volumes/networks amount #3786

lanfon72 opened this issue Apr 18, 2023 · 2 comments
Assignees
Labels
area/dashboard Harvester/Rancher dashboard area/rke2 RKE2 area/ui kind/bug Issues that are defects reported by users or that we know have reached a real release not-require/test-plan Skip to create a e2e automation test issue priority/0 Must be fixed in this release reproduce/always Reproducible 100% of the time require/release-note severity/3 Function working but has a major issue w/ workaround
Milestone

Comments

@lanfon72
Copy link
Member

Describe the bug

vokoscreenNG-2023-04-18_19-49-50.mp4

To Reproduce

Steps to reproduce the behavior:

  1. Install Harvester with any nodes
  2. Login to Rancher, import Harvester into Rancher
  3. Create RKE2 cluster having 2+ volumes/networks with duplicated cluster name
  4. Update volumes/networks to 1 and use another valid cluster name
  5. Cluster should created with 1 volume/network

Expected behavior

Cluster's Volumes and networks should be the same as the final configuration

Environment:

  • Harvester ISO version: v1.1.2-rc7
  • ui-source Option: Auto
  • Underlying Infrastructure (e.g. Baremetal with Dell PowerEdge R630): Baremetal DL360 3 nodes
  • Rancher Version: v2.7.2-rc7

Additional context

The root cause is, API(v1/rke-machine-config.cattle.io.harvesterconfigs/fleet-default) PUT method is not work correctly.

@lanfon72 lanfon72 added kind/bug Issues that are defects reported by users or that we know have reached a real release severity/3 Function working but has a major issue w/ workaround area/rke2 RKE2 need-reprioritize reproduce/always Reproducible 100% of the time labels Apr 18, 2023
@WuJun2016 WuJun2016 self-assigned this Apr 19, 2023
@WuJun2016 WuJun2016 added this to the v1.2.0 milestone Apr 19, 2023
@WuJun2016 WuJun2016 added area/dashboard Harvester/Rancher dashboard require/release-note labels Apr 19, 2023
@guangbochen guangbochen added priority/0 Must be fixed in this release and removed need-reprioritize labels Apr 28, 2023
@WuJun2016 WuJun2016 added the not-require/test-plan Skip to create a e2e automation test issue label May 6, 2023
@harvesterhci-io-github-bot
Copy link

harvesterhci-io-github-bot commented May 6, 2023

Pre Ready-For-Testing Checklist

* [ ] If labeled: require/HEP Has the Harvester Enhancement Proposal PR submitted?
The HEP PR is at:

* [ ] Is there a workaround for the issue? If so, where is it documented?
The workaround is at:

* [ ] Have the backend code been merged (harvester, harvester-installer, etc) (including backport-needed/*)?
The PR is at:

* [ ] Does the PR include the explanation for the fix or the feature?

* [ ] Does the PR include deployment change (YAML/Chart)? If so, where are the PRs for both YAML file and Chart?
The PR for the YAML change is at:
The PR for the chart change is at:

* [ ] If labeled: require/doc, require/knowledge-base Has the necessary document PR submitted or merged?
The documentation/KB PR is at:

* [ ] If NOT labeled: not-require/test-plan Has the e2e test plan been merged? Have QAs agreed on the automation test case? If only test case skeleton w/o implementation, have you created an implementation issue?
- The automation skeleton PR is at:
- The automation test case PR is at:

* [ ] If the fix introduces the code for backward compatibility Has a separate issue been filed with the label release/obsolete-compatibility?
The compatibility issue is filed at:

@lanfon72
Copy link
Member Author

lanfon72 commented Jun 5, 2023

Verified this bug has been fixed.

Test Information

  • Environment: qemu/KVM 2 nodes
  • Harvester Version: v1.2-d1c90dc4-head
  • ui-source Option: Auto
  • Rancher Version: v2.7-f0d4078f4fa75b303ce276c3f4b12fc465b1ba2a-head (Docker Image: sha256:5f03db286465685fa9c912bb6e6dd510a4b6549795c5db0047f66b4697999bc3)

Verify Steps

@lanfon72 lanfon72 closed this as completed Jun 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/dashboard Harvester/Rancher dashboard area/rke2 RKE2 area/ui kind/bug Issues that are defects reported by users or that we know have reached a real release not-require/test-plan Skip to create a e2e automation test issue priority/0 Must be fixed in this release reproduce/always Reproducible 100% of the time require/release-note severity/3 Function working but has a major issue w/ workaround
Projects
None yet
Development

No branches or pull requests

4 participants