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

Feature sitelists update workqueue elements fix 12038 #1

Open
wants to merge 44 commits into
base: feature_SeteWhitelist_SupportChangeInReqmgr2_fix-12307
Choose a base branch
from

Conversation

todor-ivanov
Copy link
Owner

Fixes #None

Status

Just for visualization purposes

Description

To spot the exact diffs between dmwm#12120 and dmwm#12038 before the former gets into master

Is it backward compatible (if not, which system it affects?)

YES

Related PRs

dmwm#12120

External dependencies / deployment changes

None

amaltaro and others added 30 commits October 16, 2024 22:55
Update parameter names to reflect WQE

Nested workqueue structure

Escape dots in the field name

Escape dots in the right field

Escape dots in AgentStatusPoller as well

Load JSON file with replications in AgentStatusPoller

Fix location of json filter; correct WQ filter reference

Remove component configuration; perform safe print for replication
Addition of a new method to StageOutImpl to log details about failing gfal commands
Replace WorkQueue replication filter by selector function
Signed-off-by: Alan Malta Rodrigues <[email protected]>
Do not set ParentQueueUrl in the selector filter
unit test with all possible fallbacks

unit test: WMCORE_ROOT has double install dir now
Move filter replication json document under etc/
Replace xquery with query, due to deprecation of the former since HTCondor 10.7.0
…roxySilentFailure_fix-12173

Fix str to bytes comparisons in checkProxy.py
Signed-off-by: Alan Malta Rodrigues <[email protected]>
Signed-off-by: Dario Mapelli <[email protected]>
…_SupportChangeInReqmgr2_fix-12307

Separate state transition validation from nonState transition valdations
Integrate packate attestations in the CI/CD pipeline
Signed-off-by: Dario Mapelli <[email protected]>
Grant attestation write access - CI/CD pipeline
Signed-off-by: Dario Mapelli <[email protected]>
Convert reusable to non-reusable GitHub Action
Signed-off-by: Dario Mapelli <[email protected]>
amaltaro and others added 12 commits November 22, 2024 21:10
…bInitFail_fix-12166

Fix delayed $WMA_DEPLOY_DIR env virable
Moving the X509 variables setup at deployment time

Fix broken path for RUCIO_HOME && Add missing ORACLE_PATH
Signed-off-by: Todor Ivanov <[email protected]>
Fix T1 site cores ResourceControl logic
…bSpam_2_fix-12184

Add back non running components to restartComponent
Module to update site lists for WMAgents
@todor-ivanov todor-ivanov force-pushed the feature_Sitelists_UpdateWorkqueueElements_fix-12038 branch 4 times, most recently from c83b52f to 93d0c3b Compare December 12, 2024 18:27
…eters

Add a method in the WorkQueue service for updating all wf elements with all parameters in a sinigle go

Pylint fixes

Remove WQE status filter

Set a WQE status filter as: 'Available', 'Negotiating', 'Acquired'

Handling assignment-approved arguments differently to avoid code duplication

Restore unhandled arguments mechanism

Add back lost else condition to the workload update exception handling

Fix undefined var in error message.

Forgotten raise from None

Call reqMgr api only once && Preserve RequestStatus in the workload.

Add arguments validation for partial workflow parameters update

Use imported ALLOWED_STAT_KEYS during validation

Validate full set of arguments instead of only reqargsDiff

Skip validating update stat arguments

Update workload args only once

Fix bad updateElementsByWorkflow argument

Be explicit in the check for stat update arguments during validation

Review Comments

Pylint fixes
Unit tests

Unit tests
@todor-ivanov todor-ivanov force-pushed the feature_Sitelists_UpdateWorkqueueElements_fix-12038 branch from 382bebc to b6f75ed Compare December 19, 2024 07:57
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.

6 participants