|
1 |
| -# Flexport Source |
| 1 | +# Flexport source connector |
2 | 2 |
|
3 |
| -This is the repository for the Flexport configuration based source connector. |
| 3 | + |
| 4 | +This is the repository for the Flexport source connector, written in Python. |
4 | 5 | For information about how to use this connector within Airbyte, see [the documentation](https://docs.airbyte.com/integrations/sources/flexport).
|
5 | 6 |
|
6 | 7 | ## Local development
|
7 | 8 |
|
8 |
| -#### Create credentials |
| 9 | +### Prerequisites |
| 10 | +* Python (~=3.9) |
| 11 | +* Poetry (~=1.7) - installation instructions [here](https://python-poetry.org/docs/#installation) |
| 12 | + |
| 13 | + |
| 14 | +### Installing the connector |
| 15 | +From this connector directory, run: |
| 16 | +```bash |
| 17 | +poetry install --with dev |
| 18 | +``` |
| 19 | + |
9 | 20 |
|
| 21 | +### Create credentials |
10 | 22 | **If you are a community contributor**, follow the instructions in the [documentation](https://docs.airbyte.com/integrations/sources/flexport)
|
11 | 23 | to generate the necessary credentials. Then create a file `secrets/config.json` conforming to the `source_flexport/spec.yaml` file.
|
12 | 24 | Note that any directory named `secrets` is gitignored across the entire Airbyte repo, so there is no danger of accidentally checking in sensitive information.
|
13 |
| -See `integration_tests/sample_config.json` for a sample config file. |
| 25 | +See `sample_files/sample_config.json` for a sample config file. |
14 | 26 |
|
15 |
| -**If you are an Airbyte core member**, copy the credentials in Lastpass under the secret name `source flexport test creds` |
16 |
| -and place them into `secrets/config.json`. |
17 | 27 |
|
18 |
| -### Locally running the connector docker image |
19 |
| - |
20 |
| -#### Build |
| 28 | +### Locally running the connector |
| 29 | +``` |
| 30 | +poetry run source-flexport spec |
| 31 | +poetry run source-flexport check --config secrets/config.json |
| 32 | +poetry run source-flexport discover --config secrets/config.json |
| 33 | +poetry run source-flexport read --config secrets/config.json --catalog sample_files/configured_catalog.json |
| 34 | +``` |
21 | 35 |
|
22 |
| -**Via [`airbyte-ci`](https://github.com/airbytehq/airbyte/blob/master/airbyte-ci/connectors/pipelines/README.md) (recommended):** |
| 36 | +### Running unit tests |
| 37 | +To run unit tests locally, from the connector directory run: |
| 38 | +``` |
| 39 | +poetry run pytest unit_tests |
| 40 | +``` |
23 | 41 |
|
| 42 | +### Building the docker image |
| 43 | +1. Install [`airbyte-ci`](https://github.com/airbytehq/airbyte/blob/master/airbyte-ci/connectors/pipelines/README.md) |
| 44 | +2. Run the following command to build the docker image: |
24 | 45 | ```bash
|
25 | 46 | airbyte-ci connectors --name=source-flexport build
|
26 | 47 | ```
|
27 | 48 |
|
28 |
| -An image will be built with the tag `airbyte/source-flexport:dev`. |
29 |
| - |
30 |
| -**Via `docker build`:** |
| 49 | +An image will be available on your host with the tag `airbyte/source-flexport:dev`. |
31 | 50 |
|
32 |
| -```bash |
33 |
| -docker build -t airbyte/source-flexport:dev . |
34 |
| -``` |
35 |
| - |
36 |
| -#### Run |
37 | 51 |
|
| 52 | +### Running as a docker container |
38 | 53 | Then run any of the connector commands as follows:
|
39 |
| - |
40 | 54 | ```
|
41 | 55 | docker run --rm airbyte/source-flexport:dev spec
|
42 | 56 | docker run --rm -v $(pwd)/secrets:/secrets airbyte/source-flexport:dev check --config /secrets/config.json
|
43 | 57 | docker run --rm -v $(pwd)/secrets:/secrets airbyte/source-flexport:dev discover --config /secrets/config.json
|
44 | 58 | docker run --rm -v $(pwd)/secrets:/secrets -v $(pwd)/integration_tests:/integration_tests airbyte/source-flexport:dev read --config /secrets/config.json --catalog /integration_tests/configured_catalog.json
|
45 | 59 | ```
|
46 | 60 |
|
47 |
| -## Testing |
48 |
| - |
| 61 | +### Running our CI test suite |
49 | 62 | You can run our full test suite locally using [`airbyte-ci`](https://github.com/airbytehq/airbyte/blob/master/airbyte-ci/connectors/pipelines/README.md):
|
50 |
| - |
51 | 63 | ```bash
|
52 | 64 | airbyte-ci connectors --name=source-flexport test
|
53 | 65 | ```
|
54 | 66 |
|
55 | 67 | ### Customizing acceptance Tests
|
56 |
| - |
57 |
| -Customize `acceptance-test-config.yml` file to configure tests. See [Connector Acceptance Tests](https://docs.airbyte.com/connector-development/testing-connectors/connector-acceptance-tests-reference) for more information. |
| 68 | +Customize `acceptance-test-config.yml` file to configure acceptance tests. See [Connector Acceptance Tests](https://docs.airbyte.com/connector-development/testing-connectors/connector-acceptance-tests-reference) for more information. |
58 | 69 | If your connector requires to create or destroy resources for use during acceptance tests create fixtures for it and place them inside integration_tests/acceptance.py.
|
59 | 70 |
|
60 |
| -## Dependency Management |
61 |
| - |
62 |
| -All of your dependencies should go in `setup.py`, NOT `requirements.txt`. The requirements file is only used to connect internal Airbyte dependencies in the monorepo for local development. |
63 |
| -We split dependencies between two groups, dependencies that are: |
64 |
| - |
65 |
| -- required for your connector to work need to go to `MAIN_REQUIREMENTS` list. |
66 |
| -- required for the testing need to go to `TEST_REQUIREMENTS` list |
| 71 | +### Dependency Management |
| 72 | +All of your dependencies should be managed via Poetry. |
| 73 | +To add a new dependency, run: |
| 74 | +```bash |
| 75 | +poetry add <package-name> |
| 76 | +``` |
67 | 77 |
|
68 |
| -### Publishing a new version of the connector |
| 78 | +Please commit the changes to `pyproject.toml` and `poetry.lock` files. |
69 | 79 |
|
| 80 | +## Publishing a new version of the connector |
70 | 81 | You've checked out the repo, implemented a million dollar feature, and you're ready to share your changes with the world. Now what?
|
71 |
| - |
72 | 82 | 1. Make sure your changes are passing our test suite: `airbyte-ci connectors --name=source-flexport test`
|
73 |
| -2. Bump the connector version in `metadata.yaml`: increment the `dockerImageTag` value. Please follow [semantic versioning for connectors](https://docs.airbyte.com/contributing-to-airbyte/resources/pull-requests-handbook/#semantic-versioning-for-connectors). |
| 83 | +2. Bump the connector version (please follow [semantic versioning for connectors](https://docs.airbyte.com/contributing-to-airbyte/resources/pull-requests-handbook/#semantic-versioning-for-connectors)): |
| 84 | + - bump the `dockerImageTag` value in in `metadata.yaml` |
| 85 | + - bump the `version` value in `pyproject.toml` |
74 | 86 | 3. Make sure the `metadata.yaml` content is up to date.
|
75 |
| -4. Make the connector documentation and its changelog is up to date (`docs/integrations/sources/flexport.md`). |
| 87 | +4. Make sure the connector documentation and its changelog is up to date (`docs/integrations/sources/flexport.md`). |
76 | 88 | 5. Create a Pull Request: use [our PR naming conventions](https://docs.airbyte.com/contributing-to-airbyte/resources/pull-requests-handbook/#pull-request-title-convention).
|
77 | 89 | 6. Pat yourself on the back for being an awesome contributor.
|
78 | 90 | 7. Someone from Airbyte will take a look at your PR and iterate with you to merge it into master.
|
| 91 | +8. Once your PR is merged, the new version of the connector will be automatically published to Docker Hub and our connector registry. |
0 commit comments