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] Cluster field is not really *required* when creating RKE2/K3s Harvester #3165

Closed
lanfon72 opened this issue Nov 17, 2022 · 5 comments
Closed
Assignees
Labels
area/rancher Rancher related issues area/ui Harvester standalone UI or Rancher UI extension 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-ui/small estimate 1-2 working days severity/3 Function working but has a major issue w/ workaround
Milestone

Comments

@lanfon72
Copy link
Member

Describe the bug

When Creating Harvseter RKE2/K3s cluster in Rancher's Cluster Management, the field Cluster is marked as Required but it could still be empty.

vokoscreenNG-2022-11-17_18-40-55.mp4

To Reproduce

Steps to reproduce the behavior:

  1. Install Harvester with any nodes
  2. Install Rancher v2.6.9
  3. Import Harvester in Virtualization Management
  4. Create Harvester RKE2/K3s Cluster in Cluster Management
  5. In Create Harvester, select Imported Harvester, keep Cluster* field empty then click Continue

Expected behavior

Error message should pop-up to ask user to select Cluster.

Environment:

  • Harvester ISO version: v1.1.1-rc1
  • Underlying Infrastructure (e.g. Baremetal with Dell PowerEdge R630): qemu/KVM 3 nodes
  • Rancher Version: v2.6.9 (via docker image)
@lanfon72 lanfon72 added kind/bug Issues that are defects reported by users or that we know have reached a real release area/ui Harvester standalone UI or Rancher UI extension severity/3 Function working but has a major issue w/ workaround area/rancher Rancher related issues need-reprioritize reproduce/always Reproducible 100% of the time labels Nov 17, 2022
@guangbochen guangbochen added this to the v1.1.2 milestone Nov 17, 2022
@guangbochen guangbochen added the priority/0 Must be fixed in this release label Nov 17, 2022
@n313893254 n313893254 added the require-ui/small estimate 1-2 working days label Nov 30, 2022
@harvesterhci-io-github-bot
Copy link

harvesterhci-io-github-bot commented Dec 14, 2022

Pre Ready-For-Testing Checklist

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

  • Where is the reproduce steps/test steps documented?
    The reproduce steps/test steps are at:

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

  • If labeled: area/ui Has the UI issue filed or ready to be merged?
    The UI issue/PR 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:

@harvesterhci-io-github-bot

Automation e2e test issue: harvester/tests#655

@lanfon72
Copy link
Member Author

@n313893254 will this not fix on v2.6-head?

@n313893254
Copy link

@lanfon72 Thank you! The v2.6-head PR is rancher/dashboard#7785 and can test it with https://releases.rancher.com/dashboard/release-2.6/index.html

@lanfon72
Copy link
Member Author

lanfon72 commented Jan 3, 2023

Verified this bug has been fixed.

image

Test Information

  • Environment: baremetal DL360G9 3 nodes
  • Harvester Version: master-37ff3072-head
  • ui-source Option: Auto
  • Rancher Version: v2.6-head(Docker Image: d6b73f4a7976) and v2.7-head(Docker Image: 964d1d1fd7a0)

Verify Steps:

@lanfon72 lanfon72 closed this as completed Jan 3, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/rancher Rancher related issues area/ui Harvester standalone UI or Rancher UI extension 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-ui/small estimate 1-2 working days severity/3 Function working but has a major issue w/ workaround
Projects
None yet
Development

No branches or pull requests

5 participants