Skip to content

Conversation

@enggnr
Copy link
Contributor

@enggnr enggnr commented Jun 28, 2023

  • What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
    This PR provides a script that conditionally enrolls elastic agent with Elastic fleet server

  • What is the current behavior? (You can also link to an open issue here)
    Elastic agent is not automatically enrolled

  • What is the new behavior (if this is a feature change)?
    Fixes Add device with elastic-agent #26

@github-actions
Copy link

github-actions bot commented Jun 28, 2023

🤖 OpenAI


Chat with 🤖 OpenAI Bot (@openai)

  • Reply on review comments left by this bot to ask follow-up questions. A review comment is a comment on a diff or a file.
  • Invite the bot into a review comment chain by tagging @openai in a reply.

Code suggestions

  • The bot may make code suggestions, but please review them carefully before committing since the line number ranges may be misaligned.
  • You can edit the comment made by the bot and manually tweak the suggestion if it is slightly off.

Ignoring further reviews

  • Type @openai: ignore anywhere in the PR description to ignore further reviews from the bot.

Files not summarized due to errors (2)

Failed to summarize

  • home/.chezmoi.yaml.tmpl (nothing obtained from openai)
  • home/.chezmoiscripts/universal/run_onchange_after_57-elastic-agent.sh.tmpl (nothing obtained from openai)

In the recent run, only the files that changed from the base of the PR and between 2dfb30a3b8eb3da1335bc095df468fca9934c8a8 and 06c0429dc596cfc40db8f5062080cb8051dcac96 commits were reviewed.

Files not reviewed due to errors in the recent run (2)

Failed to review in the last run

  • home/.chezmoi.yaml.tmpl (no response)
  • home/.chezmoiscripts/universal/run_onchange_after_57-elastic-agent.sh.tmpl (no response)

Copy link
Contributor

@ProfessorManhattan ProfessorManhattan left a comment

Choose a reason for hiding this comment

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

Couple requests

Copy link
Contributor

@ProfessorManhattan ProfessorManhattan left a comment

Choose a reason for hiding this comment

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

Please address the comments on this one

Copy link
Contributor

@ProfessorManhattan ProfessorManhattan left a comment

Choose a reason for hiding this comment

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

Please install Elastic Agent as well.

Copy link
Contributor Author

@enggnr enggnr left a comment

Choose a reason for hiding this comment

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

Review comments incorporated

Copy link
Contributor Author

@enggnr enggnr left a comment

Choose a reason for hiding this comment

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

Looks like I didn't push the changes, should be good now.

Copy link
Contributor

@ProfessorManhattan ProfessorManhattan left a comment

Choose a reason for hiding this comment

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

Please update this to use SRV records / auto-discovery like glusterfs / etcd

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.

Add device with elastic-agent

3 participants