Skip to content

Disable secrets for packages with older stack versions due to errors.#9279

Merged
ritalwar merged 2 commits intoelastic:mainfrom
ritalwar:disable_secrets_1305
Mar 7, 2024
Merged

Disable secrets for packages with older stack versions due to errors.#9279
ritalwar merged 2 commits intoelastic:mainfrom
ritalwar:disable_secrets_1305

Conversation

@ritalwar
Copy link
Copy Markdown
Contributor

@ritalwar ritalwar commented Mar 6, 2024

Proposed commit message

This PR disables the secrets for packages on versions below 8.10.0 due to errors. More details in comment section here.

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.

Related issues

@ritalwar ritalwar added the bugfix Pull request that fixes a bug issue label Mar 6, 2024
@ritalwar ritalwar self-assigned this Mar 6, 2024
@ritalwar ritalwar requested a review from a team as a code owner March 6, 2024 14:24
@ali786XI ali786XI self-requested a review March 6, 2024 14:31
@elasticmachine
Copy link
Copy Markdown

🚀 Benchmarks report

To see the full report comment with /test benchmark fullreport

@elasticmachine
Copy link
Copy Markdown

💚 Build Succeeded

cc @ritalwar

@elastic-sonarqube
Copy link
Copy Markdown

type: password
title: Client Secret
secret: true
secret: false
Copy link
Copy Markdown
Member

@ishleenk17 ishleenk17 Mar 7, 2024

Choose a reason for hiding this comment

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

I hope the removal of secrets and setting secrets:false doesn't have any difference.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Actually, that's the correct approach because we're using package format 3.0.2. If we don't set secret to false, we'll get build errors.

Copy link
Copy Markdown
Contributor

@ali786XI ali786XI left a comment

Choose a reason for hiding this comment

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

@ritalwar I tried loading the integrations with default configs in 8.11.0 and 8.10.1 versions and I didn't face any errors loading the packages present in this PR. LGTM for the changes !!

cc:- @lalit-satapathy

@lalit-satapathy
Copy link
Copy Markdown
Contributor

LGTM for reverting the secrets for older stack version.

@ritalwar ritalwar requested a review from ishleenk17 March 7, 2024 07:35
Copy link
Copy Markdown
Member

@ishleenk17 ishleenk17 left a comment

Choose a reason for hiding this comment

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

Looks good!

@ritalwar ritalwar merged commit af6092f into elastic:main Mar 7, 2024
@elasticmachine
Copy link
Copy Markdown

Package activemq - 1.1.1 containing this change is available at https://epr.elastic.co/search?package=activemq

@elasticmachine
Copy link
Copy Markdown

Package apache_tomcat - 1.3.1 containing this change is available at https://epr.elastic.co/search?package=apache_tomcat

@elasticmachine
Copy link
Copy Markdown

Package azure_app_service - 0.2.1 containing this change is available at https://epr.elastic.co/search?package=azure_app_service

@elasticmachine
Copy link
Copy Markdown

Package azure_billing - 1.4.2 containing this change is available at https://epr.elastic.co/search?package=azure_billing

@elasticmachine
Copy link
Copy Markdown

Package azure_functions - 0.3.1 containing this change is available at https://epr.elastic.co/search?package=azure_functions

@elasticmachine
Copy link
Copy Markdown

Package cassandra - 1.11.1 containing this change is available at https://epr.elastic.co/search?package=cassandra

@elasticmachine
Copy link
Copy Markdown

Package ceph - 1.3.1 containing this change is available at https://epr.elastic.co/search?package=ceph

@elasticmachine
Copy link
Copy Markdown

Package citrix_adc - 1.3.1 containing this change is available at https://epr.elastic.co/search?package=citrix_adc

@elasticmachine
Copy link
Copy Markdown

Package cockroachdb - 1.8.1 containing this change is available at https://epr.elastic.co/search?package=cockroachdb

@elasticmachine
Copy link
Copy Markdown

Package golang - 1.3.1 containing this change is available at https://epr.elastic.co/search?package=golang

@elasticmachine
Copy link
Copy Markdown

Package kafka - 1.12.1 containing this change is available at https://epr.elastic.co/search?package=kafka

@elasticmachine
Copy link
Copy Markdown

Package kafka_log - 1.5.1 containing this change is available at https://epr.elastic.co/search?package=kafka_log

@elasticmachine
Copy link
Copy Markdown

Package mysql - 1.18.2 containing this change is available at https://epr.elastic.co/search?package=mysql

@elasticmachine
Copy link
Copy Markdown

Package nginx - 1.19.1 containing this change is available at https://epr.elastic.co/search?package=nginx

@elasticmachine
Copy link
Copy Markdown

Package oracle_weblogic - 1.4.1 containing this change is available at https://epr.elastic.co/search?package=oracle_weblogic

@elasticmachine
Copy link
Copy Markdown

Package rabbitmq - 1.12.1 containing this change is available at https://epr.elastic.co/search?package=rabbitmq

@elasticmachine
Copy link
Copy Markdown

Package salesforce - 0.13.1 containing this change is available at https://epr.elastic.co/search?package=salesforce

@elasticmachine
Copy link
Copy Markdown

Package spring_boot - 1.3.2 containing this change is available at https://epr.elastic.co/search?package=spring_boot

@elasticmachine
Copy link
Copy Markdown

Package websphere_application_server - 1.2.1 containing this change is available at https://epr.elastic.co/search?package=websphere_application_server

gizas pushed a commit that referenced this pull request Mar 13, 2024
…#9279)

* Disable secrets for packages with older stack versions due to errors.
qcorporation pushed a commit that referenced this pull request Feb 3, 2025
…#9279)

* Disable secrets for packages with older stack versions due to errors.
qcorporation pushed a commit that referenced this pull request Feb 4, 2025
…#9279)

* Disable secrets for packages with older stack versions due to errors.
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.

6 participants