Skip to content

fix label key for gpudisplay - #13272

Merged
DaanHoogland merged 1 commit into
apache:4.22from
shapeblue:fixGpuLabel
Sep 8, 2026
Merged

fix label key for gpudisplay#13272
DaanHoogland merged 1 commit into
apache:4.22from
shapeblue:fixGpuLabel

Conversation

@sudo87

@sudo87 sudo87 commented May 28, 2026

Copy link
Copy Markdown
Contributor

Description

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)

Feature/Enhancement Scale or Bug Severity

Feature/Enhancement Scale

  • Major
  • Minor

Bug Severity

  • BLOCKER
  • Critical
  • Major
  • Minor
  • Trivial

Screenshots (if appropriate):

Screenshot 2026-05-28 at 3 26 11 PM

How Has This Been Tested?

How did you try to break this feature and the system with this change?

@sudo87
sudo87 requested a review from vishesh92 May 28, 2026 09:58
@vishesh92

Copy link
Copy Markdown
Member

@blueorangutan ui

@blueorangutan

Copy link
Copy Markdown

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

@vishesh92 vishesh92 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

clgtm

@codecov

codecov Bot commented May 28, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 18.11%. Comparing base (c0ce5b4) to head (f4cdd27).

Additional details and impacted files
@@            Coverage Diff            @@
##               main   #13272   +/-   ##
=========================================
  Coverage     18.10%   18.11%           
- Complexity    16749    16758    +9     
=========================================
  Files          6037     6037           
  Lines        542796   542796           
  Branches      66456    66456           
=========================================
+ Hits          98268    98320   +52     
+ Misses       433492   433430   -62     
- Partials      11036    11046   +10     
Flag Coverage Δ
uitests 3.51% <ø> (ø)
unittests 19.27% <ø> (+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.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@DaanHoogland

Copy link
Copy Markdown
Contributor

I fail to see how this is an improvement. I would argue other lebals need an extra . after gpu...

@blueorangutan

Copy link
Copy Markdown

UI build: ✔️
Live QA URL: https://qa.cloudstack.cloud/simulator/pr/13272 (QA-JID-943)

@sudo87

sudo87 commented May 28, 2026

Copy link
Copy Markdown
Contributor Author

I fail to see how this is an improvement. I would argue other lebals need an extra . after gpu...

label.gpudisplay is currently used in the offering details and is defined here:

var fields = ['name', 'id', 'displaytext', 'offerha', 'provisioningtype', 'storagetype', 'iscustomized', 'iscustomizediops', 'limitcpuuse', 'cpunumber', 'cpuspeed', 'memory', 'hosttags', 'tags', 'storageaccessgroups', 'storagetags', 'domain', 'zone', 'created', 'dynamicscalingenabled', 'diskofferingstrictness', 'encryptroot', 'purgeresources', 'leaseduration', 'gpucardid', 'gpucardname', 'vgpuprofileid', 'vgpuprofilename', 'gpucount', 'gpudisplay', 'leaseexpiryaction', 'externaldetails']

We need to define 'label.gpudisplay' to render the label value. I avoided duplicating the key 'label.gpu.display' and renamed it to label.gpudisplay to keep it single-sourced.

@weizhouapache weizhouapache added this to the 4.22.2 milestone Jun 29, 2026
@weizhouapache

Copy link
Copy Markdown
Member

@sudo87
can you rebase it with 4.20 or 4.22 ?

@sudo87

sudo87 commented Jun 30, 2026

Copy link
Copy Markdown
Contributor Author

@sudo87 can you rebase it with 4.20 or 4.22 ?

@weizhouapache The following file doesn't exist in either branch, so it doesn't look like there's any need to change the base:

ui/src/components/offering/ComputeOfferingForm.vue

@DaanHoogland DaanHoogland moved this from Backlog to Ready in CloudStack Testing Aug 31, 2026
@abh1sar abh1sar self-assigned this Sep 7, 2026
@abh1sar abh1sar moved this from Ready to In progress in CloudStack Testing Sep 7, 2026

@abh1sar abh1sar 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.

Image LGTM. But this is better rebased to 4.22 as compute.js contains the gpudisplay field in 4.22

@abh1sar abh1sar moved this from In progress to conflict/waiting in CloudStack Testing Sep 7, 2026
@shwstppr
shwstppr changed the base branch from main to 4.22 September 7, 2026 15:51
@shwstppr
shwstppr changed the base branch from 4.22 to main September 7, 2026 15:51
Copilot AI lite review requested due to automatic review settings September 7, 2026 16:52
@sudo87
sudo87 changed the base branch from main to 4.22 September 7, 2026 16:52
@sudo87

sudo87 commented Sep 7, 2026

Copy link
Copy Markdown
Contributor Author

@blueorangutan ui

@blueorangutan

Copy link
Copy Markdown

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

Copilot AI 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.

🟢 Approval recommended

The change is localized and low risk, with only a minor maintainability suggestion around keeping the old i18n key as an alias for compatibility.

Pull request overview

This PR updates the UI i18n translation key used for the GPU Display compute offering option so the label key matches the gpudisplay parameter naming.

Changes:

  • Update the Add Compute Offering form to use $t('label.gpudisplay') for the GPU Display field label.
  • Rename the English locale key from label.gpu.display to label.gpudisplay.
File summaries
File Description
ui/src/views/offering/AddComputeOffering.vue Switches the tooltip label translation key for the gpudisplay field.
ui/public/locales/en.json Renames/introduces the translation key for “GPU Display”.
Review details
  • Files reviewed: 2/2 changed files
  • Comments generated: 1
  • Review effort level: Lite

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread ui/public/locales/en.json
@blueorangutan

Copy link
Copy Markdown

UI build: ✔️
Live QA URL: https://qa.cloudstack.cloud/simulator/pr/13272 (QA-JID-992)

@abh1sar abh1sar moved this from conflict/waiting to In review in CloudStack Testing Sep 8, 2026
@DaanHoogland
DaanHoogland merged commit f725211 into apache:4.22 Sep 8, 2026
@github-project-automation github-project-automation Bot moved this from In review to Done in CloudStack Testing Sep 8, 2026
@DaanHoogland
DaanHoogland deleted the fixGpuLabel branch September 8, 2026 10:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Development

Successfully merging this pull request may close these issues.

9 participants