Skip to content

ui: Honour default.ui.page.size#5394

Merged
yadvr merged 3 commits into
apache:4.15from
shapeblue:fix-ui-fetch-all
Sep 3, 2021
Merged

ui: Honour default.ui.page.size#5394
yadvr merged 3 commits into
apache:4.15from
shapeblue:fix-ui-fetch-all

Conversation

@davidjumani
Copy link
Copy Markdown
Contributor

@davidjumani davidjumani commented Sep 1, 2021 •
edited
Loading

Fixes #5338

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)

Bug Severity

  • BLOCKER
  • Critical
  • Major
  • Minor
  • Trivial

@davidjumani
Copy link
Copy Markdown
Contributor Author

davidjumani commented Sep 1, 2021

@blueorangutan ui

@blueorangutan
Copy link
Copy Markdown

blueorangutan commented Sep 1, 2021

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

@blueorangutan
Copy link
Copy Markdown

blueorangutan commented Sep 1, 2021

UI build: ✔️
Live QA URL: http://qa.cloudstack.cloud:8080/client/pr/5394 (SL-JID-586)

@davidjumani davidjumani added this to the 4.15.2.0 milestone Sep 1, 2021
@davidjumani davidjumani mentioned this pull request Sep 1, 2021
Closed
shwstppr
shwstppr reviewed Sep 1, 2021
View reviewed changes
Comment thread engine/schema/src/main/resources/META-INF/db/schema-41500to41510.sql Outdated
-- Add support for windows2019srvNext_64Guest from VMware 7.0.1.0
INSERT INTO `cloud`.`guest_os_hypervisor` (uuid,hypervisor_type, hypervisor_version, guest_os_name, guest_os_id, created, is_user_defined) VALUES (UUID(),'VMware', '7.0.1.0', 'windows2019srvNext_64Guest', 276, now(), 0);

UPDATE `cloud`.`configuration` SET `value` = 20, `default_value` = 20 WHERE `name` = 'default.ui.page.size';
Copy link
Copy Markdown
Contributor

@shwstppr shwstppr Sep 1, 2021 •
edited
Loading

Choose a reason for hiding this comment

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

@davidjumani won't we have to create a 41510to41520 upgrade path for it to work for 4.15.1 to 4.15.2 upgrade?

Copy link
Copy Markdown
Member

@yadvr yadvr Sep 1, 2021

Choose a reason for hiding this comment

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

If config key exists we shouldn't need it

Copy link
Copy Markdown
Contributor Author

@davidjumani davidjumani Sep 1, 2021

Choose a reason for hiding this comment

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

Yes, will make the change

utchoang
utchoang approved these changes Sep 1, 2021
View reviewed changes
Copy link
Copy Markdown

@utchoang utchoang left a comment

Choose a reason for hiding this comment

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

UI LGTM!

@yadvr
Copy link
Copy Markdown
Member

yadvr commented Sep 1, 2021

@davidjumani pl remove changes for 3121, I can't accept too many core UI changes closer to the RC.

@davidjumani
Copy link
Copy Markdown
Contributor Author

davidjumani commented Sep 1, 2021

Converting to draft until the sql code is moved to the right path

@davidjumani davidjumani marked this pull request as draft September 1, 2021 11:54
@yadvr
Copy link
Copy Markdown
Member

yadvr commented Sep 2, 2021

@davidjumani can you only keep changes strictly for #5338 and remove any other changes?

@davidjumani
Copy link
Copy Markdown
Contributor Author

davidjumani commented Sep 2, 2021

@blueorangutan ui

@blueorangutan
Copy link
Copy Markdown

blueorangutan commented Sep 2, 2021

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

@blueorangutan
Copy link
Copy Markdown

blueorangutan commented Sep 2, 2021

UI build: ✔️
Live QA URL: http://qa.cloudstack.cloud:8080/client/pr/5394 (SL-JID-593)

@davidjumani davidjumani changed the title ui: Fetch all records && honour default.ui.page.size ui: Honour default.ui.page.size Sep 2, 2021
@yadvr
Copy link
Copy Markdown
Member

yadvr commented Sep 2, 2021

@davidjumani check latest 4.15, upgrade path is now available.

@davidjumani davidjumani marked this pull request as ready for review September 3, 2021 04:21
@davidjumani
Copy link
Copy Markdown
Contributor Author

davidjumani commented Sep 3, 2021

@blueorangutan ui

@blueorangutan
Copy link
Copy Markdown

blueorangutan commented Sep 3, 2021

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

@blueorangutan
Copy link
Copy Markdown

blueorangutan commented Sep 3, 2021

UI build: ✔️
Live QA URL: http://qa.cloudstack.cloud:8080/client/pr/5394 (SL-JID-598)

@yadvr
Copy link
Copy Markdown
Member

yadvr commented Sep 3, 2021

Need one more review/test - @davidjumani @shwstppr @nvazquez @sureshanaparti...

@davidjumani
Copy link
Copy Markdown
Contributor Author

davidjumani commented Sep 3, 2021

@blueorangutan package

@blueorangutan
Copy link
Copy Markdown

blueorangutan commented Sep 3, 2021

@davidjumani a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

@blueorangutan
Copy link
Copy Markdown

blueorangutan commented Sep 3, 2021

Packaging result: ✔️ el7 ✔️ el8 ✔️ debian. SL-JID 1124

shwstppr
shwstppr approved these changes Sep 3, 2021
View reviewed changes
Copy link
Copy Markdown
Contributor

@shwstppr shwstppr left a comment

Choose a reason for hiding this comment

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

Test upgrade from 4.15.1.
Config value updated to 20. Config is honored in the UI listviews

@yadvr yadvr merged commit 0bc6e05 into apache:4.15 Sep 3, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@yadvr yadvr yadvr left review comments

@shwstppr shwstppr shwstppr approved these changes

@utchoang utchoang utchoang approved these changes

@nvazquez nvazquez Awaiting requested review from nvazquez

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

4.15.2.0

Development

Successfully merging this pull request may close these issues.

5 participants

@davidjumani @blueorangutan @yadvr @shwstppr @utchoang