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

Connector versions are rolled back after Airbyte server is upgraded #5954

Closed
tuliren opened this issue Sep 9, 2021 · 3 comments · Fixed by #5964 or #6220
Closed

Connector versions are rolled back after Airbyte server is upgraded #5954

tuliren opened this issue Sep 9, 2021 · 3 comments · Fixed by #5964 or #6220
Assignees
Labels
type/bug Something isn't working

Comments

@tuliren
Copy link
Contributor

tuliren commented Sep 9, 2021

Enviroment

  • Airbyte version: example is 0.29.15-alpha, 0.29.17-alpha
  • OS Version / Instance:
  • Deployment:
  • Source Connector and version: Stripe 0.1.16, rolled back to 0.1.14
  • Severity: Medium
  • Step where error happened: Upgrade Airbyte

Current Behavior

After upgrading to Airbyte 0.29.15-alpha or 0.29.17-alpha, stripe (among with many other source and destination connectors), rolled back to 0.1.14.

Expected Behavior

Connector definition stays the same or get upgraded to the latest version when Airbyte is upgraded.

Logs

Slack thread: https://airbytehq.slack.com/archives/C019WJFR1JT/p1631118720129400

image

image (1)

LOG

Steps to Reproduce

Are you willing to submit a PR?

Yes.

@tuliren tuliren added the type/bug Something isn't working label Sep 9, 2021
@tuliren tuliren self-assigned this Sep 9, 2021
@tuliren
Copy link
Contributor Author

tuliren commented Sep 9, 2021

Need to add more logging during connector definition upgrade to debug this.

tuliren added a commit that referenced this issue Sep 10, 2021
@tuliren tuliren linked a pull request Sep 10, 2021 that will close this issue
@tuliren
Copy link
Contributor Author

tuliren commented Sep 10, 2021

Oh, currently the db config persistence checks if a connector definition is in use by querying StandardSync records. However, it should query SourceConnection and DestinationConnection. That's definitely a bug.

But even if it is not getting the used connectors correctly, no connector should be rolled back. So the root cause is still a mystery.

tuliren added a commit that referenced this issue Sep 10, 2021
* Add logging to debug #5954

* Fix query for used connector definition ids
@tuliren tuliren reopened this Sep 11, 2021
@tuliren
Copy link
Contributor Author

tuliren commented Sep 15, 2021

Also investigate if it is possible for a connector definition to get removed from the database. Slack thread.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type/bug Something isn't working
Projects
None yet
1 participant