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

Add e2e tests for create rke1 custom cluster #9058

Merged
merged 3 commits into from
Jun 27, 2023

Conversation

richard-cox
Copy link
Member

@richard-cox richard-cox commented Jun 6, 2023

Summary

Fixes #8785

Occurred changes and/or fixed issues

Technical notes summary

  • also updated lots of ~/ @/

@richard-cox richard-cox added this to the v2.7.next3 milestone Jun 6, 2023
@richard-cox richard-cox self-assigned this Jun 6, 2023
@github-actions github-actions bot modified the milestones: v2.7.next3, v2.7.next2 Jun 6, 2023
@richard-cox richard-cox modified the milestones: v2.7.next2, v2.7.next3 Jun 7, 2023
@richard-cox richard-cox marked this pull request as ready for review June 7, 2023 15:22
cypress/e2e/tests/pages/cluster-manager.spec.ts Outdated Show resolved Hide resolved
cypress/e2e/po/components/ember/ember-accordion.po.ts Outdated Show resolved Hide resolved
cypress/e2e/po/components/ember/ember-component.po.ts Outdated Show resolved Hide resolved
cypress/support/commands.ts Show resolved Hide resolved
@richard-cox richard-cox force-pushed the e2e-create-rke1 branch 2 times, most recently from c024ae5 to c2460e9 Compare June 23, 2023 16:31
@mantis-toboggan-md
Copy link
Member

The code looks fine aside from linting errors, but I think e2e ci tests wont pass until a new v2.7-head rancher/rancher image is available. Since we're running our e2e rancher docker instance with -e CATTLE_UI_OFFLINE_PREFERRED=true we need the bundled ember code to include your changes here right? That was merged 3 days ago but the latest v2.7-head rancher/rancher image is 9 days old.

@richard-cox
Copy link
Member Author

richard-cox commented Jun 27, 2023

Hadn't managed to get around to debugging this, good find! We need to bring in the latest rancher/ui build and serve it up as per the new way we build and serve the dashboard. I've created #9217 to cover this. Otherwise the last RC build will be used

@richard-cox richard-cox merged commit 6bf3ab7 into rancher:master Jun 27, 2023
8 checks passed
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.

Automation: RKE1 provisioning member roles' "custom role" modal checkbox don't work correctly
2 participants