Skip to content

Backport PR #17124 to 8.x: plugin manager: add --no-expand flag for list command#17170

Merged
yaauie merged 1 commit into8.xfrom
backport_17124_8.x
Feb 27, 2025
Merged

Backport PR #17124 to 8.x: plugin manager: add --no-expand flag for list command#17170
yaauie merged 1 commit into8.xfrom
backport_17124_8.x

Conversation

@github-actions
Copy link
Copy Markdown
Contributor

Backport PR #17124 to 8.x branch, original message:


Release notes

  • The plugin manager bin/logstash-plugin's list command now supports a --no-expand flag to avoid expanding plugin aliases and the individual plugins within integration plugins.

What does this PR do?

Adds --[no-]expand flag (default: true) to the plugin manager's list command, so that a caller can avoid expanding the integration plugins and aliases to get only a list of the plugin packages installed.

Why is it important/What is the impact to the user?

Checklist

  • My code follows the style guidelines of this project
  • I have commented my code, particularly in hard-to-understand areas
  • [ ] I have made corresponding changes to the documentation
  • [] I have made corresponding change to the default configuration files (and/or docker env variables)
  • I have added tests that prove my fix is effective or that my feature works

Author's Checklist

  • file doc issue

How to test this PR locally

bin/logstash-plugin list --no-expand
bin/logstash-plugin list --verbose --no-expand
bin/logstash-plugin list --no-expand logstash-integration-aws
bin/logstash-plugin list --no-expand --verbose logstash-integration-aws

* plugin manager: add --no-expand flag for list command

Allows us to avoid expanding aliases and integration plugins

* spec: escape expected output in regexp

(cherry picked from commit 793e8c0)
@elastic-sonarqube
Copy link
Copy Markdown

Quality Gate passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarQube

@elasticmachine
Copy link
Copy Markdown

💛 Build succeeded, but was flaky

Failed CI Steps

Copy link
Copy Markdown
Member

@yaauie yaauie left a comment

Choose a reason for hiding this comment

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

✅ Clean and Green 🎉

@yaauie yaauie merged commit 772ce89 into 8.x Feb 27, 2025
@yaauie yaauie deleted the backport_17124_8.x branch February 27, 2025 22:33
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.

2 participants