#1634 #1487 #1329 #1304 #1294 #793 Consul polling of services: enhancements and fix errors by ggnaegi · Pull Request #1670 · ThreeMammals/Ocelot

Commits on Sep 19, 2023

fixing some issues in poll consul:

Guillaume Gnaegi
authored and
raman-m
committed
line endings

Guillaume Gnaegi
authored and
raman-m
committed
adding some test cases

Guillaume Gnaegi
authored and
raman-m
committed
Using a lock instead of SemaphoreSlim

Guillaume Gnaegi
authored and
raman-m
committed
Improve code readability
raman-m
committed
CA2211: Non-constant fields should not be visible
raman-m
committed
Use IOcelotLogger to remove warnings & messages of static code analysis (aka IDE0052)
raman-m
committed
Fix errors with unit tests discovery. Remove legacy life hacks of discovering tests on .NET Core
raman-m
committed
Update unit tests
raman-m
committed
Also refactoring the kubernetes provider factory (like consul and eureka)

ggnaegi
authored and
raman-m
committed
shorten references...

ggnaegi
authored and
raman-m
committed
const before...

ggnaegi
authored and
raman-m
committed
Some minor fixes, using Equals Ordinal ignore case and a string constant for provider type definition instead of string litterals. Fixing usings.

ggnaegi
authored and
raman-m
committed
waiting a bit longer then?

ggnaegi
authored and
raman-m
committed
@RaynaldM code review
raman-m
committed
renaming PollKubernetes to PollKube

ggnaegi
authored and
raman-m
committed
... odd...

ggnaegi
authored and
raman-m
committed
... very odd, we have an issue with configuration update duration...

ggnaegi
authored and
raman-m
committed
IDE0002: Name can be simplified
raman-m
committed
All tests passing locally, hopefully it works online
ggnaegi
committed

Commits on Sep 26, 2023

Commits on Sep 28, 2023

Merge branch 'develop' into bug/polling-consul
raman-m
authored

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

#1634 #1487 #1329 #1304 #1294 #793 Consul polling of services: enhancements and fix errors#1670

#1634 #1487 #1329 #1304 #1294 #793 Consul polling of services: enhancements and fix errors#1670
raman-m merged 26 commits intoThreeMammals:developfrom
ggnaegi:bug/polling-consul

Commits on Sep 19, 2023

fixing some issues in poll consul:

line endings

adding some test cases

Using a lock instead of SemaphoreSlim

Improve code readability

CA2211: Non-constant fields should not be visible

Use IOcelotLogger to remove warnings & messages of static code analysis (aka IDE0052)

Fix errors with unit tests discovery. Remove legacy life hacks of discovering tests on .NET Core

Update unit tests

Also refactoring the kubernetes provider factory (like consul and eureka)

shorten references...

const before...

Some minor fixes, using Equals Ordinal ignore case and a string constant for provider type definition instead of string litterals. Fixing usings.

waiting a bit longer then?

@RaynaldM code review

renaming PollKubernetes to PollKube

... odd...

... very odd, we have an issue with configuration update duration...

IDE0002: Name can be simplified

All tests passing locally, hopefully it works online

Commits on Sep 26, 2023

merge

just a bit of cleanup

Commits on Sep 28, 2023

Merge branch 'develop' into bug/polling-consul

Commits on Sep 29, 2023

Merge branch 'develop' into bug/polling-consul

Some missing braces and commas

Update servicediscovery.rst: Review and update "Consul" section