-
Notifications
You must be signed in to change notification settings - Fork 25
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
disable single resource api. #264
disable single resource api. #264
Conversation
d7704b4
to
1d1d594
Compare
97e1dd4
to
6c26282
Compare
Signed-off-by: morvencao <lcao@redhat.com>
6c26282
to
070bf2e
Compare
/assign @clyang82 |
README.md
Outdated
ocm login --token=${OCM_ACCESS_TOKEN} --url=http://localhost:8000 | ||
|
||
``` | ||
|
||
#### Get a new Resource | ||
This will be empty if no Resource is ever created | ||
#### List the resource bundle |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
change it to List the consumer
. in this step, the result is empty. then create a consumer and list again.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done
|
||
## Preparation | ||
|
||
1. Enable gRPC server by passing `--enable-grpc-server=true` to the maestro server start command: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do we need to enable gPRC server manually? I think we should make it enabled by default. WDYT?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yes, we can do it.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done
/ok-to-test |
Signed-off-by: morvencao <lcao@redhat.com>
30c1a9e
to
2492c34
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
/ok-to-test |
Ref: https://issues.redhat.com/browse/ACM-17947
This PR includes the following changes:
/api/maestro/v1/resources
endpoints, which means the originalresource
API object no longer exists./api/maestro/v1/resource-bundles
GET/LIST methods for listing/getting resource bundles./api/maestro/v1/resource-bundles/<id>
to remove a resource bundle.