Skip to content

[v7r3] Python 3 hackathon fixes#5246

Merged
atsareg merged 8 commits into
DIRACGrid:integrationfrom
TaykYoku:integration_2
Jul 13, 2021
Merged

[v7r3] Python 3 hackathon fixes#5246
atsareg merged 8 commits into
DIRACGrid:integrationfrom
TaykYoku:integration_2

Conversation

@TaykYoku

@TaykYoku TaykYoku commented Jul 1, 2021

Copy link
Copy Markdown
Contributor

BEGINRELEASENOTES
Python 3 hackathon fixes.

*Core
CHANGE: modify MySQL._to_string

*WorkloadManager
FIX: decode values using _to_string method in JobDB
FIX: mock _escapeString in Test_Agent_PilotStatusAgent to pass test
FIX: escape values in PilotAgentsDB
CHANGE: remove pilots before start Test_PilotsClient test

ENDRELEASENOTES

@TaykYoku TaykYoku requested review from atsareg and fstagni as code owners July 1, 2021 10:02
Comment thread src/DIRAC/WorkloadManagementSystem/Service/WMSAdministratorHandler.py Outdated
@TaykYoku TaykYoku closed this Jul 4, 2021
@TaykYoku

TaykYoku commented Jul 4, 2021

Copy link
Copy Markdown
Contributor Author

restart tests

@TaykYoku TaykYoku reopened this Jul 4, 2021
@TaykYoku TaykYoku closed this Jul 4, 2021
@TaykYoku

TaykYoku commented Jul 4, 2021

Copy link
Copy Markdown
Contributor Author

re-test

@TaykYoku TaykYoku reopened this Jul 4, 2021
@TaykYoku TaykYoku closed this Jul 4, 2021
@TaykYoku

TaykYoku commented Jul 4, 2021

Copy link
Copy Markdown
Contributor Author

re-test

@TaykYoku TaykYoku reopened this Jul 4, 2021
@TaykYoku TaykYoku changed the title [integration] fix Failed to get info for pilot MySQL Error [v7r3] Python 3 hackathon fixes Jul 4, 2021
@TaykYoku TaykYoku mentioned this pull request Jul 5, 2021
@TaykYoku

TaykYoku commented Jul 5, 2021

Copy link
Copy Markdown
Contributor Author

rebased and ready for review.

NOTE: PR fixes do not affect the database structure, IMHO it can be considered separately from issue.

fstagni
fstagni previously approved these changes Jul 5, 2021
Comment thread src/DIRAC/Core/Utilities/MySQL.py Outdated
@atsareg atsareg merged commit 3f5ec52 into DIRACGrid:integration Jul 13, 2021
@TaykYoku TaykYoku deleted the integration_2 branch September 6, 2021 13:33
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.

4 participants