diff options
author | Zuul <zuul@review.opendev.org> | 2023-01-27 15:38:33 +0000 |
---|---|---|
committer | Gerrit Code Review <review@openstack.org> | 2023-01-27 15:38:33 +0000 |
commit | 01e6e6c113348341d50848c593d55a6c6a02ebea (patch) | |
tree | 325ae521a3b085ca508f21912f5a528935d99371 | |
parent | 501d446e1e2657e8303a40a4198cf513c5cb867b (diff) | |
parent | c8ee8c1965b280e24fb6266cbb2523230fea605f (diff) | |
download | horizon-01e6e6c113348341d50848c593d55a6c6a02ebea.tar.gz |
Merge "Fix applying OPENSTACK_SERVER_DEFAULT_USER_DATA value"
4 files changed, 18 insertions, 7 deletions
diff --git a/doc/source/configuration/settings.rst b/doc/source/configuration/settings.rst index 055eb4e88..50e045d88 100644 --- a/doc/source/configuration/settings.rst +++ b/doc/source/configuration/settings.rst @@ -2184,7 +2184,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. @@ -2293,7 +2293,7 @@ launching an instance. enable_metadata ############### -.. versionadded:: 22.3.0(Zed) +.. versionadded:: 23.0.0(Zed) Default: ``True`` @@ -2303,7 +2303,7 @@ an instance. enable_net_ports ################ -.. versionadded:: 22.3.0(Zed) +.. versionadded:: 23.0.0(Zed) Default: ``True`` @@ -2329,7 +2329,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 c38d75bc4..0c22a9a65 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 27a580698..52f1f2e76 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 @@ -188,8 +188,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}; @@ -557,6 +556,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. |