Skip to content

Conversation

Robin-Van-de-Merghel
Copy link
Contributor

BEGINRELEASENOTES

Please follow the template:
*Integration tests
NEW: Add pilot secrets into the CI

ENDRELEASENOTES

@Robin-Van-de-Merghel Robin-Van-de-Merghel force-pushed the robin-pilot-registrations branch from 7efe647 to 9f3cefd Compare August 7, 2025 06:56
@Robin-Van-de-Merghel
Copy link
Contributor Author

This PR serves as a POC for the pilot secret-exchange mechanism. We have two cases:

  1. Pilot with a secret (vacuum case with secret-exchange: DiracX)
  2. Pilot with a proxy (vacuum case with RegisterPilot: DIRAC)

With secret

Source: https://github.com/DIRACGrid/DIRAC/actions/runs/16797372818/job/47570725488?pr=8279#step:12:192

As expected we simulate a vacuum case: first exchange failed because the pilot is not registered, second exchange success because it's a vacuum registration.

Maybe I can add a test to show a classic secret-exchange without vacuum case

Then RegisterPilot is blocked because we are already registered through the secret-exchange: https://github.com/DIRACGrid/DIRAC/actions/runs/16797372818/job/47570725488?pr=8279#step:12:1880

Without secret

Source: https://github.com/DIRACGrid/DIRAC/actions/runs/16797372818/job/47570725469?pr=8279#step:12:185

As expected, continue without DiracX, tried to get the token but failed.

Then RegisterPilot is a success because we don't have a secret-exchange: https://github.com/DIRACGrid/DIRAC/actions/runs/16797372818/job/47570725469?pr=8279#step:12:1871

Refs

Linked to:

@Robin-Van-de-Merghel Robin-Van-de-Merghel marked this pull request as ready for review August 7, 2025 07:35
@Robin-Van-de-Merghel Robin-Van-de-Merghel changed the title Add secrets in DIRAC integration tests [9.1?] Add secrets in DIRAC integration tests Aug 7, 2025
@Robin-Van-de-Merghel Robin-Van-de-Merghel force-pushed the robin-pilot-registrations branch from 9f3cefd to 8a3ef42 Compare August 7, 2025 13:28
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.

1 participant