Skip to content

ui: rename autoscale instance group to simply autoscaling group#9487

Merged
yadvr merged 1 commit into
apache:mainfrom
shapeblue:autoscale-ui-label
Aug 8, 2024
Merged

ui: rename autoscale instance group to simply autoscaling group#9487
yadvr merged 1 commit into
apache:mainfrom
shapeblue:autoscale-ui-label

Conversation

@yadvr

@yadvr yadvr commented Aug 5, 2024

Copy link
Copy Markdown
Member

This just renames UI label from autoscale instance group to autoscaling group.
Shorter name in the UI menu/navigation makes it break less on some screens.

This was suggested by @msinhore

Types of changes

  • Breaking change (fix or feature that would cause existing functionality to change)
  • New feature (non-breaking change which adds functionality)
  • Bug fix (non-breaking change which fixes an issue)
  • Enhancement (improves an existing feature and functionality)
  • Cleanup (Code refactoring and cleanup, that may add test cases)
  • build/CI
  • test (unit or integration test code)

This just renames UI label from autoscale instance group to autoscaling
group.

Signed-off-by: Rohit Yadav <rohit.yadav@shapeblue.com>
@yadvr yadvr added this to the 4.20.0.0 milestone Aug 5, 2024
@yadvr yadvr marked this pull request as ready for review August 5, 2024 07:16
@yadvr

yadvr commented Aug 5, 2024

Copy link
Copy Markdown
Member Author

@blueorangutan ui

@blueorangutan

Copy link
Copy Markdown

@rohityadavcloud a Jenkins job has been kicked to build UI QA env. I'll keep you posted as I make progress.

@codecov

codecov Bot commented Aug 5, 2024

Copy link
Copy Markdown

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 15.53%. Comparing base (cd8442a) to head (e033126).
Report is 2 commits behind head on main.

Additional details and impacted files
@@             Coverage Diff              @@
##               main    #9487      +/-   ##
============================================
- Coverage     15.54%   15.53%   -0.02%     
+ Complexity    11976    11974       -2     
============================================
  Files          5495     5496       +1     
  Lines        481076   481335     +259     
  Branches      62235    60355    -1880     
============================================
- Hits          74785    74767      -18     
- Misses       398028   398303     +275     
- Partials       8263     8265       +2     
Flag Coverage Δ
uitests 4.18% <ø> (-0.03%) ⬇️
unittests 16.30% <ø> (-0.01%) ⬇️

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

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@FelipeM525 FelipeM525 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

CLGTM Well done!

Before

image

After

image

@yadvr

yadvr commented Aug 7, 2024

Copy link
Copy Markdown
Member Author

@blueorangutan ui

@blueorangutan

Copy link
Copy Markdown

@rohityadavcloud a Jenkins job has been kicked to build UI QA env. I'll keep you posted as I make progress.

@yadvr yadvr merged commit cb0527d into apache:main Aug 8, 2024
@yadvr yadvr deleted the autoscale-ui-label branch August 8, 2024 06:01
dhslove pushed a commit to ablecloud-team/ablestack-cloud that referenced this pull request Aug 22, 2024
…he#9487)

This just renames UI label from autoscale instance group to autoscaling
group.

Signed-off-by: Rohit Yadav <rohit.yadav@shapeblue.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants