parse group config on groupable nodes

[MichelleArk]

resolves #6823

Description

Checklist

• I have read the contributing guide and understand what's expected of me
• I have signed the CLA
• I have run this code in development and it appears to resolve the stated issue
• This PR includes tests, or tests are not required/relevant for this PR
• I have opened an issue to add/update docs, or docs changes are not required/relevant for this PR
• I have run changie new to create a changelog entry

[github-actions]

Thank you for your pull request! We could not find a changelog entry for this change. For details on how to document a change, see the contributing guide.

[gshank]

This looks good so far. I do have a couple of questions-- Are we going to "promote" the group config to node level in update_node_config? Are generic tests actually going to be able to be assigned to groups? I'm wondering if it might not make more sense for generic tests to belong to the group of the node they're attached to.

[jtcohen6]

I'm wondering if it might not make more sense for generic tests to belong to the group of the node they're attached to.

Agreed! This also makes sense given ref restrictions we're putting in place in #6826

[aranke]

Few comments to make this more Pythonic

[aranke]

Discussed offline, keeping this pattern for now.

[MichelleArk]

@gshank - Could you please take another look? Made some changes since the last review:

• attached node lookup via ref_lookup and disabled_lookup: 0712181
• added group_map to manifest for scheduling nodes in deleted groups for partial parsing: 43b8d09

[gshank]

Looks good!

I see what you meant about combining the ref_lookup and disabled. This actually feels like a somewhat better solution than in other places where it's two separate lookups.