Skip to content

Conversation

@zk2k2
Copy link
Contributor

@zk2k2 zk2k2 commented Jul 31, 2025

Summary

Note: This PR replaces PR #227684 to fix a CLA issue (commit was made under the wrong e-mail)

Fixes:

This PR fixes a bug where internal ingest pipelines used by the cloud_security_posture plugin were not marked as managed.
By adding the _meta.managed: true field to these pipelines, they will now be correctly recognized as managed pipelines by Elasticsearch and Kibana.

This prevents accidental user modifications and aligns with other Elastic-managed pipelines.


Checklist

  • Follows EUI writing guidelines and includes i18n support where applicable
  • Documentation updated as needed
  • Unit or functional tests updated or added
  • Checked for breaking HTTP API changes
  • Flaky Test Runner used on tests changed
  • Release Notes section included and appropriate label applied

Identify risks

This change only adds metadata to existing ingest pipeline definitions and does not affect pipeline logic or data processing.
Risk of regressions or data loss is minimal. The change improves system stability by marking pipelines as managed.


@zk2k2 zk2k2 requested a review from a team as a code owner July 31, 2025 16:57
@maxcold maxcold added release_note:skip Skip the PR/issue when compiling release notes backport:skip This PR does not require backporting Team:Cloud Security Cloud Security team related labels Aug 1, 2025
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-cloud-security-posture (Team:Cloud Security)

@maxcold
Copy link
Contributor

maxcold commented Aug 1, 2025

/ci

@maxcold
Copy link
Contributor

maxcold commented Aug 1, 2025

@elasticmachine run docs-build

@maxcold
Copy link
Contributor

maxcold commented Aug 1, 2025

/ci

@maxcold maxcold enabled auto-merge (squash) August 4, 2025 15:19
@maxcold
Copy link
Contributor

maxcold commented Aug 4, 2025

/ci

@maxcold
Copy link
Contributor

maxcold commented Aug 4, 2025

@elasticmachine run docs-build

@maxcold maxcold merged commit 8bbf5da into elastic:main Aug 4, 2025
12 checks passed
@elasticmachine
Copy link
Contributor

💛 Build succeeded, but was flaky

Failed CI Steps

Test Failures

  • [job] [logs] FTR Configs #66 / ESQL execution logic API @ess @serverless ES|QL rule type, alert suppression with exceptions should apply exceptions

Metrics [docs]

✅ unchanged

History

szaffarano pushed a commit to szaffarano/kibana that referenced this pull request Aug 5, 2025
## Summary

Note: This PR replaces PR elastic#227684
to fix a CLA issue (commit was made under the wrong e-mail)

Fixes: 
- elastic#226696

This PR fixes a bug where internal ingest pipelines used by the
cloud_security_posture plugin were not marked as managed.
By adding the `_meta.managed: true` field to these pipelines, they will
now be correctly recognized as managed pipelines by Elasticsearch and
Kibana.

This prevents accidental user modifications and aligns with other
Elastic-managed pipelines.

---

### Checklist

- [x] Follows EUI writing guidelines and includes i18n support where
applicable
- [ ] Documentation updated as needed  
- [ ] Unit or functional tests updated or added  
- [ ] Checked for breaking HTTP API changes  
- [ ] Flaky Test Runner used on tests changed  
- [ ] Release Notes section included and appropriate label applied  

---

### Identify risks

This change only adds metadata to existing ingest pipeline definitions
and does not affect pipeline logic or data processing.
Risk of regressions or data loss is minimal. The change improves system
stability by marking pipelines as managed.

---

---------

Co-authored-by: kibanamachine <42973632+kibanamachine@users.noreply.github.com>
delanni pushed a commit to delanni/kibana that referenced this pull request Aug 5, 2025
## Summary

Note: This PR replaces PR elastic#227684
to fix a CLA issue (commit was made under the wrong e-mail)

Fixes: 
- elastic#226696

This PR fixes a bug where internal ingest pipelines used by the
cloud_security_posture plugin were not marked as managed.
By adding the `_meta.managed: true` field to these pipelines, they will
now be correctly recognized as managed pipelines by Elasticsearch and
Kibana.

This prevents accidental user modifications and aligns with other
Elastic-managed pipelines.

---

### Checklist

- [x] Follows EUI writing guidelines and includes i18n support where
applicable
- [ ] Documentation updated as needed  
- [ ] Unit or functional tests updated or added  
- [ ] Checked for breaking HTTP API changes  
- [ ] Flaky Test Runner used on tests changed  
- [ ] Release Notes section included and appropriate label applied  

---

### Identify risks

This change only adds metadata to existing ingest pipeline definitions
and does not affect pipeline logic or data processing.
Risk of regressions or data loss is minimal. The change improves system
stability by marking pipelines as managed.

---

---------

Co-authored-by: kibanamachine <42973632+kibanamachine@users.noreply.github.com>
@wildemat wildemat mentioned this pull request Aug 7, 2025
10 tasks
NicholasPeretti pushed a commit to NicholasPeretti/kibana that referenced this pull request Aug 18, 2025
## Summary

Note: This PR replaces PR elastic#227684
to fix a CLA issue (commit was made under the wrong e-mail)

Fixes: 
- elastic#226696

This PR fixes a bug where internal ingest pipelines used by the
cloud_security_posture plugin were not marked as managed.
By adding the `_meta.managed: true` field to these pipelines, they will
now be correctly recognized as managed pipelines by Elasticsearch and
Kibana.

This prevents accidental user modifications and aligns with other
Elastic-managed pipelines.

---

### Checklist

- [x] Follows EUI writing guidelines and includes i18n support where
applicable
- [ ] Documentation updated as needed  
- [ ] Unit or functional tests updated or added  
- [ ] Checked for breaking HTTP API changes  
- [ ] Flaky Test Runner used on tests changed  
- [ ] Release Notes section included and appropriate label applied  

---

### Identify risks

This change only adds metadata to existing ingest pipeline definitions
and does not affect pipeline logic or data processing.
Risk of regressions or data loss is minimal. The change improves system
stability by marking pipelines as managed.

---

---------

Co-authored-by: kibanamachine <42973632+kibanamachine@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport:skip This PR does not require backporting 💝community release_note:skip Skip the PR/issue when compiling release notes Team:Cloud Security Cloud Security team related v9.2.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants