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

disable single resource api. #264

Merged
merged 2 commits into from
Feb 24, 2025

Conversation

morvencao
Copy link
Contributor

@morvencao morvencao commented Feb 19, 2025

Ref: https://issues.redhat.com/browse/ACM-17947

This PR includes the following changes:

  • Removed the CRUD API for the /api/maestro/v1/resources endpoints, which means the original resource API object no longer exists.
  • Keep the /api/maestro/v1/resource-bundles GET/LIST methods for listing/getting resource bundles.
  • Added a DELETE method to /api/maestro/v1/resource-bundles/<id> to remove a resource bundle.
  • Refactored unit, integration, and e2e tests to use the CloudEvents source client for creating, updating, and deleting resources.

@morvencao morvencao force-pushed the br_disable_resource_api branch 8 times, most recently from d7704b4 to 1d1d594 Compare February 19, 2025 14:27
@morvencao morvencao changed the title disable resource api. disable single resource api. Feb 20, 2025
@morvencao morvencao force-pushed the br_disable_resource_api branch 2 times, most recently from 97e1dd4 to 6c26282 Compare February 20, 2025 12:43
Signed-off-by: morvencao <lcao@redhat.com>
@morvencao morvencao force-pushed the br_disable_resource_api branch from 6c26282 to 070bf2e Compare February 20, 2025 12:46
@morvencao
Copy link
Contributor Author

/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
Copy link
Contributor

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.

Copy link
Contributor Author

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:
Copy link
Contributor

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?

Copy link
Contributor Author

@morvencao morvencao Feb 24, 2025

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.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done

@clyang82
Copy link
Contributor

/ok-to-test

Signed-off-by: morvencao <lcao@redhat.com>
@morvencao morvencao force-pushed the br_disable_resource_api branch from 30c1a9e to 2492c34 Compare February 24, 2025 06:43
Copy link
Contributor

@clyang82 clyang82 left a comment

Choose a reason for hiding this comment

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

LGTM

@clyang82
Copy link
Contributor

/ok-to-test

@clyang82 clyang82 merged commit 2369383 into openshift-online:main Feb 24, 2025
8 checks passed
@morvencao morvencao deleted the br_disable_resource_api branch February 24, 2025 08:36
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.

2 participants