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

merge nov 20th - include custom cmd advanced ci #6346

Merged
merged 37 commits into from
Nov 20, 2024
Merged

Conversation

mirnawong1
Copy link
Contributor

@mirnawong1 mirnawong1 commented Oct 23, 2024

this pr adds info on how usrs can exclude large models from their ci jobs to optimize running time.

[x ] Needs PM review
[ ] Must be live in product first (Merge Nov 20th)

docs notion


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

@mirnawong1 mirnawong1 requested a review from a team as a code owner October 23, 2024 15:57
Copy link

vercel bot commented Oct 23, 2024

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

Name Status Preview Updated (UTC)
docs-getdbt-com 🛑 Canceled (Inspect) Nov 20, 2024 4:40pm

@github-actions github-actions bot added content Improvements or additions to content size: small This change will take 1 to 2 days to address Docs team Authored by the Docs team @dbt Labs labels Oct 23, 2024
@mirnawong1 mirnawong1 requested a review from reubenmc October 23, 2024 16:16

- To exclude the large `fct_orders` model from the comparison, use the `--select state:modified --exclude fct_orders` syntax.
- To exclude models based on tags, use `--select state modified --exclude tag:tagname_a tag:tagname_b`.
- To include models that were directly modified and also those one step downstream, use `--select state:modified+`.
Copy link
Contributor Author

Choose a reason for hiding this comment

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

i think it's modified+1 since it's 1 node downstream: https://docs.getdbt.com/reference/node-selection/graph-operators#the-n-plus-operator


- To exclude the large `fct_orders` model from the comparison, use the `--select state:modified --exclude fct_orders` syntax.
- To exclude models based on tags, use `--select state modified --exclude tag:tagname_a tag:tagname_b`.
- To include models that were directly modified and also those one step downstream, use `--select state:modified+1`.

Choose a reason for hiding this comment

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

@joellabes are there any other examples you would list here?


:::

**Job outcome** — During a job run, the commands are "chained" together and executed as run steps. If one of the run steps in the chain fails, then the subsequent steps aren't executed, and the job will fail.
#### Compare changes custom commands

Choose a reason for hiding this comment

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

I agree this would be a good place to add a screenshot. We should have one available from the product next week once eng has it built.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

will ask @mikaylacrawford for screenshot nxt week

website/docs/docs/deploy/ci-jobs.md Show resolved Hide resolved
@github-actions github-actions bot added the size: medium This change will take up to a week to address label Nov 5, 2024
@mirnawong1 mirnawong1 changed the title include custom cmd advanced ci merge nov 20th - include custom cmd advanced ci Nov 12, 2024
@github-actions github-actions bot added size: small This change will take 1 to 2 days to address and removed size: medium This change will take up to a week to address labels Nov 18, 2024
Copy link

@reubenmc reubenmc left a comment

Choose a reason for hiding this comment

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

LGTM!

website/docs/docs/dbt-versions/release-notes.md Outdated Show resolved Hide resolved
@github-actions github-actions bot added size: medium This change will take up to a week to address and removed size: small This change will take 1 to 2 days to address labels Nov 20, 2024
@mirnawong1 mirnawong1 enabled auto-merge November 20, 2024 16:36
@github-actions github-actions bot added size: small This change will take 1 to 2 days to address and removed size: medium This change will take up to a week to address labels Nov 20, 2024
@mirnawong1 mirnawong1 merged commit c56e67f into current Nov 20, 2024
9 checks passed
@mirnawong1 mirnawong1 deleted the custom-commands branch November 20, 2024 16:40
Copy link
Contributor

Check your created Crawler

Check your created index on your Algolia Application

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 Docs team Authored by the Docs team @dbt Labs size: small This change will take 1 to 2 days to address trigger-crawl
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants