GPII-2165: Adding NP sets for manual MM QA#491
GPII-2165: Adding NP sets for manual MM QA#491kavyakomuru wants to merge 2 commits intoGPII:masterfrom
Conversation
* GPII-1884: GPII-1884: Updated provisioning/requirements.yml to use the master branches of gpii-ops/ansible-preferences-server and gpii-ops/ansible-flow-manager with the pull requests on those repos for fixing GPII-1884 have been merged.
|
Could one of the admins verify that these changes are reasonable to test? If so, please reply with "ok to test". |
|
ok to test |
|
CI job passed. |
|
CI job failed. Please visit http://lists.gpii.net/pipermail/ci/ for more details. |
|
CI job passed. |
|
Could one of the admins verify that these changes are reasonable to test? If so, please reply with "ok to test". |
|
ok to test |
|
CI job passed. |
|
Hi @kavyakomuru Sorry for being so slow at reviewing this pull request.. I think it generally looks good and pretty much ready to go - a few comments though:
|
|
Ugh, just realized that the preferences server doesn't support json5 files right now... This is something that we want to support in the future but it hasn't happened yet (https://issues.gpii.net/browse/GPII-502 ) Could you move the comments out to *.md files with the same name as the NP sets (e.g. MMQAUserA.md) and rename all the NP sets you've added to .json |
kaspermarkus
left a comment
There was a problem hiding this comment.
Added some comments that should be addressed before this can be merged
No description provided.