Skip to content

apply some code fixes

Yukuo Wang requested to merge fix-code into master

All Submissions:

  • [YES] Have you added an explanation of what your changes do and why you'd like us to include them?
  • [YES] I have updated the documentation accordingly.
  • [YES] My code follows the code style of this project.

Current Behavior or Linked Issues

  • The default value given for variable "redis_cache_tier" has miss-leading values. value for zones needs to be null when sku_name is not Premium, [] is a non-acceptable value even if sku_name = Premium.
  • Error in test code for Module/container-registry tests.name is not declared by package github.com/microsoft/cobalt/infra/modules/providers/azure/storage-account/tests while checking in the package it should be tests.ContainerName

Does this introduce a breaking change?

  • [NO]

Other information

Merge request reports