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

10 8 22 #13

Merged
merged 2,294 commits into from
Aug 10, 2022
Merged

10 8 22 #13

merged 2,294 commits into from
Aug 10, 2022

Conversation

chaitanyapraveen
Copy link

Fetched and merged upstream master of Airbyte with the following conflicts resolved

  • Removed airbyte's document reference in the google ads
  • Klaviyo source.py deletion of spec function
  • Intercom added state checkout function & parse_response
  • Recurly reverted to Airbyte's original definition of begin_time & end_time

bazarnov and others added 30 commits July 22, 2022 22:58
* Validate only on incremental

* Add test

* Format and pmd

* Update test

* PR comments

* Update airbyte-protocol/protocol-models/src/test/java/io/airbyte/protocol/models/CatalogHelpersTest.java

Co-authored-by: Lake Mossman <lake@airbyte.io>

* Fix test

Co-authored-by: Lake Mossman <lake@airbyte.io>
The openjdk 17 image has multiple critical vulnerabilities. This is no longer updated frequently since 17 is now in LTS and not actively developed on.

The JDK 19 images do not have any critical or high or medium vulnerabilities. Further, JVM has bytecode backwards compatibility guarantees. Here we update all of Cloud to use openjdk 19 slim to fix these holes.

Leave gradle code compilation at 17 for now.

Co-authored-by: Davin Chia <davinchia@gmail.com>
…irbytehq#14714)

* improve error message

* also in JdbcSourceOperations

* bump versions + changelog

* auto-bump connector version [ci skip]

* auto-bump connector version [ci skip]

* auto-bump connector version [ci skip]

* auto-bump connector version [ci skip]

* auto-bump connector version [ci skip]

* auto-bump connector version [ci skip]

* auto-bump connector version [ci skip]

Co-authored-by: Octavia Squidington III <octavia-squidington-iii@users.noreply.github.com>
…uthentication (airbytehq#14792)

* Normalization for Snowflake destination: added support for key pair authentication

* Normalization for Snowflake destination: updated changelogs

* Normalization for Snowflake destination: renamed property passphrase to password

* Normalization for Snowflake destination: bump normalization version in NormalizationRunnerFactory

* Normalization for Snowflake destination: added unit tests and change file creating process

* Normalization for Snowflake destination: added unit tests and change file creating process

* Normalization for Snowflake destination: added unit tests and change file creating process
…4976)

Co-authored-by: davinchia <davinchia@users.noreply.github.com>
…irbytehq#14828)

airbyte-5577: Respect date/datetime types for snowflake source.
…14996)

* Source TiDb Removed additionalProperties: false from spec

* updated changelog

* bump version

* auto-bump connector version [ci skip]

Co-authored-by: Octavia Squidington III <octavia-squidington-iii@users.noreply.github.com>
Follow up to airbytehq#14971 . Make sure to update this for the OSS publishing process as well.

Also update all the dockerfiles.

Connectors are not touched.
* Add callout tag for on-going known issue

Airbyte is currently working on resolving this issue with the Amazon Seller Partner connector. The issue is being tracked here: airbytehq#14734

* Update callout type

* Update to use caution

* Edited wording -- remove the word "Note"
* Fixed some minor typos and formatting

* Update based on comments
* Update with changes

- Update from Slack to Discourse forum
- Remove mentions of Singer including broken examples

* Update based on comments
* fix: fix airbyte-worker

* bump temporalio version to 1.13.0

* fix: revert removal of deploymentMode variable
…5005)

Co-authored-by: davinchia <davinchia@users.noreply.github.com>
* don't set workdir to /data

* remove debug log
* Add docstrings for auth package

* docstrings for the check package

* docstrings for the datetime package

* docstrings for the decoder package

* docstrings for extractors package and fix tests

* interpolation docstrings

* ref ->  and parser docstrings

* docstrings for parsers package

* error handler docstrings

* requester docstrings

* more docstrings

* docstrings

* docstrings

* docstrings

* Use defined type annotations

* update

* update docstrings

* Update docstrings

* update docstrings

* update docstrings

* update template

* Revert "update template"

This reverts commit eb4a118.

* update template

* update

* move to interpolated_string

* update docstring

* update

* fix tests

* format

* return type can also be an array

* Update airbyte-cdk/python/airbyte_cdk/sources/declarative/interpolation/interpolated_boolean.py

Co-authored-by: Sherif A. Nada <snadalive@gmail.com>

* Update airbyte-cdk/python/airbyte_cdk/sources/declarative/interpolation/interpolation.py

Co-authored-by: Sherif A. Nada <snadalive@gmail.com>

* Update airbyte-cdk/python/airbyte_cdk/sources/declarative/interpolation/jinja.py

Co-authored-by: Sherif A. Nada <snadalive@gmail.com>

* Update airbyte-cdk/python/airbyte_cdk/sources/declarative/interpolation/interpolated_boolean.py

