diff options
author | Zuul <zuul@review.opendev.org> | 2023-02-24 00:51:59 +0000 |
---|---|---|
committer | Gerrit Code Review <review@openstack.org> | 2023-02-24 00:51:59 +0000 |
commit | 3e27d35facb6b480aada607aa2e1894b75b2ce23 (patch) | |
tree | d1170abea1ae72928c8eb642874fdb6a5c30676d | |
parent | 3701f2d5cf404daff358f8a3f88c5ec5345188ac (diff) | |
parent | ad778d75cfcb99ca9bb0ae7e313f2cbb86f4e416 (diff) | |
download | horizon-3e27d35facb6b480aada607aa2e1894b75b2ce23.tar.gz |
Merge "Fix applying OPENSTACK_SERVER_DEFAULT_USER_DATA value" into stable/zed
4 files changed, 18 insertions, 7 deletions
diff --git a/doc/source/configuration/settings.rst b/doc/source/configuration/settings.rst index 512858256..fd5fbe9be 100644 --- a/doc/source/configuration/settings.rst +++ b/doc/source/configuration/settings.rst @@ -2172,7 +2172,7 @@ LAUNCH_INSTANCE_DEFAULTS Added the ``default_availability_zone`` option. -.. versionchanged:: 22.3.0(Zed) +.. versionchanged:: 23.0.0(Zed) Added the ``enable_metadata``, ``enable_net_ports`` options. @@ -2281,7 +2281,7 @@ launching an instance. enable_metadata ############### -.. versionadded:: 22.3.0(Zed) +.. versionadded:: 23.0.0(Zed) Default: ``True`` @@ -2291,7 +2291,7 @@ an instance. enable_net_ports ################ -.. versionadded:: 22.3.0(Zed) +.. versionadded:: 23.0.0(Zed) Default: ``True`` @@ -2317,7 +2317,7 @@ is specified. OPENSTACK_SERVER_DEFAULT_USER_DATA ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ -.. versionadded:: 22.3.0(Zed) +.. versionadded:: 23.0.0(Zed) Default: ``""`` diff --git a/openstack_dashboard/dashboards/project/static/dashboard/project/workflow/launch-instance/launch-instance-model.service.js b/openstack_dashboard/dashboards/project/static/dashboard/project/workflow/launch-instance/launch-instance-model.service.js index 26086bc14..b11af5ebb 100644 --- a/openstack_dashboard/dashboards/project/static/dashboard/project/workflow/launch-instance/launch-instance-model.service.js +++ b/openstack_dashboard/dashboards/project/static/dashboard/project/workflow/launch-instance/launch-instance-model.service.js @@ -252,7 +252,7 @@ }); settings.getSetting("OPENSTACK_SERVER_DEFAULT_USER_DATA").then( function (response) { - model.newInstanceSpec.default_user_data = response.OPENSTACK_SERVER_DEFAULT_USER_DATA; + model.newInstanceSpec.default_user_data = response; }); promise = $q.all([ diff --git a/openstack_dashboard/dashboards/project/static/dashboard/project/workflow/launch-instance/launch-instance-model.service.spec.js b/openstack_dashboard/dashboards/project/static/dashboard/project/workflow/launch-instance/launch-instance-model.service.spec.js index fa15af910..9570c0591 100644 --- a/openstack_dashboard/dashboards/project/static/dashboard/project/workflow/launch-instance/launch-instance-model.service.spec.js +++ b/openstack_dashboard/dashboards/project/static/dashboard/project/workflow/launch-instance/launch-instance-model.service.spec.js @@ -186,8 +186,7 @@ disable_volume_snapshot: false, default_availability_zone: 'Any' }, - DEFAULT_BOOT_SOURCE: 'image', - OPENSTACK_SERVER_DEFAULT_USER_DATA: '' + DEFAULT_BOOT_SOURCE: 'image' }; IMAGE = {type: 'image', label: 'Image', selected: true}; VOLUME = {type: 'volume', label: 'Volume', selected: false}; @@ -552,6 +551,13 @@ expect(model.newInstanceSpec.hide_create_volume).toBe(true); }); + it('should default user_data based on setting', function() { + settings.OPENSTACK_SERVER_DEFAULT_USER_DATA = 'default-data'; + model.initialize(true); + scope.$apply(); + expect(model.newInstanceSpec.default_user_data).toBe('default-data'); + }); + it('should not set availability zone if the zone list is empty', function () { spyOn(novaApi, 'getAvailabilityZones').and.callFake(function () { var deferred = $q.defer(); diff --git a/releasenotes/notes/fix-default-userdata-3cb75b85b1a5708f.yaml b/releasenotes/notes/fix-default-userdata-3cb75b85b1a5708f.yaml new file mode 100644 index 000000000..9c95b0cd2 --- /dev/null +++ b/releasenotes/notes/fix-default-userdata-3cb75b85b1a5708f.yaml @@ -0,0 +1,5 @@ +--- +fixes: + - Fixes the bug where an administrator could not specify default user_data + for new VMs via ``OPENSTACK_SERVER_DEFAULT_USER_DATA`` configuration + setting. |