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

Ingest 2024 #76

Merged
merged 6 commits into from
May 21, 2024
Merged

Conversation

MichaelTiemannOSC
Copy link
Contributor

Restore rmi data sources to complete the verification that we can do an end-to-end ingestion with the new Notebooks. This is as far as we can test all the latest versions before we shift to the new data mesh pattern (and more modern versions of dbt and OpenMetadata).

… pipeline

cannot be run correctly until we are properly re-filling missing tables
from the rmi ingestion pipeline (that these changes hopefully provide).

Signed-off-by: Michael Tiemann <[email protected]>
…g `dbt` momentarily to 1.4.9,

we verify that `ITR-data-production.ipynb` works as expected using the recently updated
`osc-ingest-rmi_utility_transition_hub.ipynb` notebook.  We also make explicit the `dbt-trino`
dependency.  The next step is to true up the OS-C cluster environment to work with
the latest versions of `dbt` (1.8.0) and `OpenMetadata` (1.4.0) and make any adjustments
in our metadata generation to connect those dots.

Signed-off-by: Michael Tiemann <[email protected]>
Signed-off-by: Michael Tiemann <[email protected]>
@MichaelTiemannOSC MichaelTiemannOSC marked this pull request as ready for review May 21, 2024 11:27
@MichaelTiemannOSC
Copy link
Contributor Author

There remains some disconnect between how dbt trims newlines and how the pre-commit linting wants there to be newlines, but this is a good checkpoint for switching across to the new data mesh pattern, and dealing with the newline problem at a later date.

@MichaelTiemannOSC MichaelTiemannOSC merged commit 0ee71c1 into os-climate:main May 21, 2024
14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant