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

Update Dockerfile for cypress pipeline run on harvester-ui-test repository #5

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

TachunLin
Copy link
Collaborator

@TachunLin TachunLin commented Jan 26, 2025

Which issue(s) this PR fixes:

Issue:

What this PR does / why we need it:

  • We have create the new harvester-ui-tests repository and move all existing cypress/tests code to it.

  • After we adjust the existing harvester-cypress-tests and harvester-cypress-run-tests pipeline in the harvester-baremetal-ansible repository.

  • In order for the cypress jenkins pipeline to get the correct default folder path in the harvester-ui-tests repository

  • We need to adjust the build and copy path in the cypress/Dockerfile

Fix and Improvements

  1. Update the WORKDIR and COPY command path to fit the new harvester-ui-tests structure
...

WORKDIR /harvester-ui-tests

COPY . /harvester-ui-tests

...

Test Result

Verified can well execute cypress_test and cypress_run_test pipeline well on the staging Jenkins
And can also correctly generate and display HTML report on the job page

Additional context

Need to test with PR https://github.com/harvester/harvester-baremetal-ansible/pull/95/

@TachunLin TachunLin assigned albinsun and a110605 and unassigned albinsun and a110605 Jan 26, 2025
@TachunLin TachunLin requested review from albinsun and a110605 January 26, 2025 15:25
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.

3 participants