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

feat: add nomad installation docs #467

Merged
merged 23 commits into from
Aug 16, 2023
Merged

Conversation

resmo
Copy link
Contributor

@resmo resmo commented Jul 26, 2023

See #331

@resmo resmo mentioned this pull request Jul 26, 2023
3 tasks
docs/nomad/README.md Outdated Show resolved Hide resolved
@resmo resmo changed the title WIP: docs: add nomad docs docs: add nomad docs Aug 12, 2023
@resmo resmo marked this pull request as ready for review August 12, 2023 16:09
@resmo resmo requested a review from a team as a code owner August 12, 2023 16:09
@resmo
Copy link
Contributor Author

resmo commented Aug 12, 2023

Ready for review

docs/nomad/README.md Outdated Show resolved Hide resolved
docs/nomad/README.md Outdated Show resolved Hide resolved
docs/nomad/README.md Outdated Show resolved Hide resolved
docs/nomad/README.md Outdated Show resolved Hide resolved
docs/nomad/README.md Outdated Show resolved Hide resolved
docs/nomad/README.md Outdated Show resolved Hide resolved
docs/nomad/README.md Outdated Show resolved Hide resolved
docs/nomad/README.md Outdated Show resolved Hide resolved
resmo and others added 11 commits August 14, 2023 09:32
Co-authored-by: Jonas L. <jooola@users.noreply.github.com>
Co-authored-by: Jonas L. <jooola@users.noreply.github.com>
Co-authored-by: Jonas L. <jooola@users.noreply.github.com>
Co-authored-by: Jonas L. <jooola@users.noreply.github.com>
Co-authored-by: Jonas L. <jooola@users.noreply.github.com>
Co-authored-by: Jonas L. <jooola@users.noreply.github.com>
Co-authored-by: Jonas L. <jooola@users.noreply.github.com>
Copy link
Member

@apricote apricote left a comment

Choose a reason for hiding this comment

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

Nice and exhaustive guide for using the CSI driver from Nomad :) Going to also request a review from a coworker using Nomad & the csi-driver privately.

docs/nomad/README.md Outdated Show resolved Hide resolved
docs/nomad/README.md Outdated Show resolved Hide resolved
Co-authored-by: Julian Tölle <julian.toelle97@gmail.com>
Copy link
Member

@apricote apricote left a comment

Choose a reason for hiding this comment

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

Some further comments from my coworker

docs/nomad/README.md Show resolved Hide resolved
docs/nomad/README.md Outdated Show resolved Hide resolved
Copy link
Member

@apricote apricote left a comment

Choose a reason for hiding this comment

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

All looking good from my POV :) Going to leave this open for another day in case one of the other two reviewers still have open comments.

Thank you very much for writing and testing this @resmo!

@resmo
Copy link
Contributor Author

resmo commented Aug 14, 2023

common problem on GH, my PR doesn't have access to your secrets, tests may fail. ;)

@apricote
Copy link
Member

common problem on GH, my PR doesn't have access to your secrets, tests may fail. ;)

Yea, but I am confident that a markdown document won't break our e2e tests ;)

@apricote
Copy link
Member

One thing I forgot, but I can also add it right before merging, it would be nice to link to the new docs from our readme: https://github.com/hetznercloud/csi-driver#getting-started

@resmo
Copy link
Contributor Author

resmo commented Aug 14, 2023

added "not officially supported" I guessed?

@apricote
Copy link
Member

added "not officially supported" I guessed?

Yea, until we have automated tests for this.

@apricote apricote merged commit 631b117 into hetznercloud:main Aug 16, 2023
1 of 5 checks passed
@apricote apricote changed the title docs: add nomad docs feat: add nomad installation docs Aug 16, 2023
@resmo resmo deleted the doc/nomad branch August 17, 2023 20:01
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.

None yet

4 participants