Skip to content

Conversation

@Nirbhay1997
Copy link
Contributor

@Nirbhay1997 Nirbhay1997 commented Sep 1, 2021

Description

Please describe what this change achieves. Ensure you have read the Contributing to InSpec AWS document before submitting.

Issues Resolved

List any existing issues this PR resolves, or any Discourse or StackOverflow discussion that's relevant

Check List

Please fill box or appropriate ([x]) or mark N/A.

Signed-off-by: Nirbhay Kumar <nkumar@progress.com>
@Nirbhay1997 Nirbhay1997 requested review from a team as code owners September 1, 2021 07:41
Signed-off-by: Nirbhay Kumar <nkumar@progress.com>
@Nirbhay1997 Nirbhay1997 self-assigned this Sep 1, 2021
Copy link
Contributor

@soumyo13 soumyo13 left a comment

Choose a reason for hiding this comment

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

Add Terraform and Buildkite is failing.

Nirbhay1997 and others added 3 commits September 3, 2021 07:19
Signed-off-by: Nirbhay Kumar <nkumar@progress.com>
Signed-off-by: Nirbhay Kumar <nkumar@progress.com>
Signed-off-by: Nirbhay Kumar <nkumar@progress.com>
Copy link
Contributor

@soumyo13 soumyo13 left a comment

Choose a reason for hiding this comment

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

LGTM

Signed-off-by: Ian Maddaus <Ian.Maddaus@progress.com>
@IanMadd IanMadd force-pushed the Nirbhay/NetworkmanagerDevice branch from 7170080 to 0e24d6f Compare October 4, 2021 21:26
### Ensure a zone is available.

describe aws_network_manager_devices do
its('aws_location.zone') { should include 'ZONE_NAME' }
Copy link
Contributor

Choose a reason for hiding this comment

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

Should this be aws_locations.zone?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

thanks updated

Nirbhay Kumar and others added 4 commits October 5, 2021 16:07
Signed-off-by: Nirbhay Kumar <nkumar@progress.com>
Signed-off-by: Nirbhay Kumar <nkumar@progress.com>
Signed-off-by: Nirbhay Kumar <nkumar@progress.com>
Signed-off-by: Ian Maddaus <Ian.Maddaus@progress.com>
Signed-off-by: Soumyodeep Karmakar <soumyo.k13@gmail.com>
@sonarqubecloud
Copy link

sonarqubecloud bot commented Oct 6, 2021

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@Nirbhay1997 Nirbhay1997 requested a review from IanMadd October 6, 2021 07:50
@Nirbhay1997
Copy link
Contributor Author

@IanMadd @dkumaras can you please re-check?

Signed-off-by: Ian Maddaus <ian.maddaus@progress.com>
soumyo13 and others added 5 commits November 24, 2021 10:28
Signed-off-by: Soumyodeep Karmakar <soumyo.k13@gmail.com>
Signed-off-by: Nirbhay Kumar <nkumar@progress.com>
Signed-off-by: Nirbhay Kumar <nkumar@progress.com>
@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@soumyo13 soumyo13 merged commit dca078d into main Nov 25, 2021
@soumyo13 soumyo13 deleted the Nirbhay/NetworkmanagerDevice branch November 25, 2021 06:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants