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 the Smoke test PR check for Dashboard repository #22045

Closed
dmytro-ndp opened this issue Mar 7, 2023 · 0 comments
Closed

Add the Smoke test PR check for Dashboard repository #22045

dmytro-ndp opened this issue Mar 7, 2023 · 0 comments
Assignees
Labels
area/qe area/quality-tiers kind/task Internal things, technical debt, and to-do tasks to be performed. severity/P1 Has a major impact to usage or development of the system. team/A This team is responsible for the Che Operator and all its operands as well as chectl and Hosted Che

Comments

@dmytro-ndp
Copy link
Contributor

dmytro-ndp commented Mar 7, 2023

Is your task related to a problem? Please describe

Run Smoke GUI test on new pull request or new commit to pull request in repo https://github.com/eclipse-che/che-dashboard

Describe the solution you'd like

Create GitHub actions on pull request events to do:

  1. Wait on successful build of GitHub action "docker-build" [e.g. https://github.com/eclipse-che/che-dashboard/actions/runs/4952254169/jobs/8858567817)
  2. Deploy OpenShift instance on Prow CI.
  3. Install Eclipse Che next using dashboard image from PR by patching CheCluster [1], e.g.:
apiVersion: org.eclipse.che/v2
spec:
  components:
    dashboard:
      dashboardImage: '$CI_CHE_DASHBOARD_IMAGE'

CI_CHE_DASHBOARD_IMAGE = quay.io/eclipse-che/che-dashboard:pr-<PR_NUMBER>

  1. Run smoke GUI test against Eclipse Che instance.
  2. Collect the logs and shutdown OpenShift instance.

[1] https://github.com/eclipse-che/che-dashboard/blob/main/.ci/oci-dashboard-happy-path.sh#L24-L29

Technical debts:

  1. Get rid of "test-all-devfiles" command: https://github.com/eclipse/che/blob/main/tests/e2e/package.json#L13
  2. Replace redundant outdated constant TS_SELENIUM_USERNAME with TS_SELENIUM_OCP_USERNAME
  3. Replace redundant outdated constant TS_SELENIUM_PASSWORD with TS_SELENIUM_OCP_PASSWORD

Describe alternatives you've considered

No response

Additional context

No response

@dmytro-ndp dmytro-ndp added kind/task Internal things, technical debt, and to-do tasks to be performed. severity/P1 Has a major impact to usage or development of the system. area/qe team/A This team is responsible for the Che Operator and all its operands as well as chectl and Hosted Che labels Mar 7, 2023
@ibuziuk ibuziuk mentioned this issue Mar 22, 2023
50 tasks
@ibuziuk ibuziuk moved this to 📋 Backlog in Eclipse Che Team A Backlog Apr 5, 2023
@ibuziuk ibuziuk moved this from 📋 Backlog to 🚧 In Progress in Eclipse Che Team A Backlog Apr 5, 2023
@ibuziuk ibuziuk moved this from 🚧 In Progress to Ready for Review in Eclipse Che Team A Backlog Jul 11, 2023
@ScrewTSW ScrewTSW moved this from Ready for Review to ✅ Done in Eclipse Che Team A Backlog Jul 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/qe area/quality-tiers kind/task Internal things, technical debt, and to-do tasks to be performed. severity/P1 Has a major impact to usage or development of the system. team/A This team is responsible for the Che Operator and all its operands as well as chectl and Hosted Che
Projects
None yet
Development

No branches or pull requests

2 participants