Skip to content

Enable secret for the identified fields in multiple packages - #9009

Merged
ritalwar merged 19 commits into
elastic:mainfrom
ritalwar:secret_migration_1305
Feb 13, 2024
Merged

ritalwar merged 19 commits into
elastic:mainfrom
ritalwar:secret_migration_1305

Conversation

@ritalwar

@ritalwar ritalwar commented Jan 30, 2024 •

Copy link
Copy Markdown
Contributor

Proposed commit message

  • Enable secret for identified sensitive fields.
  • Update package-spec to 3.0.2 for most packages to utilize the validation checks for secrets.
  • Omit the package-spec update to 3.0.2 for the aws package due to additional validation check involving dynamic mappings for nested fields in the waf datastream.
  • Update Kibana version condition to 8.12 for packages on versions 8.10-8.11, as 8.12 resolves all secret-related issues.
  • For packages < 8.10.0, retain existing Kibana version. Changelog descriptions for these packages indicate that secrets are supported from version 8.12, allowing users to upgrade stack at their discretion.

Checklist

  • I have reviewed tips for building integrations and this pull request is aligned with them.
  • I have verified that all data streams collect metrics or logs.
  • I have added an entry to my package's changelog.yml file.
  • I have verified that Kibana version constraints are current according to guidelines.

Author's Checklist

How to test this PR locally

Related issues

Screenshots

@ritalwar ritalwar added the enhancement New feature or request label Jan 30, 2024
@ritalwar ritalwar self-assigned this Jan 30, 2024
@elasticmachine

elasticmachine commented Jan 30, 2024 •

Copy link
Copy Markdown

🚀 Benchmarks report

To see the full report comment with /test benchmark fullreport

@ritalwar
ritalwar marked this pull request as ready for review February 7, 2024 10:23
@ritalwar
ritalwar requested review from a team as code owners February 7, 2024 10:23
Comment thread packages/salesforce/manifest.yml Resolved

@harnish-crest-data harnish-crest-data left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! 🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants