Skip to content
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

Remove technical preview for independent Elastic Agents #1972

Merged
merged 3 commits into from
Jul 12, 2024

Conversation

mrodm
Copy link
Contributor

@mrodm mrodm commented Jul 12, 2024

Follows #1959

As independent Elastic Agents were set the default mode running system tests, this PR removes the technical preview for the log messages and docs.

@mrodm mrodm requested a review from a team July 12, 2024 11:15
@mrodm mrodm self-assigned this Jul 12, 2024
@@ -599,6 +600,83 @@ Finally, when you are done running all system tests, bring down the Elastic Stac
elastic-package stack down
```

Starting with [elastic-package version `v0.101.2`](https://github.com/elastic/elastic-package/releases/tag/v0.101.2),
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Include all this information about independent Elastic Agents under the section Running system test.

A new section has been added to explain how to run system tests using the Elastic Agent from the stack (as it was performed before).

- If a package that defines the agent service deployer (`agent` folder) wants to migrate to this mode of running Elastic Agents, these would be the steps:
- Create a new `_dev/deploy/docker` adding the service container if needed.
- Define the settings required for your Elastic Agents in all the test configuration files.

#### Running system tests in parallel (technical preview)
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Should we remove this technical preview too ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

As it is running for a couple packages in the integrations repo, probably it could be kept like this (as technical preview).

Copy link
Collaborator

Choose a reason for hiding this comment

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

Leave it as tech preview for now.

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.

Thanks for adding the docs! I think it would be more accurate to reference 0.101.0, where most of the relevant changes are, than 0.101.2.

docs/howto/system_testing.md Outdated Show resolved Hide resolved
docs/howto/system_testing.md Outdated Show resolved Hide resolved
docs/howto/system_testing.md Outdated Show resolved Hide resolved
Co-authored-by: Jaime Soriano Pastor <[email protected]>
@mrodm mrodm enabled auto-merge (squash) July 12, 2024 15:23
@mrodm mrodm merged commit 20d9bd9 into elastic:main Jul 12, 2024
3 checks passed
@elasticmachine
Copy link
Collaborator

💚 Build Succeeded

History

cc @mrodm

@mrodm mrodm deleted the remove-technical-preview-independent-agents branch July 12, 2024 15:58
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.

4 participants