Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

azure module #736

Closed
dereksharman opened this issue Dec 11, 2020 · 1 comment · Fixed by #757
Closed

azure module #736

dereksharman opened this issue Dec 11, 2020 · 1 comment · Fixed by #757
Labels
Azure bug Something isn't working

Comments

@dereksharman
Copy link

func GetVirtualMachineTagsE(vmName string, resGroupName string, subscriptionID string) (map[string]string, error) {

map has not been properly intialized

func GetVirtualMachineTagsE(vmName string, resGroupName string, subscriptionID string) (map[string]string, error) {
// Setup a blank map to populate and return
var tags map[string]string

// Get VM Object
vm, err := GetVirtualMachineE(vmName, resGroupName, subscriptionID)
if err != nil {
	return nil, err
}

// Range through existing tags and populate above map accordingly
for k, v := range vm.Tags {
	tags[k] = *v
}

return tags, nil

}

@yorinasub17 yorinasub17 added Azure bug Something isn't working labels Dec 11, 2020
@rguthriemsft
Copy link
Contributor

@davesee Please take a look. We will get this fixed.

yorinasub17 pushed a commit that referenced this issue Feb 11, 2021
* Updated VM tags module and tests

* Enforce complex password requirement

* Added tag variable

* Enforce complex password requirement mysqldb

* Added missing build tag resgrp_test

* Updated unit test

* Updated module and reverted unit test.

* Updated revert.

* Updated tag test

* Updated to use output, removed additional updates

* Removed extra changes
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Azure bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants