Skip to content
This repository has been archived by the owner on Dec 2, 2022. It is now read-only.

support for nomad 0.11.1 API #47

Merged
merged 14 commits into from
May 12, 2020
Merged

support for nomad 0.11.1 API #47

merged 14 commits into from
May 12, 2020

Conversation

cgbaker
Copy link
Contributor

@cgbaker cgbaker commented Apr 6, 2020

Updated the generator with multiple changes to handle the problematic cases in the API.
Updated generator to use go modules.
Updated the API target to 0.11.1 (the latest 0.11.x release).

@cgbaker cgbaker self-assigned this Apr 6, 2020
moved script to this dir (gomodules wanted that)
updated generator to handle canonical output for corner cases
@cgbaker cgbaker changed the title support for nomad 0.11.0 API support for nomad 0.11.1 API May 10, 2020
@cgbaker cgbaker marked this pull request as ready for review May 10, 2020 23:26

@Override
public String toString() {
return NomadJson.serialize(this);
Copy link
Member

Choose a reason for hiding this comment

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

This wasn't well-documented in the code but MountOptions are sensitive. We're (with hashicorp/nomad#7923) going to redact them from the HTTP API response so that they're always shown as null here.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

that's fine. thanks.

@cgbaker cgbaker merged commit 91142b4 into master May 12, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants