Skip to content

Adjust existing pipeline scripts to support Jenkins#283

Merged
99Lys merged 9 commits intodremio:mainfrom
99Lys:DX-100314
Apr 1, 2025
Merged

Adjust existing pipeline scripts to support Jenkins#283
99Lys merged 9 commits intodremio:mainfrom
99Lys:DX-100314

Conversation

@99Lys
Copy link
Copy Markdown
Contributor

@99Lys 99Lys commented Mar 13, 2025

Summary

Adjust and reformat existing pipeline scripts to support Jenkins and have more legible and accurate testing results when running dbt against Dremio EE pipelines

Description

  • Created variables and functions to avoid code repetition
  • Handled error status
  • Included files (profiles.yml and projects.zip) for the dbt test projects creation, instead of using dbt init command

Test Results

All tests are passing, no new tests were added

Changelog

  • Added a summary of what this PR accomplishes to CHANGELOG.md

@99Lys 99Lys closed this Mar 18, 2025
@99Lys 99Lys reopened this Mar 18, 2025
@99Lys 99Lys requested review from bcmeireles, howareyouman and simonpannek and removed request for bcmeireles March 26, 2025 13:59
simonpannek
simonpannek previously approved these changes Mar 26, 2025
@howareyouman
Copy link
Copy Markdown

@99Lys Could you please modify the description part in this PR? Thank you.

@CLAassistant
Copy link
Copy Markdown

CLAassistant commented Mar 26, 2025

CLA assistant check
All committers have signed the CLA.

@99Lys 99Lys changed the title Dremio EE testing pipelines Adjust existing pipeline scripts to support Jenkins Mar 27, 2025
@99Lys 99Lys marked this pull request as ready for review March 27, 2025 12:40
@99Lys 99Lys requested a review from simonpannek March 27, 2025 12:45
Copy link
Copy Markdown

@howareyouman howareyouman left a comment

Choose a reason for hiding this comment

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

I have no blocker on merging this

@99Lys 99Lys merged commit 65ec7e9 into dremio:main Apr 1, 2025
19 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

5 participants