Skip to content

DB: fix wrong category id of guest os 'Other PV Virtio-SCSI (64-bit)'#4426

Merged
DaanHoogland merged 1 commit into
apache:masterfrom
ustcweizhou:4.15-db-fix-category-other-pv-virtio-scsi
Oct 29, 2020
Merged

DB: fix wrong category id of guest os 'Other PV Virtio-SCSI (64-bit)'#4426
DaanHoogland merged 1 commit into
apache:masterfrom
ustcweizhou:4.15-db-fix-category-other-pv-virtio-scsi

Conversation

@ustcweizhou

Copy link
Copy Markdown
Contributor

Description

The guest os 'Other PV Virtio-SCSI (64-bit)' has category_id as 6 ("Windows"), which should be 7 ("Other").

mysql> select * from guest_os where id=275\G
*************************** 1. row ***************************
             id: 275
    category_id: 6
           name: NULL
           uuid: 5f080e85-c0fd-11ea-ba74-067606003392
   display_name: Other PV Virtio-SCSI (64-bit)
        created: 2020-07-08 09:28:24
        removed: NULL
is_user_defined: 0
1 row in set (0.00 sec)

mysql> select * from guest_os_category;
+----+---------+--------------------------------------+
| id | name    | uuid                                 |
+----+---------+--------------------------------------+
|  1 | CentOS  | 45bfdf47-c0fd-11ea-ba74-067606003392 |
|  2 | Debian  | 45c0529b-c0fd-11ea-ba74-067606003392 |
|  3 | Oracle  | 45c179a5-c0fd-11ea-ba74-067606003392 |
|  4 | RedHat  | 45c1b328-c0fd-11ea-ba74-067606003392 |
|  5 | SUSE    | 45c225de-c0fd-11ea-ba74-067606003392 |
|  6 | Windows | 45c254f2-c0fd-11ea-ba74-067606003392 |
|  7 | Other   | 45c2be11-c0fd-11ea-ba74-067606003392 |
|  8 | Novel   | 45c2efe7-c0fd-11ea-ba74-067606003392 |
|  9 | Unix    | 45c35920-c0fd-11ea-ba74-067606003392 |
| 10 | Ubuntu  | 45c387e4-c0fd-11ea-ba74-067606003392 |
| 11 | None    | 45c3f75e-c0fd-11ea-ba74-067606003392 |
+----+---------+--------------------------------------+
11 rows in set (0.01 sec)

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)

Screenshots (if appropriate):

How Has This Been Tested?

@yadvr

yadvr commented Oct 28, 2020

Copy link
Copy Markdown
Member

@ustcweizhou can you fix the conflict?

@yadvr yadvr added this to the 4.15.0.0 milestone Oct 28, 2020
@yadvr

yadvr commented Oct 28, 2020

Copy link
Copy Markdown
Member

This change may affect users on upgraded, we may need to ask others for review @PaulAngus @DaanHoogland @wido @GabrielBrascher et al thnx

@yadvr yadvr requested a review from sureshanaparti October 28, 2020 10:40

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

LGTM after conflict is resolved

@DaanHoogland

Copy link
Copy Markdown
Contributor

this is a good fix I don't see how it can hurt upgrades. templates that are Other PV Virtio-SCSI (64-bit) would now be deployed as Windows and should be Other. What is your concern, @rhtyd?

@ustcweizhou ustcweizhou force-pushed the 4.15-db-fix-category-other-pv-virtio-scsi branch from c61fb00 to 01e4a4d Compare October 28, 2020 13:48
@DaanHoogland

Copy link
Copy Markdown
Contributor

@blueorangutan package

@blueorangutan

Copy link
Copy Markdown

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

@blueorangutan

Copy link
Copy Markdown

Packaging result: ✔centos7 ✔centos8 ✔debian. JID-2301

@DaanHoogland

Copy link
Copy Markdown
Contributor

@blueorangutan test

@blueorangutan

Copy link
Copy Markdown

@DaanHoogland a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests

@DaanHoogland

Copy link
Copy Markdown
Contributor

bo failed to upload results: no regressions:

Intermittent failure detected: /marvin/tests/smoke/test_privategw_acl.py
Smoke tests completed. 86 look OK, 0 have error(s)
Only failed tests results shown below:


Test | Result | Time (s) | Test File
--- | --- | --- | ---

merging

@DaanHoogland DaanHoogland merged commit e447764 into apache:master Oct 29, 2020
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.

7 participants