Skip to content

Conversation

@gsharini
Copy link
Member

Description

api-mcp server, doesn’t consider how all the services have structured their CLIs. Our service, along with few other services (eg : psql), have a different command line structure for LIST operations. This is limiting the resource access to Autonomous Recovery Service.

Fixes # (issue)
#40

Type of change

Please delete options that are not relevant.

  • Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

  • In api_mcp_server configured cline, used below prompt
    " Can you give me list of protected databases in Autonomous recovery service from compartment "

Results from cline with the changed description

Based on the previous analysis of compartment ocid1.compartment.oc1..aaaaaa...fr3a, here are the protected databases in Autonomous Recovery Service:

Protected Databases (7 total):

  1. rcvdb1 (TERMINATED)

    • Protected Database ID: ocid1.recoveryserviceprotecteddatabase.oc1.phx.....ga
    • Database ID: ocid1.database.oc1.phx.....idxiq
  2. cgds (TERMINATED)

    • Protected Database ID: ocid1.recoveryserviceprotecteddatabase.oc1.phx....7ogfblga
    • Database ID: ocid1.database.oc1.phx.....wfq
  3. july5 (TERMINATED)

    • Protected Database ID: ocid1.recoveryserviceprotecteddatabase.oc1.phx.....7aldq
    • Database ID: ocid1.database.oc1.phx......h4uwa
  4. DB0130 (TERMINATED)

    • Protected Database ID: ocid1.recoveryserviceprotecteddatabase.....aldq
    • Database ID: ocid1.database.oc1.phx.....yeq
  5. ZL032518 (TERMINATED)

    • Protected Database ID: ocid1.recoveryserviceprotecteddatabase.oc1.phx....4pa
    • Database ID: ocid1.database.oc1.phx.....tm2utiha
  6. DB0129 (TERMINATED)

    • Protected Database ID: ocid1.recoveryserviceprotecteddatabase.oc1.phx....aldq
    • Database ID: ocid1.database.oc1.phx......igkitra
  7. DB0227 (TERMINATED)

    • Protected Database ID: ocid1.recoveryserviceprotecteddatabase.oc1.phx....3g6a
    • Database ID: ocid1.database.oc1.phx.anyhqljrhlxr34qaiqydoanaz2wbz74pr7fkubetxwtfmbh5ynwxpyrr3g6a

Test Configuration:

  • Firmware version:
  • Hardware:
  • Toolchain:
  • SDK:

Checklist:

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes
  • Any dependent changes have been merged and published in downstream modules

@oracle-contributor-agreement
Copy link

Thank you for your pull request and welcome to our community! To contribute, please sign the Oracle Contributor Agreement (OCA).
The following contributors of this PR have not signed the OCA:

To sign the OCA, please create an Oracle account and sign the OCA in Oracle's Contributor Agreement Application.

When signing the OCA, please provide your GitHub username. After signing the OCA and getting an OCA approval from Oracle, this PR will be automatically updated.

If you are an Oracle employee, please make sure that you are a member of the main Oracle GitHub organization, and your membership in this organization is public.

@oracle-contributor-agreement oracle-contributor-agreement bot added the OCA Required At least one contributor does not have an approved Oracle Contributor Agreement. label Oct 23, 2025
@gebhardtr
Copy link
Member

Great contribution and great PR abstract, @gsharini! Can you follow the instructions here to ensure you've satisfied the requirements for contribution?

you may need to experiment with the level of specificity to get the help you
need.
Listing resources in Autonomous Recovery Service is as below:
oci recovery protected-database-collection list-protected-databases
Copy link
Member

Choose a reason for hiding this comment

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

nice addition. In your testing, can this be generalized to:

        1. compute subcommand subcommand ...
        2. compute subcommand
        3. compute

Or something similar? Asking because I think you identified an issue that will affect other subcommands as well.

Copy link
Member Author

Choose a reason for hiding this comment

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

@gebhardtr Thanks for the comments. I have addressed the comment and generalised the cli structure. Please take a look at .

@gsharini
Copy link
Member Author

Great contribution and great PR abstract, @gsharini! Can you follow the instructions here to ensure you've satisfied the requirements for contribution?

[gebhardtr]
I have completed all the necessary steps to get OCA verified... i.e. I am now part of OGHO and have made membership public . Do you happen to know when the status will change to “OCA verified”?

@oracle-contributor-agreement oracle-contributor-agreement bot added OCA Verified All contributors have signed the Oracle Contributor Agreement. and removed OCA Required At least one contributor does not have an approved Oracle Contributor Agreement. labels Oct 30, 2025
@gsharini
Copy link
Member Author

gsharini commented Nov 4, 2025

@gebhardtr Could you please let me know if something else is pending on this PR for me to address?
Kindly approve if this is good. Thank you.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

OCA Verified All contributors have signed the Oracle Contributor Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants