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

RP E2E Test Execution #3856

Draft
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

shubhadapaithankar
Copy link
Collaborator

Which issue this PR addresses:

Fixes: ARO-9501

What this PR does / why we need it:

This PR introduces an E2E testing stage using Docker Compose to orchestrate the RP, Selenium, and E2E test services. The pipeline pulls the required RP image from Azure Container Registry (ACR) and ensures the correct environment is set up for the E2E tests. It helps validate the end-to-end functionality of the system and ensures that tests are running using containerized RP images.

  • Why we need it: The E2E stage ensures that the entire system is tested in a environment, using containerized components (including RP) and validating workflows through Selenium for browser-based testing.
  • This change helps automate the test execution process for the complete system, ensuring that every aspect of the application functions correctly in a containerized environment before deployment.

Test plan for issue:

  1. E2E tests are executed within the pipeline's new E2E stage using Docker Compose.
  2. The tests are triggered automatically after the Containerized stage, ensuring seamless orchestration.

Is there any documentation that needs to be updated for this PR?

N/A

How do you know this will function as expected in production?

N/A

@shubhadapaithankar shubhadapaithankar force-pushed the shubhadapaithankar/ARO-9501-RP-E2E-Test branch 13 times, most recently from d3f4b1e to f24c223 Compare October 9, 2024 15:54
@github-actions github-actions bot added the needs-rebase branch needs a rebase label Oct 10, 2024
Copy link

Please rebase pull request.

@shubhadapaithankar shubhadapaithankar force-pushed the shubhadapaithankar/ARO-9501-RP-E2E-Test branch from f24c223 to d968eaa Compare October 11, 2024 18:42
@github-actions github-actions bot removed the needs-rebase branch needs a rebase label Oct 11, 2024
@shubhadapaithankar shubhadapaithankar force-pushed the shubhadapaithankar/ARO-9501-RP-E2E-Test branch 8 times, most recently from edc458b to 75bb79b Compare October 11, 2024 22:53
Copy link
Contributor

@lranjbar lranjbar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Found a typo and a couple of nits. This looks good to start the functional testing

.pipelines/ci.yml Outdated Show resolved Hide resolved
.pipelines/ci.yml Outdated Show resolved Hide resolved
.pipelines/ci.yml Outdated Show resolved Hide resolved
@shubhadapaithankar shubhadapaithankar force-pushed the shubhadapaithankar/ARO-9501-RP-E2E-Test branch from 75bb79b to a43c9ee Compare October 11, 2024 23:10
@shubhadapaithankar shubhadapaithankar force-pushed the shubhadapaithankar/ARO-9501-RP-E2E-Test branch from a43c9ee to 1d29b6f Compare October 11, 2024 23:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants