Skip to content
This repository has been archived by the owner on Jul 12, 2022. It is now read-only.

fix namespace override "deadlock" #1196

Merged
merged 1 commit into from
May 11, 2021

Conversation

lucasbfernandes
Copy link
Contributor

@lucasbfernandes lucasbfernandes commented May 10, 2021

Signed-off-by: Lucas Borges Fernandes lucasbfernandes94@gmail.com

Fixed:

Ao fazer a alteração do namespace de "A" para "B" nas configurações de deployment, depois de já ter uma release em mar aberto não é possível fazer override da release neste circulo, pois já existe outra feita no mesmo circulo em outro namespace. Como não conseguimos fazer undeploy via Charles, acontece um deadlock e acabamos matando o workspace configurado.

Signed-off-by: Lucas Borges Fernandes <lucasbfernandes94@gmail.com>
@boring-cyborg boring-cyborg bot added the butler Improvements/additions/fixes for butler module. label May 10, 2021
@lucasbfernandes lucasbfernandes changed the title fix namespace override deadlock fix namespace override "deadlock" May 10, 2021
@codecov
Copy link

codecov bot commented May 11, 2021

Codecov Report

Merging #1196 (87131aa) into charlescd-release-1.0.0 (051cf16) will increase coverage by 0.02%.
The diff coverage is 88.13%.

Impacted file tree graph

@@                      Coverage Diff                      @@
##             charlescd-release-1.0.0    #1196      +/-   ##
=============================================================
+ Coverage                      76.72%   76.74%   +0.02%     
- Complexity                      2164     2169       +5     
=============================================================
  Files                           1304     1304              
  Lines                          18703    18683      -20     
  Branches                        1707     1707              
=============================================================
- Hits                           14349    14338      -11     
+ Misses                          3844     3838       -6     
+ Partials                         510      507       -3     
Flag Coverage Δ Complexity Δ
butler 84.51% <90.19%> (+0.15%) 0.00 <0.00> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ Complexity Δ
...i/deployments/controller/deployments.controller.ts 85.71% <ø> (-0.40%) 0.00 <0.00> (ø)
...eployments/pipes/default-circle-uniqueness.pipe.ts 100.00% <ø> (ø) 0.00 <0.00> (ø)
...io/charlescd/moove/application/WorkspaceService.kt 100.00% <ø> (ø) 10.00 <0.00> (ø)
...rastructure/service/client/response/LogResponse.kt 0.00% <0.00%> (ø) 0.00 <0.00> (ø)
ui/src/core/assets/themes/modal/trigger.ts 100.00% <ø> (ø) 0.00 <0.00> (ø)
ui/src/core/components/Card/Config/index.tsx 100.00% <ø> (ø) 0.00 <0.00> (ø)
ui/src/core/components/Form/InputTitle/index.tsx 93.33% <ø> (-1.91%) 0.00 <0.00> (ø)
ui/src/core/components/Modal/Wizard/constants.ts 100.00% <ø> (ø) 0.00 <0.00> (ø)
ui/src/core/components/Text/index.tsx 100.00% <ø> (ø) 0.00 <0.00> (ø)
ui/src/mocks/handlers/circles/index.ts 0.00% <0.00%> (ø) 0.00 <0.00> (?)
... and 80 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update a9e1670...87131aa. Read the comment docs.

@lucasbfernandes lucasbfernandes merged commit 156cb1f into charlescd-release-1.0.0 May 11, 2021
@lucasbfernandes lucasbfernandes deleted the fix-namespace-override branch May 11, 2021 17:31
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
butler Improvements/additions/fixes for butler module.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants