fabio icon indicating copy to clipboard operation
fabio copied to clipboard

add option to constrain fabio instance to specific consul namespace

Open baabgai opened this issue 3 years ago • 8 comments

This helps to use fabio in a multi tenant environment by allowing to register a fabio instance in a specific consul namespace. All services that will be picked up with the specified tagprefix by this instance only come from the same consul namespace.

baabgai avatar Jan 11 '21 17:01 baabgai

CLA assistant check
All committers have signed the CLA.

CLAassistant avatar Jan 11 '21 17:01 CLAassistant

@leprechau / @pschultz / @nathanejohnson, is this contribution you would want to see in fabio? It seems useful to those of us running multiple fabio instances (each with their own tagprefix), or looking to isolate groups of services/traffic.

ketzacoatl avatar Feb 03 '21 05:02 ketzacoatl

Seems very useful to me. I just can't help with the review/testing, because we're not an enterprise customer.

pschultz avatar Feb 03 '21 09:02 pschultz

I can see this being useful, but like @pschultz we are not enterprise customers so I can't test or vet this PR.

nathanejohnson avatar Feb 03 '21 20:02 nathanejohnson

I'm not a gopher, so my review of the code would be meaningless. Would you be able to do a code review?

I could run tests to confirm the expected behavior.

ketzacoatl avatar Feb 03 '21 23:02 ketzacoatl

The code looks fine to me. It's a pretty obvious and natural change. What I can't judge is if there are any missing places where the namespace has to be respected, but should become apparent with testing.

pschultz avatar Feb 04 '21 11:02 pschultz

@baabgai are you using this patch in your environment? Can you confirm operation?

aaronhurt avatar Apr 30 '21 17:04 aaronhurt

Hi @leprechau, Yes we're using it at the moment in production and until now we didn't experience any issues with namespace addition. To be fair I have to mention that we're still in the process of setting up a multi tenant cluster where this component is used and the current production load is still rather low.

baabgai avatar May 21 '21 20:05 baabgai