Skip to content
This repository has been archived by the owner on Jun 17, 2024. It is now read-only.

Fix for issues # 59 and 60, reading cluster value in the state file #62

Merged
merged 1 commit into from
Apr 20, 2020

Conversation

Prativa20
Copy link
Contributor

  1. https://github.com/terraform-providers/terraform-provider-vra7/issues/60
    Replace the default cluster only if cluster is pecified in the config file

  2. https://github.com/terraform-providers/terraform-provider-vra7/issues/59
    Modify the logic to retrieve cluster value, not using the Scale Out template

  3. Replace the default business group id in the action template only if specified in the config

1. https://github.com/terraform-providers/terraform-provider-vra7/issues/60
   Replace the default cluster only if cluster is pecified in the config file

2. https://github.com/terraform-providers/terraform-provider-vra7/issues/59
   Modify the logic to retrieve cluster value, not using the Scale Out template

3. Replace the default business group id in the action template only if specified in the config
@ghost ghost added the size/M label Apr 20, 2020
@Prativa20 Prativa20 requested a review from markpeek April 20, 2020 17:50
Copy link
Contributor

@markpeek markpeek left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@Prativa20 Prativa20 closed this Apr 20, 2020
@Prativa20 Prativa20 reopened this Apr 20, 2020
@Prativa20 Prativa20 merged commit e07d8f4 into master Apr 20, 2020
@Prativa20 Prativa20 deleted the prativa20-crash-bug branch April 20, 2020 18:59
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants