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

[cmd/opampsupervisor]: Write agent config and logs to storage directory #34348

Conversation

BinaryFissionGames
Copy link
Contributor

*Description:

  • Writes the agent's effective.yaml and agent.log to the storage directory.

Link to tracking Issue: Closes #34341

Testing:

  • Added an e2e test testing that the supervisor writes the files to the correct location

@BinaryFissionGames BinaryFissionGames changed the title [cmd/opampsupervisor]: Write agent config and logs to [cmd/opampsupervisor]: Write agent config and logs to sotrage directory Aug 1, 2024
@evan-bradley evan-bradley changed the title [cmd/opampsupervisor]: Write agent config and logs to sotrage directory [cmd/opampsupervisor]: Write agent config and logs to storage directory Aug 1, 2024
@evan-bradley
Copy link
Contributor

@BinaryFissionGames Can you resolve the conflicts?

@BinaryFissionGames BinaryFissionGames force-pushed the fix/supervisor-write-files-to-storage branch from 3a5916c to 13e6e80 Compare August 6, 2024 17:56
@BinaryFissionGames
Copy link
Contributor Author

@evan-bradley Should be good to go!

@evan-bradley evan-bradley merged commit 35f218e into open-telemetry:main Aug 6, 2024
154 checks passed
@github-actions github-actions bot added this to the next release milestone Aug 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

supervisor does not consistently use property "storage.directory" to store effective yaml, agent.log
4 participants