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

[9.0] test: add submit and match test #7516

Closed
wants to merge 1 commit into from

Conversation

fstagni
Copy link
Contributor

@fstagni fstagni commented Mar 15, 2024

This PR adds a new github action workflow, that submits a job (to lbcertifdirac70) and then runs a pilot that tries to match a job. Few notes:

  • uses github actions secrets
  • retrieves a user proxy for submitting and running the job
  • the job is simplest "echo Hello World" JDL (no dirac-jobexec)
  • submits to and runs at site "DIRAC.CI.org"
  • the client used for submitting the job and the DIRAC version installed by the pilot are of course a pip-install of the branch under test

@fstagni fstagni requested a review from atsareg as a code owner March 15, 2024 14:56
@fstagni fstagni closed this Mar 15, 2024
@fstagni fstagni reopened this Mar 15, 2024
@fstagni fstagni force-pushed the submitAndMatch2 branch 2 times, most recently from e8530a9 to 017df2e Compare March 18, 2024 13:49
Comment on lines +5 to +7
branches:
- 'integration'
- 'rel-v8r0'
Copy link
Contributor Author

Choose a reason for hiding this comment

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

By adding these branches, we are basically restricting the run of this workflow only after the merge is made.
If a developer wants to run this, these branches should be removed, and the secrets added to the fork.

@fstagni
Copy link
Contributor Author

fstagni commented Apr 16, 2024

Will be replaced by an integration test similar to #7570

@fstagni fstagni closed this Apr 16, 2024
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.

2 participants