Upgrade the container/compute library to 0.32.0

[brendandburns]

@derekbekoe @yugangw-msft

[tjprescott]

It looks like there are some Pylint errors and at least one test (test_acs_create_update) needs to be re-recorded due to the SDK update.

[yugangw-msft]

@brendandburns, please add a new test step for list in the test case and re-record

[derekbekoe]

Change this to cli_command('acs list', list_container_services, factory) as line 114 already createes to container_services client.

[brendandburns]

done.

[derekbekoe]

... Then in this method:

• Remove ccf = _compute_client_factory() and ccf.container_services
• The first argument will now be the container service client.

def list_container_services(client, resource_group_name=None):
   svc_list = client.list_by_resource_group(resource_group_name=resource_group_name) \
       if resource_group_name else ccf.container_services.list()
    return list(svc_list)

[brendandburns]

done.

[brendandburns]

comments addressed, ci/cd fixed (i hope), please take another look.

Thanks!

[yugangw-msft]

there are lint errors still:

python -m pylint -d I0013 -r n /home/travis/build/Azure/azure-cli/src/command_modules/azure-cli-vm/azure
************* Module azure.cli.command_modules.vm.custom
C:1021, 0: Wrong continued indentation (add 1 space).
                                                     container_service_name, instance)
                                                     ^| (bad-continuation)
C:1025, 0: Line too long (106/100) (line-too-long)

[brendandburns]

@yugangw-msft all passing now, sorry for the issues.

Thanks!
--brendan

[yugangw-msft]

With your change at commands.py to reuse the factory object, the client now points to container_services already, so this line might be broken. Please double check. Also it won't hurt to invoke the list command in the e2e test case.
I usually avoid passing in operation_group object as client to custom commands, because of the inconsistency, hence confusing.

[yugangw-msft]

@brendandburns, see my comments above, please verify list still works

[brendandburns]

@yugangw-msft good catch, fixed.

How do I add it to the e2e test case?

[yugangw-msft]

@brendandburns, add a test step here. I recently added similar testing for webapp. Yes, you need to delete and re-record VCR data.
Since people are waiting for this change, I went ahead and merged it. You can catch up the test coverage in the next PR