Skip to content

[v7r2] Integration merged with v7r1#4661

Merged
atsareg merged 58 commits into
DIRACGrid:integrationfrom
atsareg:integration-merge-v7r1
Jun 29, 2020
Merged

[v7r2] Integration merged with v7r1#4661
atsareg merged 58 commits into
DIRACGrid:integrationfrom
atsareg:integration-merge-v7r1

Conversation

@atsareg

@atsareg atsareg commented Jun 29, 2020

Copy link
Copy Markdown
Contributor

No description provided.

chaen and others added 30 commits June 12, 2020 17:53
indentation was off for some, causing failure to properly markup
Co-authored-by: fstagni <federico.stagni@cern.ch>
The previous split would drop things after an underscore in the installed components name (e.g., Bdii2CSAgent_GLUE2)
allows calling the script with e.g., -ddd
[v6r22] Print error if LFN list is missing
[v7r0] Change the closing of socket in M2Crypto
[v7r0] Cancel FTS3 Operation if the RMS request does not exist
@atsareg atsareg requested review from chaen and fstagni as code owners June 29, 2020 14:13
- pytest --no-cov
- DIRAC_USE_M2CRYPTO=Yes DIRAC_M2CRYPTO_SPLIT_HANDSHAKE=Yes pytest --no-cov
# Security tests are flakey due to reference counting bugs in pyGSI/src/crypto/asn1.c
- pytest Core/Security/test || (echo "Retrying..."; pytest Core/Security/test) || (echo "Retrying again..."; pytest Core/Security/test)

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

The repeated running was added for failures due to the pyGSI library, so it should probably stay with the repetition, but adding --no-cov?

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

As this is targetting integration the repeated runs can be removed as pyGSI was removed in #4481

@atsareg atsareg merged commit 6ac2bb8 into DIRACGrid:integration Jun 29, 2020
@atsareg atsareg deleted the integration-merge-v7r1 branch January 26, 2021 13:11
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.

7 participants