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

Initial draft for config files #424

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

HorjuRares
Copy link
Contributor

Issues: #302

Definition of Done

The PR shall be merged only if all items mentioned in CONTRIBUTING.md have been followed. In case an item is not applicable as described, please provide a short explanation in the description.

Copy link
Contributor

@windsource windsource left a comment

Choose a reason for hiding this comment

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

I have added my comments.

agent/config/ank-agent.conf Outdated Show resolved Hide resolved
Comment on lines 18 to 19
# An existing path where to manage the fifo files.
# run-folder = '/tmp/ankaios/'
Copy link
Contributor

Choose a reason for hiding this comment

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

That is not clear to me. Has the user to create that folder? I have never done that.

agent/config/ank-agent.conf Show resolved Hide resolved
agent/config/ank-agent.conf Show resolved Hide resolved
agent/config/ank-agent.conf Show resolved Hide resolved
server/config/ank-server.conf Outdated Show resolved Hide resolved
Comment on lines +27 to +29
# This shall be used to manage eventual persistency.
# !!! At the moment, it is not implemented.
# max_workload_start_retry_time = '300' # seconds
Copy link
Contributor

Choose a reason for hiding this comment

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

If this is not implemented yet, we should delete it.

server/config/ank-server.conf Show resolved Hide resolved
server/config/ank-server.conf Show resolved Hide resolved
server/config/ank-server.conf Show resolved Hide resolved
Copy link
Contributor

@krucod3 krucod3 left a comment

Choose a reason for hiding this comment

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

All in all it looks good 👍
The configuration files are easy to read and understand.

I added a couple of comments to the ank-agent.conf which are also applicable to the other two files.

agent/config/ank-agent.conf Outdated Show resolved Hide resolved
agent/config/ank-agent.conf Outdated Show resolved Hide resolved
agent/config/ank-agent.conf Outdated Show resolved Hide resolved
HorjuRares and others added 3 commits December 16, 2024 13:34
Co-authored-by: Kaloyan <36224699+krucod3@users.noreply.github.com>
Co-authored-by: Holger Dormann <windsource@gmx.de>
Co-authored-by: Holger Dormann <windsource@gmx.de>
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.

3 participants