-
Notifications
You must be signed in to change notification settings - Fork 172
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] Added Pilot tests to integration tests #7570
[9.0] Added Pilot tests to integration tests #7570
Conversation
0ec0bfe
to
01b975f
Compare
23df04a
to
7fc80b5
Compare
7fc80b5
to
ed5ebbd
Compare
9a01fa2
to
0b5e7d3
Compare
0b5e7d3
to
78fbe56
Compare
assert list(result["Value"].values())[0] == { | ||
"Owner": "userName", | ||
"Jobs": 2, | ||
"OwnerGroup": "myGroup", | ||
"CPUTime": 86400, | ||
"Priority": 1.0, | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why did you remove this assert
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Because, since with this PR some jobs can be submitted and some pilots run, we can't with certainty know we can expect to see in the DB.
This PR adds a new "pilot" integration test. It basically runs the pilot into yet another container, and runs jobs. In a way, this completes the "full grid" test.
BEGINRELEASENOTES
*test
NEW: added pilot workflow tests to integration_tests
ENDRELEASENOTES