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

Feat add gcp support #340

Merged
merged 15 commits into from
Sep 19, 2023
Merged

Feat add gcp support #340

merged 15 commits into from
Sep 19, 2023

Conversation

osamamagdy
Copy link

What kind of changes does this PR include?

  • Platform support

Checklist:

  • All the contributions made are solely the work of me and my co-authors
  • I tested the changes in this PR (if applicable)
  • I added tests to ensure my change works (if applicable)
  • The PR passes pre-commit hooks and automated tests

Add Wrongsecrets files

Signed-off-by: osamamagdy <[email protected]>
Enable autoscaling and csi driver

Signed-off-by: osamamagdy <[email protected]>
Signed-off-by: osamamagdy <[email protected]>
Signed-off-by: osamamagdy <[email protected]>
Signed-off-by: osamamagdy <[email protected]>
Copy link
Collaborator

@commjoen commjoen left a comment

Choose a reason for hiding this comment

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

Initial review done, that's a big lift @osamamagdy ! Thank you!
Can you please have a look at the feedback?

Signed-off-by: osamamagdy <[email protected]>
…crets-ctf-party into feat--add-gcp-support

Signed-off-by: osamamagdy <[email protected]>
@osamamagdy osamamagdy force-pushed the feat--add-gcp-support branch from 0df4463 to 6f68c47 Compare September 16, 2023 12:15
@osamamagdy osamamagdy requested a review from commjoen September 16, 2023 16:30
Copy link
Collaborator

@bendehaan bendehaan left a comment

Choose a reason for hiding this comment

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

One minor nit

@osamamagdy osamamagdy force-pushed the feat--add-gcp-support branch 2 times, most recently from 9825fa5 to 37866f8 Compare September 18, 2023 22:35
@osamamagdy osamamagdy force-pushed the feat--add-gcp-support branch from 2441fa1 to b86f930 Compare September 18, 2023 22:37
Signed-off-by: osamamagdy <[email protected]>
@osamamagdy osamamagdy force-pushed the feat--add-gcp-support branch from b86f930 to 41f8627 Compare September 18, 2023 22:53
Copy link
Collaborator

@commjoen commjoen left a comment

Choose a reason for hiding this comment

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

Thank you @osamamagdy !

@commjoen commjoen merged commit c32f3fc into OWASP:main Sep 19, 2023
@osamamagdy
Copy link
Author

Fixes #157

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.

3 participants