Odigos uninstall console output bug

[edeNFed]

When running odigos uninstall the checkmarks of the steps Rolling back odigos changes to pods and Rolling back odigos changes to namespaces are misplaced.

See the following image:

[ankur12-1610]

Hey, I would like to work on this issue.

[ankur12-1610]

Hi @edeNFed, what should be the correct order of the checkmarks?

[HeetVekariya]

I would like to work on this issue as this is good first issue, can you please assign it to me.

[edeNFed]

The checkmarks should be in the same line of the current operation. So this should be fixed to:

Rolling back odigos changes to pods ..... V
Rolling back odigos changes to namespaces ... V

[HeetVekariya]

@edeNFed Please look into it.

• This is the current code:

• Changes made to get the desired output:

  • Remove "\n" from end of the log.print to print the tic mark in the same line

  • Added l.success() after log.Print("Rolling back odigos changes to pods")

  • I have to codes:

(A)

(B)

• If my solution is correct, please let me know which code you want to continue with from above two

[HeetVekariya]

@ankur12-1610 I think the current is assigned to me, so PR should be submitted by me as written in the contributing.md

[HeetVekariya]

@edeNFed I have submitted the PR, can please review it and comment if change needed.

[HeetVekariya]

@edeNFed Thank you for merging, close this issue as completed.

[blumamir]

fixed in #627