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

Revert "chore(deps): update ec" #1510

Merged
merged 1 commit into from
Oct 15, 2024
Merged

Revert "chore(deps): update ec" #1510

merged 1 commit into from
Oct 15, 2024

Conversation

zregvart
Copy link
Member

Reverts #1460

@chmeliik
Copy link
Contributor

Could you give some more context about why the revert is needed?

@zregvart
Copy link
Member Author

Could you give some more context about why the revert is needed?

The commit/description is a bit terse as I just used the revert option on GitHub

We had report that the EC started failing with:

rego_type_error: package annotation redeclared

Seems that we pass in the same Rego file twice to the OPA engine and this causes a compile error. Still investigating why has this started to happen and why it wasn't caught by the tests.

@zregvart
Copy link
Member Author

/retest

@mmorhun mmorhun disabled auto-merge October 15, 2024 13:53
@mmorhun mmorhun merged commit a4a367d into main Oct 15, 2024
13 of 14 checks passed
@mmorhun mmorhun deleted the revert-1460-renovate/ec branch October 15, 2024 13:53
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.

4 participants