Skip to content
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

OPA Policy Not Running When Using Nested Abstract Components #404

Open
hypnotisttom opened this issue Jul 5, 2023 · 2 comments
Open

OPA Policy Not Running When Using Nested Abstract Components #404

hypnotisttom opened this issue Jul 5, 2023 · 2 comments
Labels
bug 🐛 An issue with the system

Comments

@hypnotisttom
Copy link

Describe the Bug

When I add an OPA policy to a component that has nested abstract inheritance, the OPA policy isn't running.

Expected Behavior

I would expect "atmos validate" etc. to run the policy.

Steps to Reproduce

1, Create 3 abstract components, each inheriting from the previous one
2. Create a real component that inherits from the last abstracted component
3. Place OPA policy settings/validation on any of the components
4. Run atmos validate

Note that simply inheriting the very first abstract component into a real component will work. But when the abstract components inherit from each other, an OPA policy doesn't run when validating a real component that inherits from such abstracts, even if the OPA policy is on the first abstract component.

Screenshots

No response

Environment

No response

Additional Context

No response

@aknysh
Copy link
Member

aknysh commented Jul 14, 2023

@hypnotisttom thank you for testing and opening the issue.

We tested the scenario that you described, and added the stack config and tests to this Atmos release https://github.com/cloudposse/atmos/releases/tag/v1.41.0
Please review the release description and examples.

I'd say what you described is working in our tests.

I guess you might have a different configuration.
You can send your config to me (by email or Slack), and I'll take a look.

Thank you

@hypnotisttom
Copy link
Author

Thanks @aknysh. I'll more than likely be sending an email this week since this is probably due to our specific implementation. You'll hear from me soon. I appreciate your time.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug 🐛 An issue with the system
Projects
None yet
Development

No branches or pull requests

2 participants