Co-authored-by: Sherif A. Nada <snadalive@gmail.com>

* Update airbyte-cdk/python/airbyte_cdk/sources/declarative/requesters/error_handlers/backoff_strategy.py

* Update as per comments

Co-authored-by: Sherif A. Nada <snadalive@gmail.com>
Follow up to airbytehq#15023. Use the latest to get rid of all the security vulnerabilities.
Curl was the last remaining security vulnerability in the image. Instead of using curl, use wget to avoid this issue.

This also has the side effect of decreasing the image size by 150 MB.
…irbytehq#14914)

* interpolatedauth

* fix tests

* fix import

* no need for default

* Bump version

* Missing docstrings

* example

* missing example

* more docstrings

* interpolated types
* 13539 Fix integration tests source-clickhouse Mac OS

* 13539 Updated clickhouse jdbc driver

* 13539 Updated destination-clickhouse-strict-encrypt

* 13539 Updated SSL configuration and tests for clickhouse-destination

* 13539 Updated SSL for source-clickhouse-strict-encrypt

* 13539 Resolved host by ip

* 13539 Fixed code formatting

* 13539 Bump up source-clickhouse-strict-encrypt version

* auto-bump connector version [ci skip]

* auto-bump connector version [ci skip]

Co-authored-by: Octavia Squidington III <octavia-squidington-iii@users.noreply.github.com>
davinchia and others added 24 commits August 8, 2022 17:04
It's difficult today to parse all the logs from tests. Engineers waste time scrolling through the log outputs and looking for the relevant stack trace.

This PR adds an action to generate a JUnit report so devs can understand test results at a glance. This generates 3 reports for each of the main build jobs when the build completes. We leave the frontend build out since this is aggregated by cypress.

See https://github.com/airbytehq/airbyte/pull/15271/checks?check_run_id=7683783016 for an example of how this works.

Use the https://github.com/dorny/test-reporter action and configure this to look at the Jacoco test report output for top level and second level builds. Note that most of the parameters into the action don't work.

Comment out the trap commands to output logs. Though this can be useful for debugging, there is little practical use in the day-to-day, and results in extremely noisy logs.
* fix parse and format methods

* define constant

* remove timestamp magic keyword

* comment

* test for ci

* uncomment test

* use timestamp()

* Bump cdk version

* bump to 0.1.72
…q#15424)

* Fix template

* Bump cdk version

* move url_base to requester

* Revert "move url_base to requester"

This reverts commit 87592a7.

* Bump to 0.1.72
* #151-alphabeta source Typeform: upd Forms schema

* #151 source Typeform: upd changelog

* auto-bump connector version [ci skip]

Co-authored-by: Octavia Squidington III <octavia-squidington-iii@users.noreply.github.com>
* 14899 Update tests instance type to ATLAS
* sqlite destination skeleton

* Add doc

* Add destination to  general docs

* clean unused files

* add seed and icon

* sqlite destination skeleton

* Add doc

* Add destination to  general docs

* add seed and icon

* correct config file

* remove unused import

* run seed spec file

Co-authored-by: marcosmarxm <marcosmarxm@gmail.com>
Co-authored-by: Marcos Marx <marcosmarxm@users.noreply.github.com>
* Fixes button and icons on billing page. Updates credits icon in sidebar.

* Requested changes.

Co-authored-by: Tim Roes <tim@airbyte.io>
* Adds PAT check to shared pr check

* Name change

* Removes "safe_to_push" string

* Adds OCTAVIA_PAT and uses the found PAT

found PAT was not used in all locales, so this could have still failed
on an expired OCTAVIA_PAT before this change
…EADME files (airbytehq#15135)

* solve readme conflict

* updated jira sources with open PR details

* correct additionalProperties test discover

Co-authored-by: marcosmarxm <marcosmarxm@gmail.com>
* Fix copy link to logs functionality

* Update airbyte-webapp/src/components/JobItem/JobItem.tsx

Co-authored-by: Edmundo Ruiz Ghanem <168664+edmundito@users.noreply.github.com>

* Fix scrolling

* Remove smooth scrolling

* Improve effect for better return statements

* Better scroll

Co-authored-by: Edmundo Ruiz Ghanem <168664+edmundito@users.noreply.github.com>
* Implement ErrorProne PMD rules:
AssignmentInOperand
AvoidAccessibilityAlteration
AvoidBranchingStatementAsLastInLoop
AvoidCatchingNPE
AvoidCatchingThrowable
AvoidDuplicateLiterals rule
* Add number_of_streams to job sync tracking
…hq#15314)

* snowflake discovers ints as ints

* version bump+changelog

* bump version+changelog

* auto-bump connector version [ci skip]

Co-authored-by: Octavia Squidington III <octavia-squidington-iii@users.noreply.github.com>
)

Co-authored-by: benmoriceau <benmoriceau@users.noreply.github.com>
* add test case for new field(s) appearing in data

* rework test to verify that sync at least not failed if new fields are present
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment