Merge "Fix Launch Instance without Cinder" into stable/newton

This commit is contained in:
Jenkins 2016-09-27 19:48:40 +00:00 committed by Gerrit Code Review
commit 74c83bdcae
4 changed files with 19 additions and 5 deletions

View File

@ -182,7 +182,7 @@
source_type: null,
source: [],
// REQUIRED for JS logic
vol_create: true,
vol_create: false,
// May be null
vol_device_name: 'vda',
vol_delete_on_instance_delete: false,

View File

@ -814,7 +814,7 @@
});
it('sets volume options appropriately', function() {
expect(model.newInstanceSpec.vol_create).toBe(true);
expect(model.newInstanceSpec.vol_create).toBe(false);
expect(model.newInstanceSpec.vol_device_name).toBe('vda');
expect(model.newInstanceSpec.vol_delete_on_instance_delete).toBe(false);
expect(model.newInstanceSpec.vol_size).toBe(1);

View File

@ -401,7 +401,11 @@
function updateBootSourceSelection(selectedSource) {
ctrl.currentBootSource = selectedSource;
$scope.model.newInstanceSpec.vol_create = true;
if ((selectedSource === bootSourceTypes.IMAGE) && $scope.model.volumeBootable) {
$scope.model.newInstanceSpec.vol_create = true;
} else {
$scope.model.newInstanceSpec.vol_create = false;
}
$scope.model.newInstanceSpec.vol_delete_on_instance_delete = false;
changeBootSource(selectedSource);
validateBootSourceType();

View File

@ -229,12 +229,12 @@
var tableKeys = ['available', 'allocated',
'displayedAvailable', 'displayedAllocated'];
it('updates the scope appropriately', function() {
it('updates the scope appropriately, without Cinder available', function() {
var selSource = 'image';
ctrl.updateBootSourceSelection(selSource);
expect(ctrl.currentBootSource).toEqual('image');
expect(scope.model.newInstanceSpec.vol_create).toBe(true);
expect(scope.model.newInstanceSpec.vol_create).toBe(false);
expect(scope.model.newInstanceSpec.vol_delete_on_instance_delete).toBe(false);
// check table data
@ -244,6 +244,16 @@
expect(ctrl.tableBodyCells.length).toBeGreaterThan(0);
});
it('updates the scope appropriately, with Cinder available', function() {
scope.model.volumeBootable = true;
var selSource = 'image';
ctrl.updateBootSourceSelection(selSource);
expect(ctrl.currentBootSource).toEqual('image');
expect(scope.model.newInstanceSpec.vol_create).toBe(true);
expect(scope.model.newInstanceSpec.vol_delete_on_instance_delete).toBe(false);
});
it('should broadcast event when boot source changes', function() {
spyOn(scope, '$broadcast');
scope.$apply();