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

[8.16] [Automatic Import] Cel template fixes (#196945) #196978

Merged
merged 2 commits into from
Oct 21, 2024

Conversation

kibanamachine
Copy link
Contributor

Backport

This will backport the following commits from main to 8.16:

Questions ?

Please refer to the Backport tool documentation

## Summary

This PR corrects the CEL template so that it can successfully complete
package policy installation. The fixes are 1) adding a template path,
and 2) removing the common template values from the cel templates.

Tested and verified with and without the CEL generation FF enabled.

(cherry picked from commit 15390e4)
@elasticmachine
Copy link
Contributor

elasticmachine commented Oct 19, 2024

💔 Build Failed

Failed CI Steps

Metrics [docs]

✅ unchanged

History

cc @kgeller

@kibanamachine kibanamachine merged commit b2c151a into elastic:8.16 Oct 21, 2024
26 checks passed
kibanamachine added a commit that referenced this pull request Oct 23, 2024
# Backport

This will backport the following commits from `main` to `8.x`:
- [[Automatic Import] Cel template fixes
(#196945)](#196945)

<!--- Backport version: 9.4.3 -->

### Questions ?
Please refer to the [Backport tool
documentation](https://github.com/sqren/backport)

<!--BACKPORT [{"author":{"name":"Kylie
Meli","email":"[email protected]"},"sourceCommit":{"committedDate":"2024-10-19T13:56:55Z","message":"[Automatic
Import] Cel template fixes (#196945)\n\n## Summary\r\n\r\nThis PR
corrects the CEL template so that it can successfully
complete\r\npackage policy installation. The fixes are 1) adding a
template path,\r\nand 2) removing the common template values from the
cel templates.\r\n\r\nTested and verified with and without the CEL
generation FF
enabled.","sha":"15390e4cdb9aecca5ba5782b0081cb244dbd1a08","branchLabelMapping":{"^v9.0.0$":"main","^v8.17.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","v9.0.0","backport:prev-minor","v8.16.0","Team:Security-Scalability","backport:version","v8.17.0"],"title":"[Automatic
Import] Cel template
fixes","number":196945,"url":"https://github.com/elastic/kibana/pull/196945","mergeCommit":{"message":"[Automatic
Import] Cel template fixes (#196945)\n\n## Summary\r\n\r\nThis PR
corrects the CEL template so that it can successfully
complete\r\npackage policy installation. The fixes are 1) adding a
template path,\r\nand 2) removing the common template values from the
cel templates.\r\n\r\nTested and verified with and without the CEL
generation FF
enabled.","sha":"15390e4cdb9aecca5ba5782b0081cb244dbd1a08"}},"sourceBranch":"main","suggestedTargetBranches":["8.x"],"targetPullRequestStates":[{"branch":"main","label":"v9.0.0","branchLabelMappingKey":"^v9.0.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/196945","number":196945,"mergeCommit":{"message":"[Automatic
Import] Cel template fixes (#196945)\n\n## Summary\r\n\r\nThis PR
corrects the CEL template so that it can successfully
complete\r\npackage policy installation. The fixes are 1) adding a
template path,\r\nand 2) removing the common template values from the
cel templates.\r\n\r\nTested and verified with and without the CEL
generation FF
enabled.","sha":"15390e4cdb9aecca5ba5782b0081cb244dbd1a08"}},{"branch":"8.16","label":"v8.16.0","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"url":"https://github.com/elastic/kibana/pull/196978","number":196978,"state":"MERGED","mergeCommit":{"sha":"b2c151a12f4db817e7aeb2ed7f76fe2217f82c2c","message":"[8.16]
[Automatic Import] Cel template fixes (#196945) (#196978)\n\n#
Backport\n\nThis will backport the following commits from `main` to
`8.16`:\n- [[Automatic Import] Cel template
fixes\n(#196945)](https://github.com/elastic/kibana/pull/196945)\n\n<!---
Backport version: 9.4.3 -->\n\n### Questions ?\nPlease refer to the
[Backport
tool\ndocumentation](https://github.com/sqren/backport)\n\n<!--BACKPORT
[{\"author\":{\"name\":\"Kylie\nMeli\",\"email\":\"[email protected]\"},\"sourceCommit\":{\"committedDate\":\"2024-10-19T13:56:55Z\",\"message\":\"[Automatic\nImport]
Cel template fixes (#196945)\\n\\n## Summary\\r\\n\\r\\nThis
PR\ncorrects the CEL template so that it can
successfully\ncomplete\\r\\npackage policy installation. The fixes are
1) adding a\ntemplate path,\\r\\nand 2) removing the common template
values from the\ncel templates.\\r\\n\\r\\nTested and verified with and
without the CEL\ngeneration
FF\nenabled.\",\"sha\":\"15390e4cdb9aecca5ba5782b0081cb244dbd1a08\",\"branchLabelMapping\":{\"^v9.0.0$\":\"main\",\"^v8.17.0$\":\"8.x\",\"^v(\\\\d+).(\\\\d+).\\\\d+$\":\"$1.$2\"}},\"sourcePullRequest\":{\"labels\":[\"release_note:skip\",\"v9.0.0\",\"v8.16.0\",\"Team:Security-Scalability\",\"backport:version\"],\"title\":\"[Automatic\nImport]
Cel
template\nfixes\",\"number\":196945,\"url\":\"https://github.com/elastic/kibana/pull/196945\",\"mergeCommit\":{\"message\":\"[Automatic\nImport]
Cel template fixes (#196945)\\n\\n## Summary\\r\\n\\r\\nThis
PR\ncorrects the CEL template so that it can
successfully\ncomplete\\r\\npackage policy installation. The fixes are
1) adding a\ntemplate path,\\r\\nand 2) removing the common template
values from the\ncel templates.\\r\\n\\r\\nTested and verified with and
without the CEL\ngeneration
FF\nenabled.\",\"sha\":\"15390e4cdb9aecca5ba5782b0081cb244dbd1a08\"}},\"sourceBranch\":\"main\",\"suggestedTargetBranches\":[\"8.16\"],\"targetPullRequestStates\":[{\"branch\":\"main\",\"label\":\"v9.0.0\",\"branchLabelMappingKey\":\"^v9.0.0$\",\"isSourceBranch\":true,\"state\":\"MERGED\",\"url\":\"https://github.com/elastic/kibana/pull/196945\",\"number\":196945,\"mergeCommit\":{\"message\":\"[Automatic\nImport]
Cel template fixes (#196945)\\n\\n## Summary\\r\\n\\r\\nThis
PR\ncorrects the CEL template so that it can
successfully\ncomplete\\r\\npackage policy installation. The fixes are
1) adding a\ntemplate path,\\r\\nand 2) removing the common template
values from the\ncel templates.\\r\\n\\r\\nTested and verified with and
without the CEL\ngeneration
FF\nenabled.\",\"sha\":\"15390e4cdb9aecca5ba5782b0081cb244dbd1a08\"}},{\"branch\":\"8.16\",\"label\":\"v8.16.0\",\"branchLabelMappingKey\":\"^v(\\\\d+).(\\\\d+).\\\\d+$\",\"isSourceBranch\":false,\"state\":\"NOT_CREATED\"}]}]\nBACKPORT-->\n\nCo-authored-by:
Kylie Meli
<[email protected]>"}},{"branch":"8.x","label":"v8.17.0","branchLabelMappingKey":"^v8.17.0$","isSourceBranch":false,"state":"NOT_CREATED"}]}]
BACKPORT-->

Co-authored-by: Kylie Meli <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants