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

Feature/swagger: swagger 문서 #23

Closed
wants to merge 13 commits into from
Closed

Feature/swagger: swagger 문서 #23

wants to merge 13 commits into from

Conversation

rrgks6221
Copy link
Member

@rrgks6221 rrgks6221 commented Jul 29, 2022

swagger url

local: localhost:8080/api-docs
product: http://uncloudy.duckdns.org:8080/api-docs

@rrgks6221 rrgks6221 added the essential Essential feature label Jul 29, 2022
@rrgks6221 rrgks6221 requested review from 100kimch and jnubugo July 29, 2022 09:11
@rrgks6221 rrgks6221 self-assigned this Jul 29, 2022
@rrgks6221 rrgks6221 linked an issue Jul 29, 2022 that may be closed by this pull request
@rrgks6221 rrgks6221 added the documentation Improvements or additions to documentation label Jul 29, 2022
@100kimch
Copy link
Member

@rrgks6221 본 브랜치가 remodel-prometheus-exporter 브랜치와 섞여서 커밋이 된 것 같네요. 이전 PR들 합친 후에 PR merge 할게요.

swagger 배포 이후의 URL도 적어주세요.

@rrgks6221
Copy link
Member Author

@100kimch 배포 이후 swagger 주소 추가했고 remodel-prometheus-exporter 브랜치 머지 되고 충돌 상황보고 다시 해결 할게요.

@rrgks6221
Copy link
Member Author

@100kimch

  • Uncloudy prometheus exporter 지표 추가 작업 #30 에 따른 swagger 문서 수정
  • response schema 추가
    1. node resource의 value가 한개일 경우
    2. node resource의 value가 한개일 경우
    3. pod resource의 value가 여러개일 경우
    4. pod resource의 value가 여러개일 경우

20220731회의 때 말한 커스텀 매트릭 마다의 example을 만드는건 swagger ui 측에서 같은 endpoint에 여러개를 만드는게 안돼서 안될거같네요

@rrgks6221 rrgks6221 mentioned this pull request Apr 3, 2023
@rrgks6221 rrgks6221 closed this May 31, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation essential Essential feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

uncloudy-prometheus-exporter swagger REST API 문서
2 participants