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

zedmanager: Parse the snapshot-related configuration. #3105

Merged

Conversation

OhmSpectator
Copy link
Member

In the scope of the PR, I introduce the structures to store snap-related configurations coming from the controller.
Handling of the data and the population of the other data structures are coming in the further PRs.

@OhmSpectator OhmSpectator force-pushed the feature/snapshot-parse-config branch from b6c669f to 4005af8 Compare March 16, 2023 15:28
@OhmSpectator
Copy link
Member Author

Fix the Yetus errors.

Add the corresponding structures to zedmanagertypes.go and parse the values
coming from the controller.

Signed-off-by: Nikolay Martyanov <nikolay@zededa.com>
@OhmSpectator OhmSpectator force-pushed the feature/snapshot-parse-config branch from 4005af8 to 1624e4f Compare March 16, 2023 16:49
@OhmSpectator
Copy link
Member Author

@rouming @eriknordmark I would be happy if you could look at the PR. I would like to avoid it just hanging for a while...

Copy link
Contributor

@eriknordmark eriknordmark left a comment

Choose a reason for hiding this comment

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

One nit in a separate comment.
Kicking off the tests.

@eriknordmark eriknordmark merged commit 9e6a639 into lf-edge:master Mar 22, 2023
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.

2 participants