Skip to content

Conversation

@kmurudi
Copy link
Contributor

@kmurudi kmurudi commented Mar 17, 2021

feat: This API exposes one of the Httpservice struct field to
display map of OrchestratorContext->PodIpID (UUID) and
includes debug command getPodContexts for CLI use-case.

@codecov
Copy link

codecov bot commented Mar 17, 2021

Codecov Report

Merging #822 (2d318a8) into master (455f5cb) will decrease coverage by 0.31%.
The diff coverage is 53.84%.

@@            Coverage Diff             @@
##           master     #822      +/-   ##
==========================================
- Coverage   41.92%   41.60%   -0.32%     
==========================================
  Files         152      157       +5     
  Lines       14373    14973     +600     
==========================================
+ Hits         6026     6230     +204     
- Misses       7616     7992     +376     
- Partials      731      751      +20     

@kmurudi kmurudi force-pushed the CNS_debugApi_getPodContexts branch from ce697fd to 452a5e5 Compare March 17, 2021 18:46
This API exposes the Httpservice struct field to display
map of OrchestratorContext->PodIpID (UUID) and includes
debug command getPodContexts for CLI use-case.
@kmurudi kmurudi force-pushed the CNS_debugApi_getPodContexts branch from 452a5e5 to dbb4e12 Compare March 17, 2021 19:26
@kmurudi kmurudi requested a review from ramiro-gamarra March 25, 2021 19:27
@csfmomo
Copy link
Contributor

csfmomo commented Mar 26, 2021

Looks good to me, left 3 format comments.

@kmurudi kmurudi force-pushed the CNS_debugApi_getPodContexts branch from 2f72144 to fafbcc5 Compare March 26, 2021 17:06
csfmomo
csfmomo previously approved these changes Mar 26, 2021
Copy link
Contributor

@ramiro-gamarra ramiro-gamarra left a comment

Choose a reason for hiding this comment

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

few comments

Added GET endpoint instead for API Handler
instead of POST.
Added separate test function.
Formatting on multiple files under /cns.
Copy link
Contributor

@ramiro-gamarra ramiro-gamarra left a comment

Choose a reason for hiding this comment

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

:shipit:

@kmurudi kmurudi merged commit 8513ed1 into Azure:master Mar 27, 2021
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.

3 participants