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: Google Cloud public IP spec #3762

Merged
merged 0 commits into from
Apr 25, 2023
Merged

Conversation

lzap
Copy link
Contributor

@lzap lzap commented Apr 24, 2023

All Pull Requests:

Check all that apply:

  • Have you followed the guidelines in our Contributing document, including the instructions about commit messages?
  • Is this PR to correct an issue?
  • Is this PR an enhancement?

Complete Description of Additions/Changes:

This patch adds Azure public IP address:

Refs https://issues.redhat.com/browse/ESSNTL-4680

Related to: #3741

I am going to do an end-to-end test on GCP shortly.

@lzap lzap force-pushed the gcp-spec branch 3 times, most recently from c6a5018 to 01d1f38 Compare April 25, 2023 06:09
Copy link
Contributor

@xiangce xiangce left a comment

Choose a reason for hiding this comment

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

Other than the flake8 errors, this PR looks good to me. Would you please fix the flake8 error?

insights/specs/default.py Show resolved Hide resolved
@xiangce xiangce merged commit 2dcf9a8 into RedHatInsights:master Apr 25, 2023
@lzap lzap deleted the gcp-spec branch April 25, 2023 09:21
@lzap
Copy link
Contributor Author

lzap commented Apr 25, 2023

Sure, I added the single quotes, I was not sure but it looked like these commands are executed via shell and there is a question mark. Thank you!

xiangce pushed a commit that referenced this pull request Apr 26, 2023
Signed-off-by: Lukas Zapletal <[email protected]>
(cherry picked from commit 2dcf9a8)
xiangce pushed a commit that referenced this pull request Sep 6, 2024
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.

2 participants