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.

Sign up for GitHub

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

Jump to bottom

[8.1] Integration test fixes for DiracX #7231

Merged
merged 5 commits into from
Oct 9, 2023
Merged

[8.1] Integration test fixes for DiracX #7231

merged 5 commits into from
Oct 9, 2023

Conversation

chrisburr
Copy link
Member

This contains changes that are required for DIRACGrid/diracx#127. In particular:

  • Assign a VO to all groups in the CS when running the integration tests
  • Use the DiracX CS conversion CLI instead of the script in @chaen's GitLab area
  • Follow the change of API route naming (/ -> /api/)
  • Disable the /api/jobs/ router for now as it isn't needed at the moment and requires S3-like storage to be configured

@chrisburr chrisburr force-pushed the fix-diracx-int-tests branch from 59d1489 to bf90cdc Compare October 9, 2023 08:32
@fstagni fstagni merged commit 4e2fbd9 into DIRACGrid:integration Oct 9, 2023
24 checks passed
@DIRACGridBot DIRACGridBot added the sweep:ignore Prevent sweeping from being ran for this PR label Oct 9, 2023
@chrisburr chrisburr deleted the fix-diracx-int-tests branch October 9, 2023 09:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@fstagni fstagni fstagni approved these changes

@atsareg atsareg Awaiting requested review from atsareg atsareg is a code owner

Assignees
No one assigned
Labels
sweep:ignore Prevent sweeping from being ran for this PR
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@chrisburr @fstagni @DIRACGridBot