Skip to content
This repository was archived by the owner on Sep 28, 2021. It is now read-only.

Conversation

@moredure
Copy link

@moredure moredure commented Apr 9, 2018

No description provided.

@moredure
Copy link
Author

moredure commented Apr 9, 2018

As I see, nomad-ruby not supported because there is missing methods not only for dispatch and stop but for job versions, that's why spec for read job not working fails

@hashicorp-cla
Copy link

hashicorp-cla commented Dec 6, 2019

CLA assistant check

Thank you for your submission! We require that all contributors sign our Contributor License Agreement ("CLA") before we can accept the contribution. Read and sign the agreement

Learn more about why HashiCorp requires a CLA and what the CLA includes


0 out of 2 committers have signed the CLA.

  • moredure
  • Mike

Mike seems not to be a GitHub user.
You need a GitHub account to be able to sign the CLA. If you already have a GitHub account, please add the email address used for this commit to your account.

Have you signed the CLA already but the status is still pending? Recheck it.

Copy link

@drewbailey drewbailey left a comment

Choose a reason for hiding this comment

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

Thanks for this contribution! Do you mind adding integration tests for dispatch and stop? The Job integration specs can be found here https://github.com/hashicorp/nomad-ruby/blob/master/spec/integration/api/job_spec.rb

# dispatch job meta parameters
#
# @return [JobDispatch]
def dispatch(name, payload_contents, meta, **options)

Choose a reason for hiding this comment

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

Do you mind changing payload_contents to contents to match create?

Copy link
Author

Choose a reason for hiding this comment

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

It was long time ago) Currently I will prefer Golang integration over Ruby one

@spuder
Copy link

spuder commented May 31, 2020

I would really like to see this feature implemented.

@moredure moredure closed this Jun 15, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants