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

[ISSUE #270] Bug fixed, OAM example can not run correctlly. #271

Merged
merged 1 commit into from
Oct 27, 2020

Conversation

linjiemiao
Copy link
Contributor

@linjiemiao linjiemiao commented Oct 27, 2020

I fixed four examples in total:

1) component revsion name format changed in example component-versioning
2) add clusterrole [KIND=Foo] in dependency example
3) add clusterrole [KIND=Ingress] in flight-tracker example
4) change example-health-scope's workloadrefs in typed-component example

Comment on lines 95 to 102
- host: x.oam.dev
http:
paths:
- backend:
serviceName: web-ui
servicePort: 8080
path: /
Copy link
Member

Choose a reason for hiding this comment

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

Users don't own this domain and could not view it if we setting like this. Please keep the original way, thanks.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

ok, i will keep the original way.

Comment on lines -12 to -14
- apiVersion: core.oam.dev/v1alpha2
kind: ContainerizedWorkload
name: example-component2
Copy link
Member

Choose a reason for hiding this comment

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

why remove

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Because Application does not contain a component named example-component2 or example-component1。It contains a component named web-service-component.

Copy link
Member

Choose a reason for hiding this comment

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

I see, thanks

…ssion to

add, delete, modify and check the CRD, the oam example cannot be run
correcttly.

Signed-off-by: linjie.miao <linjie.miao@silot.ai>
Copy link
Member

@wonderflow wonderflow left a comment

Choose a reason for hiding this comment

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

Excellent Job!

@wonderflow wonderflow merged commit 4898d0c into crossplane:master Oct 27, 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.

2 participants