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

Support ad directory objects #406

Merged
merged 14 commits into from
Aug 6, 2021
Merged

Conversation

sathish-progress
Copy link
Contributor

@sathish-progress sathish-progress commented Jul 6, 2021

Description

Support ad directory objects

Issues Resolved

#391

Check List

Signed-off-by: Sathish <sbabu@progress.com>
Signed-off-by: Sathish <sbabu@progress.com>
Signed-off-by: Sathish <sbabu@progress.com>
@sathish-progress sathish-progress requested a review from a team as a code owner July 6, 2021 05:43
@sathish-progress sathish-progress added do not merge WIP Work in Progress labels Jul 6, 2021
Signed-off-by: Sathish <sbabu@progress.com>
Signed-off-by: Sathish <sbabu@progress.com>
Signed-off-by: Sathish <sbabu@progress.com>
Signed-off-by: Sathish <sbabu@progress.com>
Signed-off-by: Sathish <sbabu@progress.com>
Signed-off-by: Sathish <sbabu@progress.com>
@sathish-progress sathish-progress added Documentation Version: Bump Minor Used by github.minor_bump_labels to bump the Minor version number. and removed WIP Work in Progress do not merge labels Jul 16, 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.

Please check the comment.

@sathish-progress sathish-progress requested a review from IanMadd July 23, 2021 06:11
Signed-off-by: Ian Maddaus <Ian.Maddaus@progress.com>
@IanMadd IanMadd force-pushed the support-ad-directory-objects branch from fa6512c to 40c4182 Compare July 28, 2021 23:25
@sathish-progress
Copy link
Contributor Author

Please check the comment.

Please review again..

@sonarqubecloud
Copy link

sonarqubecloud bot commented Aug 5, 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

}

variable "sample_directory_object" {
default = "adc07321-ef2b-44d5-a210-559aa5f10f2d"
Copy link
Contributor

Choose a reason for hiding this comment

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

Do we have to specify the hardcoded value? In AWS we specify the hardcoded values in test/integration/configuration/aws_inspec_config.rb

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.

@sathish-progress sathish-progress merged commit f693850 into master Aug 6, 2021
@sathish-progress sathish-progress deleted the support-ad-directory-objects branch August 6, 2021 05:46
@ppradhan9 ppradhan9 linked an issue Sep 24, 2021 that may be closed by this pull request
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Documentation Version: Bump Minor Used by github.minor_bump_labels to bump the Minor version number.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Azure Directory Objects
4 participants