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

[FEATURE] Enhance the import experience to Rancher #1330

Closed
yasker opened this issue Sep 29, 2021 · 12 comments
Closed

[FEATURE] Enhance the import experience to Rancher #1330

yasker opened this issue Sep 29, 2021 · 12 comments
Assignees
Labels
area/ui Harvester UI kind/enhancement Issues that improve or augment existing functionality priority/0 Must be fixed in this release
Milestone

Comments

@yasker
Copy link
Member

yasker commented Sep 29, 2021

Currently user need to log into a Harvester physical node to run kubectl to import the Harvester cluster to a Rancher cluster.

We should allow that in the UI. Also, we need to enforce that one Harvester cluster can only be managed by one Rancher.

@yasker yasker added the kind/enhancement Issues that improve or augment existing functionality label Sep 29, 2021
@yasker yasker added this to the v1.0.0 milestone Sep 29, 2021
@yasker yasker added the priority/0 Must be fixed in this release label Sep 29, 2021
@jnfinitym
Copy link

I also noticed the binding script in Rancher 2.6.1 seems to look for k3s.yaml instead of rke2.yaml which is created in the installation of Harvester 0.3; copying /etc/rancher/rke2/rke2.yaml to /etc/rancher/k3s/k3s.yaml allowed me to add default as admin-role and subsequently join the Rancher. Should I file this in a dedicated issue or is this just part of this one?

@yasker
Copy link
Member Author

yasker commented Oct 13, 2021

@jnfinitym can you elaborate? Which binding script you're talking about? Might be another issue.

@jnfinitym
Copy link

I will file a seperate issue then, so I have mroe space to describe it - I am not home right now, so I will try it this weekend, so I can add the exact error messages I got and what exactly I did to get around them; I am specifically talking about importing Harvester to a Rancher instance, but it looks to be easily resolvable, but I am thinking it may actually be a Rancher Problem and not a Harvester Problem… My workaround was on the Harvester side though

@jnfinitym
Copy link

jnfinitym commented Oct 17, 2021 via email

@guangbochen
Copy link
Contributor

Refer from #1437

Expected:

that explain how to import a Harvester cluster. E.g. the specific steps to be able to run the "kubectl" command in harvester.

Actual:

user is shown generic instructions to run the command on "an existing Kubernetes cluster running a supported Kubernetes version"

Screenshot 2021-10-15 at 12 56 32

@gitlawr
Copy link
Contributor

gitlawr commented Nov 2, 2021

Proposal for the import UX:

  1. In Rancher UI, we show the instructions for importing Harvester using Harvester UI.
    image

  2. In Harvester settings, add a cluster-registration-url settings. Users import the Harvester cluster by clicking the Import to Rancher button and inputing the registration URL.
    image

  3. Under the hood, Harvester settings controller applies the manifests for the import.

@guangbochen
Copy link
Contributor

LGTM

@guangbochen
Copy link
Contributor

guangbochen commented Nov 21, 2021

To Test: (and please sync to close rancher/dashboard#4531 if passes)

  1. Spin up the Rancher v2.6.3-rc1 and In Rancher UI, we show the instructions for importing Harvester using Harvester UI.
    image

  2. In Harvester settings, add a cluster-registration-url settings. Users import the Harvester cluster by clicking the Import to Rancher button and inputing the registration URL.
    image

  3. Under the hood, the Harvester settings controller applies the manifests for the import and the cluster will be visible in the Rancher UI.

please also close the issue of rancher/dashboard#4531 if the validation passes, thanks.

@TachunLin TachunLin self-assigned this Nov 22, 2021
@TachunLin
Copy link

@gitlawr , @n313893254
Verified not fixed on master-81141c98-head with latest rancher v2.6-head. Move it to implement pipeline.
Please check whether the feature and UI have merged to the latest master release.

On Rancher v2.6-head, we can see the instructions for importing Harvester under registration tab
image

While we can't find the cluster-registration-url configuration in harvester settings page
image

@n313893254
Copy link

@TachunLin My bad. As the PR is merged by rancher/dashboard, we should validate cluster-registration-url after harvester/dashboard rebase rancher/dashboard.

@guangbochen guangbochen added the area/ui Harvester UI label Nov 23, 2021
@guangbochen guangbochen assigned WuJun2016 and unassigned gitlawr Nov 23, 2021
@WuJun2016
Copy link
Contributor

rebase, Done.

@TachunLin
Copy link

Verified fixed on master-b7e562d3-head with Rancher v2.6.3-rc1 .Close this issue.

Result

User can input registration url from rancher into harvester settings then import harvester in rancher dashboard

image

image

Test Information

  • Test Environment: 1 nodes harvester on local kvm/qemu
  • Harvester version: master-b7e562d3-head
  • Rancher version: v2.6.3-rc1

Environment Preparation

  1. Install harvester v2.6.3-rc1 in docker command
  2. Update the ui-dashboard-index and ui-offline-preferred in rancher global settings

Verify Steps

  1. Environment preparation as above steps
  2. Access virtualization management page in rancher
  3. Press ctrl+R to reload page
  4. Import existing harvester
  5. Copy the registration url
    image
  6. Access harvester dashboard settings
  7. Edit the cluster-registration-url
    image
  8. Paste the registration url and save
    image

image

  1. Back to rancher dashboard and wait for import process
    image

  2. Access harvester from rancher virtualization management page

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/ui Harvester UI kind/enhancement Issues that improve or augment existing functionality priority/0 Must be fixed in this release
Projects
None yet
Development

No branches or pull requests

7 participants