-
Notifications
You must be signed in to change notification settings - Fork 1.3k
IP ranges updates #9328
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
base: master
Are you sure you want to change the base?
IP ranges updates #9328
Conversation
jekyll/_cci2/ip-ranges.adoc
Outdated
|
||
Use conditional logic to control when IP ranges is enabled with pipeline parameters. | ||
Use conditional logic to control when IP ranges is enabled with pipeline parameters. You can set the pipeline parameter `pipeline.parameters.ip_ranges` to `true` to enable IP ranges for the `build` job. FOr more information on triggering pipelines with pipeline parameters, see the xref:triggers-overview#[Trigger a pipeline] page. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
In that example, the parameter's name is ip_ranges
. The notation pipeline.parameters.ip_ranges
is how you refer to the parameter.
Typo: FOr
.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just checking - are you suggesting a change to the paragraph here? I've fixed the FOr typo but from your comment it seems like I need to do something else?
jekyll/_cci2/ip-ranges.adoc
Outdated
* link:https://ip-ranges.amazonaws.com/ip-ranges.json[AWS]: CircleCI uses the _us-east-1_ and _us-east-2_ regions. | ||
* link:https://www.gstatic.com/ipranges/cloud.json[GCP]: CircleCI uses the _us-east1_ and _us-central1_ regions. | ||
|
||
CircleCI _does not recommend_ configuring an IP-based firewall using the AWS or GCP IP addresses. The vast majority are not CircleCI's machines. CircleCI can _not guarantee_ that the addresses in the AWS or GCP endpoints persist from day-to-day, as these addresses are reassigned continuously. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Due to the security aspect, I think this paragraph could be a "CAUTION".
Description
Reasons
The flow of the page was confusing, combining information on the feature "IP Ranges" and also the IP adresses customers can expect core CircleCI services to use. This page is in our top 10 most read so we wanted to make changes to improve readability and structure.
Content Checklist
Please follow our style when contributing to CircleCI docs. Our style guide is here: https://circleci.com/docs/style/style-guide-overview.
Please take a moment to check through the following items when submitting your PR (this is just a guide so will not be relevant for all PRs) 😸: