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

[backport v2.10.2] [Enhancement] Virtualization manager shows missing repo if repo URL ends with .git or / #13203

Closed
github-actions bot opened this issue Jan 24, 2025 · 1 comment

Comments

@github-actions
Copy link
Contributor

This is a backport issue for #13199, automatically created via GitHub Actions workflow initiated by @richard-cox

Original issue body:

Setup

  • Rancher version: 2.10.1
  • Rancher UI Extensions:
  • Browser type & version: Chrome

Describe the bug
Virt management page shows repo missing if user manually input harvester-ui-extension URL with .git or /. e.g.

To Reproduce

  1. Go to Local -> Apps -> Respositories page, create a harvester repo with above url
  2. import a harvester cluster
  3. The missing repo message pop up

Result

Image Image

**Expected
The missing repo message should NOT popup up

Additional context

@github-actions github-actions bot added area/harvester kind/bug kind/enhancement QA/dev-automation Issues that engineers have written automation around so QA doesn't have look at this QA/None team/harvester labels Jan 24, 2025
@github-actions github-actions bot added this to the v2.10.2 milestone Jan 24, 2025
@torchiaf torchiaf added QA/manual-test Indicates issue requires manually testing and removed QA/dev-automation Issues that engineers have written automation around so QA doesn't have look at this labels Jan 27, 2025
@yonasberhe23
Copy link
Contributor

yonasberhe23 commented Jan 27, 2025

Reproduced in 2.10.1 with the provided repro steps

Image

Fix tested in:

  • Rancher v2.10-7fafafe2aeaf808afbd8cbd4eec82fa02cbd00f9-head
  • Dashboard release-2.10 da5c6f2
Image

Tested:

Created an automated task to add coverage for this test case rancher/qa-tasks#1651

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants