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

tuning ssh keepalive settings #33727

Merged
merged 8 commits into from
Jan 5, 2024
Merged

tuning ssh keepalive settings #33727

merged 8 commits into from
Jan 5, 2024

Conversation

gisripa
Copy link
Contributor

@gisripa gisripa commented Dec 21, 2023

What

Shooting in the dark to see if SSH keepalive settings alleviate the woes caused by ssh tunnel closing DB connections

How

Using Apache MINA's configurable properties to try both Session level heartbeats, client level keepalive, TCP keepalive.

Copy link

vercel bot commented Dec 21, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Ignored Deployment
Name Status Preview Comments Updated (UTC)
airbyte-docs ⬜️ Ignored (Inspect) Visit Preview Jan 5, 2024 6:34pm

@octavia-squidington-iii octavia-squidington-iii added the CDK Connector Development Kit label Dec 21, 2023
Copy link
Contributor

Warning

Soft code freeze is in effect until 2024-01-02. Please avoid merging to master. #freedom-and-responsibility

Signed-off-by: Gireesh Sreepathi <gisripa@gmail.com>
Signed-off-by: Gireesh Sreepathi <gisripa@gmail.com>
@octavia-squidington-iii octavia-squidington-iii removed area/documentation Improvements or additions to documentation area/connectors Connector related issues labels Jan 3, 2024
Signed-off-by: Gireesh Sreepathi <gisripa@gmail.com>
Signed-off-by: Gireesh Sreepathi <gisripa@gmail.com>
Signed-off-by: Gireesh Sreepathi <gisripa@gmail.com>
@gisripa gisripa removed request for a team January 4, 2024 19:01
Copy link
Contributor

@edgao edgao left a comment

Choose a reason for hiding this comment

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

:shipit:

(I have a CDK release waiting on connector CI in #33730, so, uh, don't actually ship it yet though)

Optional<JsonNode> connectionOptionsConfig = Jsons.getOptional(clone, CONNECTION_OPTIONS_KEY);
if (connectionOptionsConfig.isEmpty()) {
LOGGER.info("No SSH connection options found, using defaults");
if (clone instanceof ObjectNode) { // Defensive check, it will always be object node
Copy link
Contributor

Choose a reason for hiding this comment

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

nit: you can pattern-match the instanceof to avoid the explicit cast

if (clone instanceof ObjectNode objectClone) {
  ... = objectClone.putObject(...)

@gisripa
Copy link
Contributor Author

gisripa commented Jan 4, 2024

:shipit:

(I have a CDK release waiting on connector CI in #33730, so, uh, don't actually ship it yet though)

Looks like released already ? I have to test this anyway in one of our workspace before shipping it in CDK

@edgao
Copy link
Contributor

edgao commented Jan 4, 2024

feel free to do a prerelease here - but I need to merge my pr to master (and then you can merge master into this branch) before you can make another cdk release :(

also connector publishes might be down (https://airbytehq-team.slack.com/archives/C03VDJ4FMJB/p1704395954729139?thread_ts=1704385086.967279&cid=C03VDJ4FMJB) so I'm holding off on merging my pr until conn ops confirms it's ok 😭

Signed-off-by: Gireesh Sreepathi <gisripa@gmail.com>
@gisripa
Copy link
Contributor Author

gisripa commented Jan 5, 2024

/publish-java-cdk

🕑 https://github.com/airbytehq/airbyte/actions/runs/7425481585
✅ Successfully published Java CDK version=0.11.1!

@gisripa gisripa enabled auto-merge (squash) January 5, 2024 18:52
@gisripa gisripa merged commit 8524b32 into master Jan 5, 2024
16 checks passed
@gisripa gisripa deleted the gireesh/ssh-timeouts branch January 5, 2024 18:54
jatinyadav-cc pushed a commit to ollionorg/datapipes-airbyte that referenced this pull request Feb 26, 2024
Signed-off-by: Gireesh Sreepathi <gisripa@gmail.com>
jatinyadav-cc pushed a commit to ollionorg/datapipes-airbyte that referenced this pull request Feb 26, 2024
Signed-off-by: Gireesh Sreepathi <gisripa@gmail.com>
jatinyadav-cc pushed a commit to ollionorg/datapipes-airbyte that referenced this pull request Feb 26, 2024
Signed-off-by: Gireesh Sreepathi <gisripa@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CDK Connector Development Kit
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants