Skip to content

Update kubernetes templates for elastic-agent#138260

Closed
elasticmachine wants to merge 2 commits intomainfrom
update-k8s-templates-20220808130309
Closed

Update kubernetes templates for elastic-agent#138260
elasticmachine wants to merge 2 commits intomainfrom
update-k8s-templates-20220808130309

Conversation

@elasticmachine
Copy link
Copy Markdown
Contributor

@elasticmachine elasticmachine requested a review from a team as a code owner August 8, 2022 13:03
@botelastic botelastic bot added the Team:Fleet Team label for Observability Data Collection Fleet team label Aug 8, 2022
@cla-checker-service
Copy link
Copy Markdown

❌ Author of the following commits did not sign a Contributor Agreement:
e5b5df8

Please, read and sign the above mentioned agreement if you want to contribute to this project

@elasticmachine
Copy link
Copy Markdown
Contributor Author

Pinging @elastic/fleet (Team:Fleet)

@kibana-ci
Copy link
Copy Markdown

💚 Build Succeeded

Metrics [docs]

✅ unchanged

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

# If FLEET_ENROLLMENT_TOKEN is empty then KIBANA_HOST, KIBANA_FLEET_USERNAME, KIBANA_FLEET_PASSWORD are needed
- name: FLEET_ENROLLMENT_TOKEN
value: "token-id"
value: ""
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Looks like it's needed as we replace this by the ENROLLMENT token in the instructions

Suggested change
value: ""
value: "token-id"

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

oh nice let me change this

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

value: "changeme"
value: ""
# The Elasticsearch host to communicate with
- name: ES_HOST
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

just wondering if this variable wasn't skipped on purpose - for standalone mode ES_HOST will get rendered to the actual value in configmap

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@gizas gizas closed this Aug 9, 2022
@kpollich kpollich deleted the update-k8s-templates-20220808130309 branch July 30, 2024 13:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

automation Team:Fleet Team label for Observability Data Collection Fleet team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants