-
Notifications
You must be signed in to change notification settings - Fork 56
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
Overrides of Writable fields are undone by first update notification from Consul #72
Comments
lenny-goodell
added
bug
Something isn't working
3-high
priority denoting release-blocking issues
geneva
labels
Apr 24, 2020
lenny-goodell
referenced
this issue
in lenny-goodell/go-mod-bootstrap
Apr 25, 2020
First notification doesn't actual have any changes, but causes issues with overrides that might have just been applied. closes #72 Signed-off-by: lenny <leonard.goodell@intel.com>
13 tasks
Do the following to recreate this issue:
|
This was referenced Apr 30, 2020
Merged
lenny-goodell
pushed a commit
to lenny-goodell/edgex-go
that referenced
this issue
Apr 30, 2020
Fixes this issue edgexfoundry/go-mod-bootstrap#72 closes edgexfoundry#2535 Signed-off-by: lenny <leonard.goodell@intel.com>
Merged
13 tasks
brandonforster
pushed a commit
to edgexfoundry/edgex-go
that referenced
this issue
Apr 30, 2020
…e. (#2536) Fixes this issue edgexfoundry/go-mod-bootstrap#72 closes #2535 Signed-off-by: lenny <leonard.goodell@intel.com>
lenny-goodell
pushed a commit
to lenny-goodell/device-sdk-go
that referenced
this issue
Apr 30, 2020
Fixes this issue edgexfoundry/go-mod-bootstrap#72 closes #518 Signed-off-by: lenny <leonard.goodell@intel.com>
13 tasks
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
On startup we have always received an initial update notification from consul when nothing actually changed in consul. Previously when Consul was the
Record of True
this was not an issue.Now it is an issue as overrides to Writable fields, which applied after loading from Consul, are undone when this initial update notification is processed. This only occurs when config already exists in Consul and a Writable filed that has an override has been changed in Consul or the override was added after to config was pushed into Consul
We somehow need to ignore this first
bogus
update notification from Consul.The text was updated successfully, but these errors were encountered: