Skip to content

fix(pci-kubernetes): add node pool total price, improve node name input #17532

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

ppprevost
Copy link
Contributor

@ppprevost ppprevost commented Jun 10, 2025

ref: #TAPC-3341

Description

The goal of this pull request is to correctly display the price of nodes based on the number of machines. It includes the following changes:

Correction of the size of the input fields for name cluster and name node.
Fixes for TypeScript types.

Ticket Reference: #...

Additional Information

@github-actions github-actions bot added universe-public-cloud bug Something isn't working labels Jun 10, 2025
@ppprevost ppprevost force-pushed the fix/pci-kubernetes_add-node-price_tapc-3341 branch 3 times, most recently from 2667696 to 9b06c91 Compare June 10, 2025 13:19
@github-actions github-actions bot added has conflicts Has conflicts to resolve before merging container labels Jun 10, 2025
@ppprevost ppprevost force-pushed the fix/pci-kubernetes_add-node-price_tapc-3341 branch from 9b06c91 to 165d7c3 Compare June 10, 2025 15:27
@github-actions github-actions bot removed container has conflicts Has conflicts to resolve before merging labels Jun 10, 2025
@ppprevost ppprevost force-pushed the fix/pci-kubernetes_add-node-price_tapc-3341 branch 3 times, most recently from 49084d9 to ca808be Compare June 10, 2025 15:47
@ppprevost ppprevost marked this pull request as ready for review June 10, 2025 16:15
@ppprevost ppprevost requested a review from a team as a code owner June 10, 2025 16:15
@ppprevost ppprevost force-pushed the fix/pci-kubernetes_add-node-price_tapc-3341 branch 2 times, most recently from 5761e58 to 7489ee3 Compare June 11, 2025 15:43
SimonChaumet
SimonChaumet previously approved these changes Jun 11, 2025
@ppprevost ppprevost force-pushed the fix/pci-kubernetes_add-node-price_tapc-3341 branch from 7489ee3 to 314a062 Compare June 11, 2025 16:09
@ppprevost ppprevost requested a review from SimonChaumet June 12, 2025 07:04
@ppprevost ppprevost force-pushed the fix/pci-kubernetes_add-node-price_tapc-3341 branch from 314a062 to b29f186 Compare June 12, 2025 07:07
SimonChaumet
SimonChaumet previously approved these changes Jun 12, 2025
@ppprevost ppprevost force-pushed the fix/pci-kubernetes_add-node-price_tapc-3341 branch from b29f186 to 20adfc6 Compare June 12, 2025 14:46
ref: #TAPC-3341

Signed-off-by: Pierre-Philippe <[email protected]>
@ppprevost ppprevost force-pushed the fix/pci-kubernetes_add-node-price_tapc-3341 branch from 20adfc6 to b2f1caf Compare June 12, 2025 16:48
Copy link
Contributor

@fredericvilcot fredericvilcot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

All good, thanks for the changes, that is a real step forward 👍

Copy link
Contributor

@Tsiorifamonjena Tsiorifamonjena left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just a question: when you prefix type wit T and when you don't ?

@Tsiorifamonjena Tsiorifamonjena self-requested a review June 13, 2025 08:05
Copy link
Contributor

@Tsiorifamonjena Tsiorifamonjena left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Test and build failed

@Tsiorifamonjena Tsiorifamonjena self-requested a review June 13, 2025 08:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working universe-public-cloud
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants