Skip to content

CDK Connectors Tests: Fix NotGitRepo error #43394

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

Merged
merged 1 commit into from
Aug 12, 2024
Merged

Conversation

clnoll
Copy link
Contributor

@clnoll clnoll commented Aug 8, 2024

CDK Connectors Tests were getting a 404, NotGitRepo error when setting up the regression tests container. This was caused by the use of a token not scoped for cloning the connection-retriever. This PR updates the token.

We previously had a backup token that we could use if the primary token was rate-limited, but no longer have one with the right permissions, so we'll want to create it.

Copy link

vercel bot commented Aug 8, 2024

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

1 Skipped Deployment
Name Status Preview Comments Updated (UTC)
airbyte-docs ⬜️ Ignored (Inspect) Visit Preview Aug 10, 2024 6:10pm

@clnoll clnoll force-pushed the catherine/fix-notagitrepo branch from e8da78e to 54f6df9 Compare August 10, 2024 18:05
@octavia-squidington-iii octavia-squidington-iii removed area/connectors Connector related issues CDK Connector Development Kit labels Aug 10, 2024
@clnoll clnoll marked this pull request as ready for review August 10, 2024 18:08
@clnoll clnoll requested review from a team and alafanechere August 10, 2024 18:08
@clnoll clnoll changed the title test CDK Connectors Tests: Fix NotGitRepo error Aug 12, 2024
@clnoll clnoll merged commit 49e0277 into master Aug 12, 2024
34 checks passed
@clnoll clnoll deleted the catherine/fix-notagitrepo branch August 12, 2024 16:36
LouisAuneau pushed a commit to LouisAuneau/airbyte that referenced this pull request Aug 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants