Skip to content

Allow to run Elastic stack with other licenses #2501

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 10 commits into from
Apr 2, 2025

Conversation

mrodm
Copy link
Contributor

@mrodm mrodm commented Mar 31, 2025

This PR adds support into elastic-package to run an Elastic stack with a different subscription.
By default it keeps the trial subscription. And in this PR it is added support to run an Elastic stack with basic subscription.

Author's Checklist

  • Update documentation
  • Add test package running with basic subscription (copied from apache)
  • Add steps in CI to test stack with basic subscription

How to test this locally

# Test with profile parameter basic
elastic-package stack up -v -d -U stack.elastic_subscription=basic
## it must show the subscription type basic
curl -k -s -u elastic:changeme https://localhost:9200/_license | jq -r '.license.type'

elastic-package stack down -v

# Test with profile parameter trial
elastic-package stack up -v -d -U stack.elastic_subscription=trial
## it must show the subscription type trial
curl -k -s -u elastic:changeme https://localhost:9200/_license | jq -r '.license.type'

elastic-package stack down -v

# Test with default value
elastic-package stack up -v -d
## it must show the subscription type trial
curl -k -s -u elastic:changeme https://localhost:9200/_license | jq -r '.license.type'

@mrodm mrodm self-assigned this Mar 31, 2025
@@ -22,6 +22,8 @@ STACK_COMMAND_TESTS=(
test-stack-command-8x
test-stack-command-9x
test-stack-command-with-apm-server
test-stack-command-with-basic-license
test-stack-command-with-self-monitor
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This was added as a target in the Makefile but it was missing in the script that triggers the steps.

@mrodm mrodm marked this pull request as ready for review March 31, 2025 17:48
@mrodm mrodm requested a review from a team as a code owner March 31, 2025 17:48
Copy link
Member

@jsoriano jsoriano left a comment

Choose a reason for hiding this comment

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

👍

@mrodm
Copy link
Contributor Author

mrodm commented Apr 2, 2025

Updated license references to be subscription instead in 3bdf337

@elasticmachine
Copy link
Collaborator

💚 Build Succeeded

History

cc @mrodm

@mrodm mrodm merged commit 4c38619 into elastic:main Apr 2, 2025
3 checks passed
@mrodm mrodm deleted the allow_set_other_license branch April 2, 2025 11:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants