Skip to content

scope: add example health scope#156

Closed
hongchaodeng wants to merge 1 commit intooam-dev:masterfrom
hongchaodeng:example_health
Closed

scope: add example health scope#156
hongchaodeng wants to merge 1 commit intooam-dev:masterfrom
hongchaodeng:example_health

Conversation

@hongchaodeng
Copy link
Copy Markdown
Member

No description provided.

Comment thread 4.application_scopes.md
appScopeType: core.hydra.io/v1alpha1.Health
properties:
- name: probe-method
value: get-kube-object
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

We should have a list of all the probe method

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Sure. I will try to give a list shortly.

Comment thread 4.application_scopes.md
@wonderflow
Copy link
Copy Markdown
Member

I think this example is important and valuable, can you refine and try to merge it?

@hongchaodeng
Copy link
Copy Markdown
Member Author

After offline discussion, put off until after alpha release.

@technosophos technosophos added this to the backlog milestone Oct 15, 2019
@technosophos
Copy link
Copy Markdown
Contributor

@hongchaodeng The new CLA bot requires us all to re-sign the license agreement.

@wonderflow
Copy link
Copy Markdown
Member

I have implemented health scope in rudr like this spec describe: oam-dev/rudr#367

@hongchaodeng
Copy link
Copy Markdown
Member Author

closing this. It will be included in alpha2 branch.

@hongchaodeng hongchaodeng deleted the example_health branch March 11, 2020 02:10
@hongchaodeng hongchaodeng restored the example_health branch March 11, 2020 02:10
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.

3 participants