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

Inconsistent swagger description with implementation (Alerts) #1022

Open
te-koyanagi opened this issue Mar 18, 2024 · 1 comment
Open

Inconsistent swagger description with implementation (Alerts) #1022

te-koyanagi opened this issue Mar 18, 2024 · 1 comment

Comments

@te-koyanagi
Copy link

Issue/Feature Description:
Found an inconsistentency in the swagger description with implementation (openapi-spec/swagger.yaml) about Alerts.

Why this issue to fixed / feature is needed(give scenarios or use cases):
It must be correct for better user experience as no API document exists other than this swagger description.

How to reproduce, in case of a bug:
Following API works as expected, but not in the swagger.yaml.

$ curl -s -X GET http://192.168.231.101:31000/v1/storages/014a38ed-c60e-4994-9f5a-fc7b7f0a5d4d/alerts | jq .
{
  "alerts": [
    {
      "alert_id": 65766686728,
      "alert_name": "CA Certificate Has Expired",
      "severity": "Critical",
      "category": "Fault",
      "type": "NotSpecified",
      "sequence_number": 21168209,
      "occur_time": 1698715016000,
      "description": "The (CA certificate of Call Home authentication, use (N/A)) has expired, and a new one must be imported and activated.",
      "recovery_advice": "Generate and import a valid CA certificate.",
      "resource_type": "Storage",
      "location": "CertificateType=8,Deadline=2023-10-30,use=N/A",
      "storage_id": "014a38ed-c60e-4994-9f5a-fc7b7f0a5d4d",
      "storage_name": "OceanStor",
      "vendor": "Huawei",
      "model": "dorado01",
      "serial_number": "2102352KRP10K1000001"
    }
  ]
}

Other Notes / Environment Information: (Please give the env information, log link or any useful information for this issue)

@te-koyanagi
Copy link
Author

See the following document for the test configuration:
https://docs.google.com/document/d/1WesXmZV0eRDgG2cjg2myqd5BK1t-Eg36O0IzkA6BTto/edit?usp=sharing

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

No branches or pull requests

1 participant