Skip to content

GPII-2165: Adding NP sets for manual MM QA#491

Open
kavyakomuru wants to merge 2 commits intoGPII:masterfrom
kaspermarkus:GPII-2165
Open

GPII-2165: Adding NP sets for manual MM QA#491
kavyakomuru wants to merge 2 commits intoGPII:masterfrom
kaspermarkus:GPII-2165

Conversation

@kavyakomuru
Copy link
Copy Markdown

No description provided.

amb26 and others added 2 commits November 30, 2016 14:18
* 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.
@gpii-bot
Copy link
Copy Markdown

Could one of the admins verify that these changes are reasonable to test? If so, please reply with "ok to test".

@javihernandez
Copy link
Copy Markdown
Member

ok to test

@gpii-bot
Copy link
Copy Markdown

gpii-bot commented Dec 1, 2016

CI job passed.

@gpii-bot
Copy link
Copy Markdown

gpii-bot commented Dec 9, 2016

CI job failed. Please visit http://lists.gpii.net/pipermail/ci/ for more details.

@gpii-bot
Copy link
Copy Markdown

gpii-bot commented Dec 9, 2016

CI job passed.

@gpii-bot
Copy link
Copy Markdown

Could one of the admins verify that these changes are reasonable to test? If so, please reply with "ok to test".

@gtirloni
Copy link
Copy Markdown
Contributor

gtirloni commented Feb 3, 2017

ok to test

@gpii-bot
Copy link
Copy Markdown

gpii-bot commented Feb 3, 2017

CI job passed.

@kaspermarkus kaspermarkus changed the title Gpii 2165 GPII-2165: Adding NP sets for manual MM QA Dec 5, 2017
@kaspermarkus kaspermarkus self-requested a review December 5, 2017 10:00
@kaspermarkus
Copy link
Copy Markdown
Member

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:

  • I think it would be ok for now to drop all these directly into the "preferences" folder, since I think it'll be a while before we get the functionality to search recursively through the preferences folder. It's up to you, but if you want to avoid having to copy them over each time you do the QA, I think it would be ok to move them out of the QA folder
  • The description in each NP set is great, only suggestion is to put a link to the MM QA test plan in each of the NP set, so one can get a clearer context of what they're used for

@kaspermarkus
Copy link
Copy Markdown
Member

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

Copy link
Copy Markdown
Member

@kaspermarkus kaspermarkus left a comment

Choose a reason for hiding this comment

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

Added some comments that should be addressed before this can be merged

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