Skip to content

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

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

Merged
merged 2 commits into from
Mar 7, 2024

Conversation

ritalwar
Copy link
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

🚀 Benchmarks report

To see the full report comment with /test benchmark fullreport

@elasticmachine
Copy link

💚 Build Succeeded

cc @ritalwar

Copy link

@@ -16,7 +16,7 @@ streams:
- name: client_secret
type: password
title: Client Secret
secret: true
secret: false
Copy link
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
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
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
Contributor

LGTM for reverting the secrets for older stack version.

@ritalwar ritalwar requested a review from ishleenk17 March 7, 2024 07:35
Copy link
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

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

@elasticmachine
Copy link

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

@elasticmachine
Copy link

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

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

@elasticmachine
Copy link

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

@elasticmachine
Copy link

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

@elasticmachine
Copy link

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

@elasticmachine
Copy link

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

@elasticmachine
Copy link

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

@elasticmachine
Copy link

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

@elasticmachine
Copy link

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

@elasticmachine
Copy link

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

@elasticmachine
Copy link

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

@elasticmachine
Copy link

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

@elasticmachine
Copy link

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

@elasticmachine
Copy link

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

@elasticmachine
Copy link

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

@elasticmachine
Copy link

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

@elasticmachine
Copy link

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