Skip to content

Deprecate EL6 and Add 4.13-4.14 Upgrade Path #3591

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.

Sign up for GitHub

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

Jump to bottom
Merged
yadvr merged 4 commits into from
Sep 19, 2019
Merged

Deprecate EL6 and Add 4.13-4.14 Upgrade Path #3591

yadvr merged 4 commits into from
Sep 19, 2019

Conversation

@yadvr
Copy link
Member

@yadvr yadvr commented Sep 10, 2019 •
edited
Loading

This PR wants to do the following on 4.14/master:

@yadvr
remove support for el6 packaging
65d673f 
Signed-off-by: Rohit Yadav <rohit.yadav@shapeblue.com>
@yadvr yadvr added this to the 4.14.0.0 milestone Sep 10, 2019
@yadvr
add upgrade path
f155caa 
Signed-off-by: Rohit Yadav <rohit.yadav@shapeblue.com>
@yadvr
el7+: support live storage migration for kvm by default
c7ffb3e 
Considering that we have addressed a considerable effort on allowing KVM
to perform storage data motion, this PR proposes updating the
'hypervisor_capabilities' table setting the 'storage_motion_supported'
to '1' for KVM.

PRs that implemented KVM storage motion features:

Non-managed storages
 apache#2997 KVM VM live migration with ROOT volume on file storage type
 apache#2983 KVM live storage migration intra cluster from NFS source and destination
Managed storages
 apache#2298 CLOUDSTACK-9620: Enhancements for managed storage

Signed-off-by: Rohit Yadav <rohit.yadav@shapeblue.com>
@yadvr yadvr changed the title [WIP] Deprecate EL6, move to JDK11, Debian 10 [WIP] Deprecate EL6 Sep 18, 2019
@yadvr yadvr changed the title [WIP] Deprecate EL6 Deprecate EL6 and Add 4.13-4.14 Upgrade Path Sep 18, 2019
@yadvr
remove unused imports
13bc02b 
Signed-off-by: Rohit Yadav <rohit.yadav@shapeblue.com>
@yadvr
Copy link
Member Author

yadvr commented Sep 18, 2019

Moved #3508 to the 4.13->4.14 upgrade path @GabrielBrascher

@yadvr
Copy link
Member Author

yadvr commented Sep 18, 2019

(el6 pkg failure is expected)
@blueorangutan package

@blueorangutan
Copy link

blueorangutan commented Sep 18, 2019

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

wido
wido approved these changes Sep 18, 2019
View reviewed changes
Copy link
Contributor

@wido wido left a comment

Choose a reason for hiding this comment

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

LGTM

@blueorangutan
Copy link

blueorangutan commented Sep 18, 2019

Packaging result: ✖centos6 ✔centos7 ✔debian. JID-281

@yadvr
Copy link
Member Author

yadvr commented Sep 18, 2019

EL6 failure LGTM ;)

@blueorangutan test

@blueorangutan
Copy link

blueorangutan commented Sep 18, 2019

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

yadvr
yadvr commented Sep 18, 2019
View reviewed changes
engine/schema/src/main/resources/META-INF/db/schema-41300to41400.sql
-- Schema upgrade from 4.13.0.0 to 4.14.0.0
--;

-- KVM: enable storage data motion on KVM hypervisor_capabilities
Copy link
Member Author

@yadvr yadvr Sep 18, 2019

Choose a reason for hiding this comment

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

@GabrielBrascher cc @wido this can now be safely enabled by default for KVM as el6 is no longer supported :)

@blueorangutan
Copy link

blueorangutan commented Sep 18, 2019

Trillian test result (tid-371)
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 31728 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr3591-t371-kvm-centos7.zip
Intermittent failure detected: /marvin/tests/smoke/test_vpc_redundant.py
Smoke tests completed. 76 look OK, 1 have error(s)
Only failed tests results shown below:

Test Result Time (s) Test File
test_04_rvpc_network_garbage_collector_nics Failure 310.50 test_vpc_redundant.py

shwstppr
shwstppr approved these changes Sep 19, 2019
View reviewed changes
Copy link
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.

LGTM based on changes!

@yadvr yadvr merged commit 89053c7 into apache:master Sep 19, 2019
@yadvr yadvr deleted the deprecate-centos6 branch September 19, 2019 04:31
@yadvr yadvr mentioned this pull request Sep 19, 2019
Closed
5 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@shwstppr shwstppr shwstppr approved these changes

@wido wido wido approved these changes

@andrijapanicsb andrijapanicsb Awaiting requested review from andrijapanicsb

@borisstoyanov borisstoyanov Awaiting requested review from borisstoyanov

@GabrielBrascher GabrielBrascher Awaiting requested review from GabrielBrascher

@kiwiflyer kiwiflyer Awaiting requested review from kiwiflyer

@nathanejohnson nathanejohnson Awaiting requested review from nathanejohnson

@nvazquez nvazquez Awaiting requested review from nvazquez

@PaulAngus PaulAngus Awaiting requested review from PaulAngus

@svenvogel svenvogel Awaiting requested review from svenvogel

Assignees

No one assigned

Labels

type:cleanup

Projects

None yet

Milestone

4.14.0.0

Development

Successfully merging this pull request may close these issues.

4 participants

@yadvr @blueorangutan @shwstppr @wido