Skip to content

Conversation

@rbtr
Copy link
Collaborator

@rbtr rbtr commented Oct 13, 2021

Signed-off-by: Evan Baker rbtr@users.noreply.github.com

Reason for Change:

  • Adds a context to the NMAgent client method to delegate the timeout cancellation, simplifying the calling code in HTTPRestService.SyncHostNCVersion significantly.
  • Tidyies up things in nmagent package such as type naming to remove stutter.

Issue Fixed:

Closes #850
Fixes #855

Requirements:

Notes:

@rbtr rbtr force-pushed the x/nmagent-client branch from b40da22 to 7a0c11d Compare October 13, 2021 19:28
@rbtr rbtr requested review from csfmomo, pjohnst5 and ramiro-gamarra and removed request for csfmomo October 13, 2021 19:50
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.

couple comments, let me know what you think

@rbtr rbtr force-pushed the x/nmagent-client branch from 0bdf8b4 to 66c3bc7 Compare October 14, 2021 00:40
@rbtr
Copy link
Collaborator Author

rbtr commented Oct 15, 2021

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@rbtr rbtr force-pushed the x/nmagent-client branch 4 times, most recently from 68227a9 to 4238f81 Compare October 19, 2021 20:00
rbtr added 5 commits October 20, 2021 14:26
Signed-off-by: Evan Baker <rbtr@users.noreply.github.com>
Signed-off-by: Evan Baker <rbtr@users.noreply.github.com>
Signed-off-by: Evan Baker <rbtr@users.noreply.github.com>
Signed-off-by: Evan Baker <rbtr@users.noreply.github.com>
@rbtr rbtr force-pushed the x/nmagent-client branch from 493321b to 50e61a2 Compare October 20, 2021 19:48
Signed-off-by: Evan Baker <rbtr@users.noreply.github.com>
@rbtr rbtr force-pushed the x/nmagent-client branch from 50e61a2 to 041a91e Compare October 20, 2021 20:45
Signed-off-by: Evan Baker <rbtr@users.noreply.github.com>
@rbtr rbtr force-pushed the x/nmagent-client branch from 9c5b2d6 to 67a522d Compare October 20, 2021 23:47
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:

@rbtr rbtr merged commit 4a4370b into Azure:master Oct 21, 2021
@rbtr rbtr deleted the x/nmagent-client branch October 21, 2021 18:14
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.

[CNS] Possible goroutine leak in internalapi

2 participants