Store access token missing#15089
Merged
ph merged 3 commits intoelastic:fleetfrom Dec 16, 2019
ph:store-access-token-missing
Merged
Store access token missing#15089ph merged 3 commits intoelastic:fleetfrom ph:store-access-token-missing
ph merged 3 commits intoelastic:fleetfrom
ph:store-access-token-missing
Conversation
This PR add a few things
- It takes the keystore encryption implementation and create an
io.Reader and io.Writer for it, we currently work with variable lenght
blocks and we are not compatible for now with the one in libbeat.
- It implements a few storage mechanism and wrapper:
- DiskStore: Save a io.reader directly to disk, the content is saved
in a temporary file and the target is replace in an atomic
operation.
- EncryptedDiskStore: Same as Keystore but use the encrypted io.Reader
and io.Writer.
- A fleet configuration is created in _meta/agent.fleet.yml, the content
of this file is packed in the binary. When a user enroll the agent,
we do a backup of the current agent.yml and the content is replaced
with the agent.fleet.yml
- The enrollment information is saved into a "fleet.yml"
- The managed mode now read the content of the fleet.yml and creates a
Kibana API client.
- A Separates Fleet Config struct is created.
- Enroll will now ask for confirmation before replacing the user
configuration.
Contributor
|
Pinging @elastic/ingest (Project:fleet) |
Contributor
Author
|
Have you seen this error before CI? |
Contributor
|
seems like a permission problem. I've seen this happening in an incorrectly setup VM, root privileges are missing so you can |
michalpristas
approved these changes
Dec 16, 2019
Contributor
michalpristas
left a comment
michalpristas
left a comment
There was a problem hiding this comment.
small nits. this includes this missing PR right?
Contributor
Author
|
Weird, I can reproduce locally it in a freshly reset environment |
Contributor
|
check if you are the owner of a directory |
Contributor
|
@ph if you fix |
Contributor
Author
|
@michalpristas Just pushed the commits for the previous comments, let's wait for the CI. 🤞 |
leweafan
pushed a commit
to leweafan/beats
that referenced
this pull request
Apr 28, 2023
* Feature: Allow to persist AccessToken and other connection data on disk
This PR add a few things
- It takes the keystore encryption implementation and create an
io.Reader and io.Writer for it, we currently work with variable lenght
blocks and we are not compatible for now with the one in libbeat.
- It implements a few storage mechanism and wrapper:
- DiskStore: Save a io.reader directly to disk, the content is saved
in a temporary file and the target is replace in an atomic
operation.
- EncryptedDiskStore: Same as Keystore but use the encrypted io.Reader
and io.Writer.
- A fleet configuration is created in _meta/agent.fleet.yml, the content
of this file is packed in the binary. When a user enroll the agent,
we do a backup of the current agent.yml and the content is replaced
with the agent.fleet.yml
- The enrollment information is saved into a "fleet.yml"
- The managed mode now read the content of the fleet.yml and creates a
Kibana API client.
- A Separates Fleet Config struct is created.
- Enroll will now ask for confirmation before replacing the user
configuration.
* allow to use the obfuscated store.
* fix build
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This PR add a few things
ref: #14951