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

Added Nomad to Container Orchestrators #421 #422

Merged
merged 2 commits into from
May 1, 2020

Conversation

mpron
Copy link
Contributor

@mpron mpron commented Mar 30, 2020

Added a bullet link for Nomad to the readme file linking to the information about CSI support. New feature so it may change as documentation pages are created.

Added a bullet link for Nomad to the readme file linking to the information about CSI support. New feature so it may change as documentation pages are created.
Copy link
Member

@jieyu jieyu left a comment

Choose a reason for hiding this comment

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

LGTM. @saad-ali @jdef @julian-hj @ddebroy can you guys take a look? this one should be quick.

@tgross
Copy link

tgross commented Apr 6, 2020

Hi folks, I'm on the Nomad team and was prepping to make a PR myself when I saw this one. We would recommend https://nomadproject.io/docs/internals/plugins/csi/ as a better link.

@jieyu
Copy link
Member

jieyu commented Apr 7, 2020

@mpron could you update link according to @tgross ?

Updated link to a better reference for the integration: https://nomadproject.io/docs/internals/plugins/csi/
@mpron
Copy link
Contributor Author

mpron commented Apr 7, 2020

Done. Sorry I jumped the gun @tgross

@tgross
Copy link

tgross commented Apr 8, 2020

Sorry I jumped the gun @tgross

Not at all! Thanks!

@shantanugadgil
Copy link

can this be merged now?

@mpron
Copy link
Contributor Author

mpron commented Apr 13, 2020

@shantanugadgil yes.

@shantanugadgil
Copy link

Gak! Celebrated too soon. Will wait for this to be merged!

@shantanugadgil
Copy link

@tgross @jieyu @jdef anything holding this back from merging? 😃

@saad-ali
Copy link
Member

LGTM

@jieyu
Copy link
Member

jieyu commented Apr 15, 2020

@mpron
Copy link
Contributor Author

mpron commented Apr 15, 2020

@tgross I leave this to you and your team to run this up the corporate ladder. I'm definitely not the right person to sign that CLA. I'll bow out of this thread now. :)

@jieyu
Copy link
Member

jieyu commented Apr 20, 2020

@tgross @mpron let me know how you want us to make progress here. Unfortunately, the CLA is a legal boilerplate we'll have to do. I am happy to submit a PR for you guys (I already signed the CLA).

@tgross
Copy link

tgross commented May 1, 2020

Hi @jieyu, sorry about the delay.

We're still in the process of getting the CLA through our legal team. If you have time to submit the PR for us that'd be great. HashiCorp are going to continue to extend our CSI support in Nomad. So I'm sure we'll have contributions going forward anyways and our internal efforts won't be wasted even if you do so. Thanks!

@jieyu
Copy link
Member

jieyu commented May 1, 2020

@tgross ok, sounds good. We'll wait for you guys to sort out the CLA then.

@galeep
Copy link

galeep commented May 1, 2020

@jieyu A signed CLA has been sent your way!

@jieyu
Copy link
Member

jieyu commented May 1, 2020

Signed CLA received. Merging.

@jieyu jieyu merged commit 87a27a2 into container-storage-interface:master May 1, 2020
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.

7 participants