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

added hot reload to receptor config #1135

Merged
merged 6 commits into from
Sep 9, 2024

Conversation

resoluteCoder
Copy link
Contributor

No description provided.

Copy link

codecov bot commented Sep 4, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Please upload report for BASE (devel@07f101e). Learn more about missing BASE report.
Report is 2 commits behind head on devel.

@@           Coverage Diff            @@
##             devel    #1135   +/-   ##
========================================
  Coverage         ?   43.43%           
========================================
  Files            ?       36           
  Lines            ?     7192           
  Branches         ?        0           
========================================
  Hits             ?     3124           
  Misses           ?     3829           
  Partials         ?      239           
Components Coverage Δ
Go 42.92% <ø> (?)
Receptorctl 49.31% <ø> (?)

@resoluteCoder resoluteCoder force-pushed the hot-reload-config branch 2 times, most recently from 03e92e0 to 68fd0de Compare September 4, 2024 13:21
cmd/config.go Show resolved Hide resolved
cmd/config.go Show resolved Hide resolved
cmd/config.go Show resolved Hide resolved
cmd/config.go Show resolved Hide resolved
Copy link
Contributor

@AaronH88 AaronH88 left a comment

Choose a reason for hiding this comment

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

Overall looking good,
I would like a few more comments around the reflection function to make it quicker to read once we revisit later

The functionality looks good to me,
@thom-at-redhat and @matoval can you look also please

cmd/root.go Outdated Show resolved Hide resolved
cmd/root.go Show resolved Hide resolved
cmd/root.go Outdated Show resolved Hide resolved
Copy link
Collaborator

@matoval matoval left a comment

Choose a reason for hiding this comment

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

Looks great, I have a few questions

Copy link

sonarcloud bot commented Sep 6, 2024

@resoluteCoder resoluteCoder merged commit b9bab8f into ansible:devel Sep 9, 2024
25 checks passed
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