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

Git providers supporting CI #6692

Open
wants to merge 26 commits into
base: current
Choose a base branch
from
Open

Git providers supporting CI #6692

wants to merge 26 commits into from

Conversation

nataliefiann
Copy link
Contributor

@nataliefiann nataliefiann commented Dec 19, 2024

What are you changing in this pull request and why?

I've created this PR with reference to this git issue: #3704 to help users clarify which Git providers support CI.

In the PR, I have updated the Continuous integration page, created a card on the Git configuaration page and linked the table I created in the CI doc to it. I also created and used a snippet from https://docs.getdbt.com/docs/deploy/ci-jobs#trigger-a-ci-job-with-the-api

Update - I've also added the snippet to the Import a project by git URL and Continuous integration jobs in dbt Cloud pages.

Closes: #3704

Checklist

  • I have reviewed the Content style guide so my content adheres to these guidelines.
  • The topic I'm writing about is for specific dbt version(s) and I have versioned it according to the version a whole page and/or version a block of content guidelines.
  • I have added checklist item(s) to this list for anything anything that needs to happen before this PR is merged, such as "needs technical review" or "change base branch."
  • The content in this PR requires a dbt release note, so I added one to the release notes page.

🚀 Deployment available! Here are the direct links to the updated files:

@nataliefiann nataliefiann requested a review from a team as a code owner December 19, 2024 14:17
Copy link

vercel bot commented Dec 19, 2024

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

Name Status Preview Updated (UTC)
docs-getdbt-com ✅ Ready (Inspect) Visit Preview Dec 23, 2024 0:05am

@github-actions github-actions bot added content Improvements or additions to content size: small This change will take 1 to 2 days to address labels Dec 19, 2024
@@ -0,0 +1,2 @@
If you're not using dbt Cloud’s native Git integration with [GitHub](/docs/cloud/git/connect-github), [GitLab](/docs/cloud/git/connect-gitlab), or [Azure DevOps](/docs/cloud/git/connect-azure-devops), you can use the [Administrative API](/docs/dbt-cloud-apis/admin-cloud-api) to trigger a CI job to run. However, dbt Cloud will not automatically delete the temporary schema for you. This is because automatic deletion relies on incoming webhooks from Git providers, which is only available through the native integrations.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this is a silly question but why are you turning this into a snippet? and why isn't it be used if turning into a snippet?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

it seems that that content is part of this section: https://docs.getdbt.com/docs/deploy/ci-jobs#trigger-a-ci-job-with-the-api

@@ -0,0 +1,8 @@
If your git provider has a native dbt Cloud integration, you can set up [continuous integration (CI) job](/docs/deploy/ci-jobs) within dbt Cloud. For providers without native integration, you can still use a Git URL. The following table explains the available integration options and their capabilities.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

also for this snippet, to address Joel's issue and my suggestion -- should this snippet table also be used in the git configuration page (e.g. maybe the landing page or git url page)?

t'd be useful to just have that listed in the CI docs as well as the configure git docs

Copy link
Contributor

@mirnawong1 mirnawong1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks much better, thanks for linking the snippet in other pages @nataliefiann ! approved and one question that can be followed up with another pr - should the table include api functionality? eg how you can trigger a job via the cloud api: https://docs.getdbt.com/docs/deploy/ci-jobs#trigger-a-ci-job-with-the-api?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
content Improvements or additions to content size: small This change will take 1 to 2 days to address
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Clarify what git providers support CI
2 participants