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

Remove generation validation from Pipelines Test #1955

Merged
merged 1 commit into from
Oct 23, 2024

Conversation

diegolovison
Copy link
Contributor

@diegolovison diegolovison commented Oct 22, 2024

Remove generation validation from Pipelines Test

That step is not needed

Tested with: rhoai-test-flow/967

Copy link
Contributor

github-actions bot commented Oct 22, 2024

Robot Results

✅ Passed ❌ Failed ⏭️ Skipped Total Pass %
550 0 0 550 100

@diegolovison diegolovison force-pushed the remove_test branch 3 times, most recently from 027e786 to 5401b3b Compare October 23, 2024 13:20
@diegolovison diegolovison added the verified This PR has been tested with Jenkins label Oct 23, 2024
@diegolovison diegolovison marked this pull request as ready for review October 23, 2024 13:23
Copy link

sonarcloud bot commented Oct 23, 2024

@@ -63,10 +63,6 @@ Create PipelineServer Using Custom DSPA
... ${assert_install}=${TRUE} ${configure_pip_index}=${TRUE}
Copy link
Contributor

Choose a reason for hiding this comment

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

maybe this argument is not needed anymore now? could you remove it in case?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@diegolovison diegolovison merged commit 8529b6e into red-hat-data-services:master Oct 23, 2024
8 checks passed
tonyxrmdavidson pushed a commit to tonyxrmdavidson/ods-ci that referenced this pull request Oct 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
verified This PR has been tested with Jenkins
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants