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

Fix dsp tests on interop 2.17 & fix variable conflict in Permissions provoking xpath issue #2219

Merged

Conversation

jgarciao
Copy link
Contributor

@jgarciao jgarciao commented Feb 7, 2025

  • When creating a project in DSP tests, set the as_user parameter for compatibility with the interop pipeline
  • Modify variable name in Permissions.resource to fix an xpath error due to variable name conflict

Currently being tested in rhoai-test-flow/2751

Needed when running the tests with the interop pipeline

Signed-off-by: Jorge Garcia Oncins <jgarciao@redhat.com>
`${SAVE_BUTTON}` is also defined in ODHDashboardSettings.resource that
value was used, provoking an xpath error when setting permissions

Signed-off-by: Jorge Garcia Oncins <jgarciao@redhat.com>
Copy link

sonarqubecloud bot commented Feb 7, 2025

@jgarciao jgarciao requested a review from antowaddle February 7, 2025 14:03
@jgarciao jgarciao added the enhancements Bugfixes, enhancements, refactoring, ... in tests or libraries (PR will be listed in release-notes) label Feb 7, 2025
@jgarciao jgarciao requested a review from jiripetrlik February 7, 2025 14:03
Copy link

openshift-ci bot commented Feb 7, 2025

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: bdattoma, jgarciao, jiripetrlik

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@jgarciao jgarciao changed the title Fix dsp tests on interop 2.17 & fix xpath issue when setting Permissions Fix dsp tests on interop 2.17 & fix variable conflict in Permissions provoking xpath issue Feb 7, 2025
@jgarciao jgarciao merged commit 7132d95 into red-hat-data-services:master Feb 7, 2025
11 of 12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancements Bugfixes, enhancements, refactoring, ... in tests or libraries (PR will be listed in release-notes) lgtm
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants