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

Use new github action runner ami with 100 gb disk #9450

Merged
merged 4 commits into from
Jan 16, 2022
Merged

Use new github action runner ami with 100 gb disk #9450

merged 4 commits into from
Jan 16, 2022

Conversation

tuliren
Copy link
Contributor

@tuliren tuliren commented Jan 12, 2022 •
edited by davinchia
Loading

This change is Reviewable

@tuliren
Copy link
Contributor Author

tuliren commented Jan 12, 2022

/test connector=connectors/destination-bigquery-denormalized

@tuliren
Copy link
Contributor Author

tuliren commented Jan 12, 2022

/test connector=connectors/destination-bigquery

@tuliren tuliren temporarily deployed to more-secrets January 12, 2022 20:39 Inactive
@octavia-squidington-iii octavia-squidington-iii temporarily deployed to more-secrets January 12, 2022 20:39 Inactive
@octavia-squidington-iii octavia-squidington-iii temporarily deployed to more-secrets January 12, 2022 20:39 Inactive
@octavia-squidington-iii octavia-squidington-iii temporarily deployed to more-secrets January 12, 2022 21:23 Inactive
@octavia-squidington-iii octavia-squidington-iii temporarily deployed to more-secrets January 13, 2022 03:57 Inactive
@octavia-squidington-iii octavia-squidington-iii requested a deployment to more-secrets January 13, 2022 17:53 Abandoned
@octavia-squidington-iii octavia-squidington-iii temporarily deployed to more-secrets January 13, 2022 17:53 Inactive
@octavia-squidington-iii octavia-squidington-iii temporarily deployed to more-secrets January 13, 2022 17:53 Inactive
@tuliren tuliren temporarily deployed to more-secrets January 13, 2022 17:53 Inactive
@tuliren
Copy link
Contributor Author

tuliren commented Jan 13, 2022 •
edited by github-actions bot
Loading

/test connector=connectors/destination-bigquery-denormalized

🕑 connectors/destination-bigquery-denormalized https://github.com/airbytehq/airbyte/actions/runs/1694105865
❌ connectors/destination-bigquery-denormalized https://github.com/airbytehq/airbyte/actions/runs/1694105865
🐛

@tuliren
Copy link
Contributor Author

tuliren commented Jan 13, 2022 •
edited by github-actions bot
Loading

/test connector=connectors/destination-bigquery

🕑 connectors/destination-bigquery https://github.com/airbytehq/airbyte/actions/runs/1694106495
❌ connectors/destination-bigquery https://github.com/airbytehq/airbyte/actions/runs/1694106495
🐛

@tuliren tuliren temporarily deployed to more-secrets January 13, 2022 17:58 Inactive
@octavia-squidington-iii octavia-squidington-iii temporarily deployed to more-secrets January 13, 2022 17:58 Inactive
@octavia-squidington-iii octavia-squidington-iii temporarily deployed to more-secrets January 13, 2022 17:58 Inactive
@grubberr grubberr temporarily deployed to more-secrets January 15, 2022 11:30 Inactive
@grubberr grubberr temporarily deployed to more-secrets January 15, 2022 21:20 Inactive
@grubberr
Copy link
Contributor

@tuliren any plans to merge this ?
I see that it really helps for
"Airbyte CI" -> "Connectors Base: Build"
which constantly failed with not enough space error

@tuliren
Copy link
Contributor Author

tuliren commented Jan 15, 2022

Oh, I did not realize that the connector base build passed on this PR.

This PR has one issue. It looks like the new AMI breaks test command. So I still need to look into that.

Will merge later today.

@tuliren
Copy link
Contributor Author

tuliren commented Jan 16, 2022 •
edited by github-actions bot
Loading

/test connector=connectors/destination-bigquery-denormalized

🕑 connectors/destination-bigquery-denormalized https://github.com/airbytehq/airbyte/actions/runs/1703333236
❌ connectors/destination-bigquery-denormalized https://github.com/airbytehq/airbyte/actions/runs/1703333236
🐛

@tuliren
Copy link
Contributor Author

tuliren commented Jan 16, 2022 •
edited by github-actions bot
Loading

/test connector=connectors/destination-s3

🕑 connectors/destination-s3 https://github.com/airbytehq/airbyte/actions/runs/1703333301
❌ connectors/destination-s3 https://github.com/airbytehq/airbyte/actions/runs/1703333301
🐛

@octavia-squidington-iii octavia-squidington-iii temporarily deployed to more-secrets January 16, 2022 04:01 Inactive
@octavia-squidington-iii octavia-squidington-iii temporarily deployed to more-secrets January 16, 2022 04:01 Inactive
@tuliren
Copy link
Contributor Author

tuliren commented Jan 16, 2022 •
edited by github-actions bot
Loading

/test connector=connectors/destination-s3

🕑 connectors/destination-s3 https://github.com/airbytehq/airbyte/actions/runs/1703360101
✅ connectors/destination-s3 https://github.com/airbytehq/airbyte/actions/runs/1703360101
No Python unittests run

@tuliren
Copy link
Contributor Author

tuliren commented Jan 16, 2022 •
edited by github-actions bot
Loading

/test connector=connectors/destination-bigquery-denormalized

🕑 connectors/destination-bigquery-denormalized https://github.com/airbytehq/airbyte/actions/runs/1703360175
✅ connectors/destination-bigquery-denormalized https://github.com/airbytehq/airbyte/actions/runs/1703360175
Python tests coverage:

	 ---------- coverage: platform linux, python 3.8.10-final-0 -----------
	 Name                                                              Stmts   Miss  Cover
	 -------------------------------------------------------------------------------------
	 main_dev_transform_catalog.py                                         3      3     0%
	 main_dev_transform_config.py                                          3      3     0%
	 normalization/__init__.py                                             4      0   100%
	 normalization/destination_type.py                                    13      0   100%
	 normalization/transform_catalog/__init__.py                           2      0   100%
	 normalization/transform_catalog/catalog_processor.py                143     77    46%
	 normalization/transform_catalog/destination_name_transformer.py     155      8    95%
	 normalization/transform_catalog/reserved_keywords.py                 13      0   100%
	 normalization/transform_catalog/stream_processor.py                 520    333    36%
	 normalization/transform_catalog/table_name_registry.py              174     34    80%
	 normalization/transform_catalog/transform.py                         45     26    42%
	 normalization/transform_catalog/utils.py                             33      7    79%
	 normalization/transform_config/__init__.py                            2      0   100%
	 normalization/transform_config/transform.py                         148     34    77%
	 -------------------------------------------------------------------------------------
	 TOTAL                                                              1258    525    58%

@tuliren tuliren temporarily deployed to more-secrets January 16, 2022 04:14 Inactive
@octavia-squidington-iii octavia-squidington-iii temporarily deployed to more-secrets January 16, 2022 04:15 Inactive
@octavia-squidington-iii octavia-squidington-iii temporarily deployed to more-secrets January 16, 2022 04:15 Inactive
@tuliren tuliren merged commit 2bad2cf into master Jan 16, 2022
@tuliren tuliren deleted the liren/try-new-ami branch January 16, 2022 04:38
@octavia-squidington-iii octavia-squidington-iii mentioned this pull request Jan 20, 2022
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@tuliren @grubberr @octavia-squidington-iii