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

[8.x](backport #6437) fix(6023): replace list with items #6459

Merged
merged 3 commits into from
Jan 2, 2025

Conversation

mergify[bot]
Copy link
Contributor

@mergify mergify bot commented Dec 31, 2024

  • Bug

What does this PR do?

Replaces list with items from from kibanaFetchToken as the list is deprecated in the api response and will be removed.

Why is it important?

The list in the api response is going to be removed in favor of item

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
  • I have added tests that prove my fix is effective or that my feature works
  • I have added an entry in ./changelog/fragments using the changelog tool
  • I have added an integration test or an E2E test

Disruptive User Impact

There shouldn't be any user impact

How to test this PR locally

  • Deploy ESS
  • Build the agent by running the following command
SNAPSHOT=true PLATFORMS="linux/arm64" PACKAGES="docker" EXTERNAL=true mage -v package
  • Then get the image id for any of the created elastic-agent images and run the following command
docker run \
  --env FLEET_ENROLL=1 \
  --env KIBANA_HOST=<KIBANA HOST> \
  --env KIBANA_USERNAME=<KIBANA USER NAME> \
  --env KIBANA_PASSWORD=<KIBANA PASSWORD> \
  --env FLEET_TOKEN_POLICY_NAME=<POLICY NAME> \
  --env FLEET_URL=<FLEET URL> \
  --rm <IMAGE ID>
  • Validate that the agent is able to enroll

Related issues


This is an automatic backport of pull request #6437 done by [Mergify](https://mergify.com).

* fix(6023): replace list with items

* fix(6023): added changelog

* fix(6023): added test validating the new items matches the deprecated list

* fix(6023): added comment to better trace where the integration test comes from

The integration test added in this pull request needs to be removed once the list component is removed from the kibana api

* fixs(6023): ran mage update

* fix(6023): add build tag to integration test

* fix(6023): added unit test for kibanaFetchToken

* fix(6023): added error checks in unit test

(cherry picked from commit 68d5833)
@mergify mergify bot added the backport label Dec 31, 2024
@mergify mergify bot requested a review from a team as a code owner December 31, 2024 13:57
@mergify mergify bot requested review from andrzej-stencel and pchila and removed request for a team December 31, 2024 13:57
@jlind23 jlind23 enabled auto-merge (squash) December 31, 2024 13:58
@jlind23 jlind23 merged commit 1f89c72 into 8.x Jan 2, 2025
13 checks passed
@jlind23 jlind23 deleted the mergify/bp/8.x/pr-6437 branch January 2, 2025 12:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants