Skip to:

  1. Skip to Jira Navigation
  2. Skip to Side Navigation
  3. Skip to Main Content

swagger docs for services:discoverRDS say required params

Description

https://pmmdemo.percona.com/swagger-ui#/ManagementService/DiscoverRDS

Swagger doc says a body payload is required, but this is not true.

Running the API command, without a body payload, results in success:

# curl -k -X POST https://admin:xxxx@localhost/v1/management/services:discoverRDS { "rds_instances": [ { "region": "us-east-1", "az": "us-east-1f", "instance_id": "rds-mysql-80", "node_model": "db.t4g.medium", "address": "rds-mysql-80.cg8slbmxcsve.us-east-1.rds.amazonaws.com", "port": 3306, "engine": "DISCOVER_RDS_ENGINE_MYSQL", "engine_version": "8.0.40" } ] }

Please update swagger docs to be correct.

How to test

None

How to document

None

Attachments

1
  • 02 Apr 2025, 11:28 PM

Activity

Matthew Boehm 5 days ago

I clarified the bug. Swagger says “body” is required, but it’s not.

Nurlan Moldomurov 5 days ago

they are not required, they are optional.

Screenshot 2025-04-03 at 02.25.40.png

Matthew Boehm 5 days ago

It literally says ‘required’ in red font in the screenshot you attached.

Details

Assignee

Reporter

Priority

Needs QA

Yes

Needs Doc

Yes

Smart Checklist

Created last week
Updated 5 days ago

Flag notifications