Skip to content

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
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

3 participants