bugfix: multicontainer honors slots

src/command_modules/azure-cli-appservice/HISTORY.rst

@@ -2,6 +2,10 @@
 
 Release History
 ===============
+0.2.9
++++++
+* webapp: az webapp config container now honors --slot parameter
+
 0.2.8
 +++++
 * webapp: adding support for az webapp up command (Preview) that helps in creating & deploying contents to app

src/command_modules/azure-cli-appservice/azure/cli/command_modules/appservice/custom.py

@@ -797,11 +797,12 @@ def update_container_settings(cmd, resource_group_name, name, docker_registry_se
     if multicontainer_config_file and multicontainer_config_type:
         encoded_config_file = _get_linux_multicontainer_encoded_config_from_file(multicontainer_config_file)
         linux_fx_version = _format_fx_version(encoded_config_file, multicontainer_config_type)
-        update_site_configs(cmd, resource_group_name, name, linux_fx_version=linux_fx_version)
+        update_site_configs(cmd, resource_group_name, name, linux_fx_version=linux_fx_version, slot=slot)
     elif multicontainer_config_file or multicontainer_config_type:
         logger.warning('Must change both settings --multicontainer-config-file FILE --multicontainer-config-type TYPE')
 
-    return _mask_creds_related_appsettings(_filter_for_container_settings(cmd, resource_group_name, name, settings))
+    return _mask_creds_related_appsettings(_filter_for_container_settings(cmd, resource_group_name, name, settings,
+                                                                          slot=slot))
 
 
 def _get_acr_cred(cli_ctx, registry_name):

src/command_modules/azure-cli-appservice/azure/cli/command_modules/appservice/tests/latest/sample-compose-slot.yml

@@ -0,0 +1,8 @@
+version: '3'
+services:
+  web:
+    image: "patle/python_app_slot:latest"
+    ports:
+     - "5000:5000"
+  redis:
+    image: "redis:alpine"

src/command_modules/azure-cli-appservice/azure/cli/command_modules/appservice/tests/latest/test_webapp_commands.py

@@ -480,6 +480,43 @@ def test_linux_webapp(self, resource_group):
         self.assertEqual(result2, [])
 
 
+class LinuxWebappMulticontainerSlotScenarioTest(ScenarioTest):
+    @ResourceGroupPreparer()
+    def test_linux_webapp_multicontainer_slot(self, resource_group):
+        webapp_name = self.create_random_name(prefix='webapp-linux-multi', length=24)
+        plan = self.create_random_name(prefix='plan-linux-multi', length=24)
+        config_file = os.path.join(TEST_DIR, 'sample-compose.yml')
+        slot = "stage"
+        slot_webapp_name = "{}-{}".format(webapp_name, slot)
+        slot_config_file = os.path.join(TEST_DIR, 'sample-compose-slot.yml')
+
+        self.cmd('appservice plan create -g {} -n {} --is-linux --sku S1'.format(resource_group, plan))
+        self.cmd("webapp create -g {} -n {} --plan {} --multicontainer-config-file \"{}\" "
+                 "--multicontainer-config-type COMPOSE".format(resource_group, webapp_name, plan, config_file))
+
+        last_number_seen = 99999999
+        for x in range(0, 10):
+            r = requests.get('http://{}.azurewebsites.net'.format(webapp_name), timeout=240)
+            # verify the web page
+            self.assertTrue('Hello World! I have been seen' in str(r.content))
+            current_number = [int(s) for s in r.content.split() if s.isdigit()][0]
+            self.assertNotEqual(current_number, last_number_seen)
+            last_number_seen = current_number
+
+        self.cmd('webapp deployment slot create -g {} -n {} --slot {}'.format(resource_group, webapp_name, slot))
+        self.cmd("webapp config container set -g {} -n {} --slot {} --multicontainer-config-file \"{}\" "
+                 "--multicontainer-config-type COMPOSE".format(resource_group, webapp_name, slot, slot_config_file))
+
+        last_number_seen = 99999999
+        for x in range(0, 10):
+            r = requests.get('http://{}.azurewebsites.net'.format(slot_webapp_name), timeout=240)
+            # verify the web page
+            self.assertTrue('Hello from a slot! I have been seen' in str(r.content))
+            current_number = [int(s) for s in r.content.split() if s.isdigit()][0]
+            self.assertNotEqual(current_number, last_number_seen)
+            last_number_seen = current_number
+
+
 class WebappACRScenarioTest(ScenarioTest):
     @ResourceGroupPreparer(location='japanwest')
     def test_acr_integration(self, resource_group):

src/command_modules/azure-cli-appservice/setup.py

@@ -14,7 +14,7 @@
     logger.warn("Wheel is not available, disabling bdist_wheel hook")
     cmdclass = {}
 
-VERSION = "0.2.8"
+VERSION = "0.2.9"
 CLASSIFIERS = [
     'Development Status :: 4 - Beta',
     'Intended Audience :: Developers',