Skip to content

Conversation

@jsoriano
Copy link
Member

Policy templates ignore namespaces when they are default.

This was originally done in #2784, but updatecli overwrote and close it.

@jsoriano jsoriano self-assigned this Jul 31, 2025
@jsoriano jsoriano requested a review from a team as a code owner July 31, 2025 10:45
@jsoriano jsoriano changed the title Updatecli main bump elastic stack default version Update default stack version to 9.1.0. Jul 31, 2025
@jsoriano jsoriano changed the title Update default stack version to 9.1.0. Update default stack version to 9.1.0 Jul 31, 2025
@jsoriano
Copy link
Member Author

/test integrations

@jsoriano
Copy link
Member Author

The failure in sql package is related to a fix included in this change: elastic/kibana#214216, and may affect other input packages.

Testing with integrations to assess what to do. I think it would be good to validate that these fields are in the policy, as they should. So if it doesn't break many packages I would prefer to update the test files than adding more exceptions.

@github-actions github-actions bot force-pushed the updatecli_main_bump-elastic-stack-default-version branch from 79f86b2 to ce4c437 Compare August 1, 2025 01:29
@mrodm
Copy link
Contributor

mrodm commented Aug 1, 2025

test integrations

@elastic-vault-github-plugin-prod

Created or updated PR in integrations repository to test this version. Check elastic/integrations#14768

@mrodm
Copy link
Contributor

mrodm commented Aug 1, 2025

The failure in sql package is related to a fix included in this change: elastic/kibana#214216, and may affect other input packages.

Testing with integrations to assess what to do. I think it would be good to validate that these fields are in the policy, as they should. So if it doesn't break many packages I would prefer to update the test files than adding more exceptions.

Currently, in the integrations repository just one integration package defines policy tests:

 $ find packages -type d -name policy | grep "_dev/test/policy"
packages/auditd_manager/data_stream/auditd/_dev/test/policy

Would this issue raise just for input packages and with 9.1.0 onwards?

@jsoriano
Copy link
Member Author

jsoriano commented Aug 1, 2025

Would this issue raise just for input packages and with 9.1.0 onwards?

Yes.

I am going to remove these fields at the end. As it is now, this is going to cause issues for packages supporting versions < 9.1 and >= 9.1.

Done in 20b0fcc.

@elasticmachine
Copy link
Collaborator

💚 Build Succeeded

History

cc @jsoriano

@jsoriano jsoriano merged commit ebe2c40 into elastic:updatecli_main_bump-elastic-stack-default-version Aug 1, 2025
3 checks passed
@jsoriano jsoriano deleted the updatecli_main_bump-elastic-stack-default-version branch August 1, 2025 16:08
@jsoriano
Copy link
Member Author

jsoriano commented Aug 4, 2025

Trying yet another time in #2804.

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.

3 participants