gitops-engine icon indicating copy to clipboard operation
gitops-engine copied to clipboard

Update condition to select right pvc as child for statefulset

Open hanzala1234 opened this issue 2 years ago • 10 comments

We have 2 pvc in namespace, one with name emqx-data-sw-broker-0 for statefulset sw-broker and other one is emqx-data-sw-broker-internal-0 for statefulset sw-broker-internal and both have same template name emqx-data. since emqx-data-sw-broker-internal-0 contains prefix emqx-data-sw-broker-, it consider it as part of sw-broker statefulset which is not right. fixing condition to check if it contains number at the end of prefix

hanzala1234 avatar Nov 09 '23 15:11 hanzala1234

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

sonarqubecloud[bot] avatar Nov 09 '23 15:11 sonarqubecloud[bot]

Any updates on this, I've experienced this issue as well and it would be nice to get it resolved.

bradenwright avatar Dec 06 '23 04:12 bradenwright

any update on this issue?

bradenwright avatar Jan 19 '24 18:01 bradenwright

any update on this? It's a bug fix and its been open for over 4 months

bradenwright avatar Mar 14 '24 05:03 bradenwright

@hanzala1234 thank you for your contribution.

Could you please cover your changes with tests and also could you please add example yamls so i can test the fix locally?

pasha-codefresh avatar Mar 18 '24 16:03 pasha-codefresh

i know @hanzala1234 did the work along with my company, I can try to take a look at that. I should have time this week (I'm gonna be traveling tomorrow).

bradenwright avatar Mar 20 '24 00:03 bradenwright

Thank you @bradenwright

pasha-codefresh avatar Mar 20 '24 20:03 pasha-codefresh

I've had a few fires pop up this week, so it will probably be a week or 2 til I have time to look into this.

bradenwright avatar Mar 21 '24 23:03 bradenwright

Hello @pasha-codefresh We have added some test cases/examples into the code, it is good to review now

nerdeveloper avatar May 02 '24 19:05 nerdeveloper

Just another reminder that need to be checked please @pasha-codefresh

nerdeveloper avatar May 06 '24 22:05 nerdeveloper

@nerdeveloper will review it tomorrow, sorry, didnt notice your msg

pasha-codefresh avatar May 07 '24 16:05 pasha-codefresh

Looks great for me, just doing manual testing and will merge it

pasha-codefresh avatar May 14 '24 12:05 pasha-codefresh

Снимок экрана 2024-05-14 в 21 59 56

Issue is resolved, LGTM!

pasha-codefresh avatar May 14 '24 19:05 pasha-codefresh

@hanzala1234 , thank you for your contribution, feel free to open ArgoCD PR with gitops-engine version update, you can assign or tag me, i will review it

pasha-codefresh avatar May 14 '24 19:05 pasha-codefresh

thanks @pasha-codefresh for merging MR. I will work on it tomorrow

hanzala1234 avatar May 14 '24 23:05 hanzala1234