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

healthchecks.active.unhealthy.interval field is missing from the usptream #6

Closed
hbagdi opened this issue Aug 5, 2019 · 0 comments · Fixed by #7
Closed

healthchecks.active.unhealthy.interval field is missing from the usptream #6

hbagdi opened this issue Aug 5, 2019 · 0 comments · Fixed by #7
Assignees

Comments

@hbagdi
Copy link
Owner

hbagdi commented Aug 5, 2019

healthchecks.active.unhealthy.interval is missing from the upstream definition.
Also, ensure that all the properties are covered and there is not another one that is missing.

@hbagdi hbagdi self-assigned this Aug 5, 2019
@hbagdi hbagdi closed this as completed in #7 Aug 6, 2019
hbagdi added a commit to Kong/kubernetes-ingress-controller that referenced this issue Aug 26, 2019
decK
----

Changelog:
https://github.com/hbagdi/deck/blob/master/CHANGELOG.md

go-kong
-------

Changelog:
https://github.com/hbagdi/go-kong/blob/master/CHANGELOG.md

go-kong bump is large because of multiple releases that happened while
preparing for this commit.

Noteworthy changes apart from additions:
- hbagdi/go-kong#6
- String() method has been dropped from all types defined in the
  library.
hbagdi added a commit to Kong/kubernetes-ingress-controller that referenced this issue Aug 26, 2019
decK
----

Changelog:
https://github.com/hbagdi/deck/blob/master/CHANGELOG.md

go-kong
-------

Changelog:
https://github.com/hbagdi/go-kong/blob/master/CHANGELOG.md

go-kong bump is large because of multiple releases that happened while
preparing for this commit.

Noteworthy changes apart from additions:
- hbagdi/go-kong#6
- String() method has been dropped from all types defined in the
  library.


From #370
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 a pull request may close this issue.

1 participant