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

Bump Inspec, Adds Azurerm Disks Plural resource #215

Merged
8 commits merged into from
Nov 6, 2019
Merged

Conversation

ghost
Copy link

@ghost ghost commented Oct 30, 2019

Signed-off-by: Ruairi Fennell rfennell@chef.io

Description

  • Make use of latest Inspec via inspec-bin gem.
  • Adds azurerm_virtual_machine_disks plural resource w/ tests, docs, etc.
  • Adds Azure ID to_h conversion to base azurerm_resource class to remove growing number of 'magic number' string manipulations (see example).

Check List

Ruairi Fennell added 2 commits October 30, 2019 17:30
…es magic number id splitting

Signed-off-by: Ruairi Fennell <rfennell@chef.io>
Signed-off-by: Ruairi Fennell <rfennell@chef.io>
@ghost ghost added the do not merge label Oct 30, 2019
@ghost ghost self-requested a review October 30, 2019 17:41
@ghost ghost added the Version: Bump Minor Used by github.minor_bump_labels to bump the Minor version number. label Oct 30, 2019
@@ -36,7 +37,7 @@ def to_s
def with_resource_group
lambda do |group|
# Get resource group from ID string
name = group.id.split('/')[4]
name = id_to_h(group.id)[:resource_group]
Copy link
Author

Choose a reason for hiding this comment

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

.

@ghost ghost removed the do not merge label Oct 31, 2019
Signed-off-by: Ruairi Fennell <rfennell@chef.io>
Copy link

@skpaterson skpaterson left a comment

Choose a reason for hiding this comment

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

Great work @r-fennell !

libraries/azurerm_resource.rb Outdated Show resolved Hide resolved
docs/resource_creation_guide.md Outdated Show resolved Hide resolved
Ruairi Fennell added 2 commits November 5, 2019 16:05
Signed-off-by: Ruairi Fennell <rfennell@chef.io>
Signed-off-by: Ruairi Fennell <rfennell@chef.io>
Ruairi Fennell and others added 3 commits November 5, 2019 16:20
Signed-off-by: Ruairi Fennell <rfennell@chef.io>
@ghost ghost merged commit 539be72 into master Nov 6, 2019
@ghost ghost deleted the bump-inspec-add-disks branch November 6, 2019 10:49
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant