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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

[#184537961] Moved personal static ips to paas-trusted-people #3163

Merged
merged 1 commit into from
May 25, 2023

Conversation

malcgds
Copy link
Contributor

@malcgds malcgds commented Apr 11, 2023

What

Moved personal static ips from paas-aws-account-wide-terraform, paas-bootstrap and paas-cf as required by this pivotal tracker storey. Made related changes to paas-aws-account-wide-terraform, paas-bootstrap and paas-trusted-people repos.

How to review

Run this branch into a dev env.

Ensure that the PAAS_TRUSTED_PEOPLE_BRANCH var is set to the related branch in the paas-trusted-people repo before uploading the secrets.

Ensure that you have the connectivity to the dev env that you had previously.

Who can review

Anyone with a static ip in the paas-trusted-people repository.

馃毃鈿狅笍 Please do not merge this pull request via the GitHub UI 鈿狅笍馃毃

@malcgds malcgds force-pushed the 184537961-mv-personal-ips-from-repos branch 4 times, most recently from 52d9bb9 to b2faea5 Compare April 12, 2023 10:08
@malcgds malcgds force-pushed the 184537961-mv-personal-ips-from-repos branch from b2faea5 to 47e6cc8 Compare April 12, 2023 15:54
@malcgds malcgds force-pushed the 184537961-mv-personal-ips-from-repos branch from 47e6cc8 to 7cc5198 Compare April 21, 2023 07:39
@malcgds malcgds force-pushed the 184537961-mv-personal-ips-from-repos branch 6 times, most recently from d54c072 to 9602d5e Compare May 4, 2023 13:29
@whi-tw whi-tw force-pushed the 184537961-mv-personal-ips-from-repos branch 2 times, most recently from f2de838 to b167482 Compare May 19, 2023 09:59
@whi-tw
Copy link
Member

whi-tw commented May 19, 2023

Rebased against main

@whi-tw whi-tw force-pushed the 184537961-mv-personal-ips-from-repos branch from b167482 to 668281a Compare May 22, 2023 14:44
Copy link
Member

@whi-tw whi-tw left a comment

Choose a reason for hiding this comment

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

LGTM - tested in dev03 and working

@malcgds malcgds force-pushed the 184537961-mv-personal-ips-from-repos branch from 668281a to ef51a71 Compare May 24, 2023 08:44
@malcgds malcgds force-pushed the 184537961-mv-personal-ips-from-repos branch from ef51a71 to f22cf73 Compare May 24, 2023 08:52
@malcgds malcgds merged commit 5720cfa into main May 25, 2023
8 checks passed
@malcgds malcgds deleted the 184537961-mv-personal-ips-from-repos branch May 25, 2023 11:07
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.

None yet

3 participants