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.

Sign up for GitHub

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

Jump to bottom

Edit multi-cloud docs #18972

Merged
merged 3 commits into from
Nov 5, 2022
Merged

Edit multi-cloud docs #18972

merged 3 commits into from
Nov 5, 2022

Conversation

sophia-wiley
Copy link
Contributor

Made some edits to multi-cloud docs.

@github-actions github-actions bot added the area/documentation Improvements or additions to documentation label Nov 4, 2022
sophia-wiley
sophia-wiley commented Nov 4, 2022
View reviewed changes
docs/cloud/managing-airbyte-cloud.md Outdated
Your data is processed in the chosen residency, but some data is still stored in the US.

:::
Your data is processed on a data plane in the chosen residency, but some data, like data associated with the sync mode, cursor, and primary key, is still stored in the US on our control plane. For streams with data that must stay in the chosen data residency, do not configure sync modes that use cursor or primary key.
Copy link
Contributor Author

Choose a reason for hiding this comment

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

@davinchia @andyjih Is this accurate? Do we want to include or remove any info?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@malikdiarra

davinchia
davinchia reviewed Nov 5, 2022
View reviewed changes
docs/cloud/getting-started-with-airbyte-cloud.md Outdated
@@ -183,8 +183,7 @@ Verify the sync by checking the logs:
3. Check the data at your destination. If you added a Destination Stream Prefix while setting up the connection, make sure to search for the stream name with the prefix.

## Allowlist IP address

Depending on your data residency location, you may need to allowlist the following IP addresses to enable access to Airbyte:
Depending on your [data residency](https://docs.airbyte.com/cloud/managing-airbyte-cloud#choose-your-default-data-residency) location, you may need to allowlist the following IP addresses to enable access to Airbyte:

### United States and Airbyte Default
#### GCP region: us-west2
Copy link
Contributor

@davinchia davinchia Nov 5, 2022 •
edited
Loading

Choose a reason for hiding this comment

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

one correction: GCP should be us-west3

davinchia
davinchia reviewed Nov 5, 2022
View reviewed changes
docs/cloud/getting-started-with-airbyte-cloud.md
@@ -193,10 +192,12 @@ Depending on your data residency location, you may need to allowlist the followi
* 34.106.60.246

### European Union
#### AWS region: eu-west-3
Copy link
Contributor

Choose a reason for hiding this comment

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

eu-west-3 is correct for aws

davinchia
davinchia reviewed Nov 5, 2022
View reviewed changes
docs/cloud/managing-airbyte-cloud.md Outdated
Your data is processed in the chosen residency, but some data is still stored in the US.

:::
Your data is processed on a data plane in the chosen residency, but some data, like data associated with the sync mode, cursor, and primary key, is still stored in the US on our control plane. For streams with data that must stay in the chosen data residency, do not configure sync modes that use cursor or primary key.
Copy link
Contributor

@davinchia davinchia Nov 5, 2022 •
edited
Loading

Choose a reason for hiding this comment

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

Suggested change
Your data is processed on a data plane in the chosen residency, but some data, like data associated with the sync mode, cursor, and primary key, is still stored in the US on our control plane. For streams with data that must stay in the chosen data residency, do not configure sync modes that use cursor or primary key.
Although your data is processed on a data plane in the chosen residency, configuration data, e.g. data associated with the sync mode, cursor, and primary key, is still stored on our control plane in the US. Because of this, data that must stay in the chosen data residency should not be used as a stream's cursor or primary key.

Copy link
Contributor

Choose a reason for hiding this comment

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

Some reword to clarify that the streams can use cursors/primary keys as long as the sensitive data is neither.

@sophia-wiley sophia-wiley merged commit e298049 into master Nov 5, 2022
@sophia-wiley sophia-wiley deleted the edit_multicloud_documentation branch November 5, 2022 01:33
letiescanciano added a commit that referenced this pull request Nov 7, 2022
@letiescanciano
Merge branch 'master' into leti/experiment-add-source-in-signup
c3edfb7 
* master: (69 commits)
  🪟 🐛 Fix wrong geography dropdown type #19021
  SAT: basic read on full catalog when `test_strictness_level == high` (#18937)
  Unhide DynamoDB destination (#18994)
  Fixed tests for destination connectors (#19007)
  🐛 Source Facebook Marketing: handle FacebookBadObjectError (#18971)
  Edit multi-cloud docs (#18972)
  🪟 🎉 Load credits consumption separate (#18986)
  Bmoric/extract source api (#18944)
  Migrating InvalidCursorException -> ConfigErrorException  (#18995)
  🪟 🎨 Fix banner link color (#18978)
  Handling configuration exceptions in IntegrationRunner (#18989)
  Add new workspace api endpoint (#18983)
  Add normalization to destination definition and actor definition table (#18300)
  Fix oauth controller (#18981)
  Fix migration dev center schema dump by run db-specific initialization script (#18984)
  fix master build failure (#18982)
  cleanup: delete debezium 1-4-2 module (#18733)
  Remove unused job persistence methods. (#18952)
  Hash filenames of extracted CSS (#18976)
  Fix typo in source code comment DataDaog ==> Datadog (#18911)
  ...
letiescanciano added a commit that referenced this pull request Nov 7, 2022
@letiescanciano
Merge branch 'master' into leti/move-oauth-to-the-top
6c7a97f 
* master: (73 commits)
  🪟 🐛 Fix wrong geography dropdown type #19021
  SAT: basic read on full catalog when `test_strictness_level == high` (#18937)
  Unhide DynamoDB destination (#18994)
  Fixed tests for destination connectors (#19007)
  🐛 Source Facebook Marketing: handle FacebookBadObjectError (#18971)
  Edit multi-cloud docs (#18972)
  🪟 🎉 Load credits consumption separate (#18986)
  Bmoric/extract source api (#18944)
  Migrating InvalidCursorException -> ConfigErrorException  (#18995)
  🪟 🎨 Fix banner link color (#18978)
  Handling configuration exceptions in IntegrationRunner (#18989)
  Add new workspace api endpoint (#18983)
  Add normalization to destination definition and actor definition table (#18300)
  Fix oauth controller (#18981)
  Fix migration dev center schema dump by run db-specific initialization script (#18984)
  fix master build failure (#18982)
  cleanup: delete debezium 1-4-2 module (#18733)
  Remove unused job persistence methods. (#18952)
  Hash filenames of extracted CSS (#18976)
  Fix typo in source code comment DataDaog ==> Datadog (#18911)
  ...
@sophia-wiley sophia-wiley mentioned this pull request Nov 10, 2022
Closed
@github-actions github-actions bot mentioned this pull request Nov 15, 2022
Closed
@octavia-squidington-iii octavia-squidington-iii mentioned this pull request Nov 18, 2022
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@davinchia davinchia davinchia approved these changes

@andyjih andyjih Awaiting requested review from andyjih

@josephkmh josephkmh Awaiting requested review from josephkmh

Assignees
No one assigned
Labels
area/documentation Improvements or additions to documentation
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@sophia-wiley @davinchia