Work-around for Policy API Missing OrgUnit in Settings (0.4.1) release #605
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
🗣 Description
This contains the fix for the Policy API not including the orgunit with the policy setting in some rare cases. Google is looking into this issue, but this change will isolate us in the meantime. The basic rule for the policy settings is that the top-level orgunit has all the settings and any subunit or group will only have settings that deviate from the top-level orgunit's settings. It's appropriate to associate any setting without an orgunit with the top-level orgunit.
This also comes with the version update from 0.4.0 to 0.4.1.
Closes #599
🧪 Testing
Manual testing by faking out a setting without an orgunit. The setting gets the top-level orgunit.
The smoke test does not run successfully as of when this PR was created, because the meet 6.* baseline was added in a prior commit in the main branch and there's no corresponding Rego implementation.
✅ Pre-approval checklist
✅ Pre-merge Checklist
Squash and merge
button.✅ Post-merge Checklist