Skip to content

feat(elt-pipelines): Port opralogweb to elt-pipelines - #462

Merged
ambolt314 merged 14 commits into
mainfrom
458a_port_opralogweb_to_elt
Sep 8, 2026
Merged

feat(elt-pipelines): Port opralogweb to elt-pipelines#462
ambolt314 merged 14 commits into
mainfrom
458a_port_opralogweb_to_elt

Conversation

@ambolt314

Copy link
Copy Markdown
Contributor

Summary

Move OPRALOGWEB staging tables from warehouses to elt-pipelines

Ref #458 .

@ambolt314
ambolt314 requested review from a team as code owners September 4, 2026 15:30
@coderabbitai

coderabbitai Bot commented Sep 4, 2026

Copy link
Copy Markdown
Contributor

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Advanced

Run ID: 61063c26-7bca-487a-8cf8-326647f240b9


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@ambolt314 ambolt314 changed the title feat(elt-pipelines): Port opralogweb to elt feat(elt-pipelines): Port opralogweb to elt-pipelines Sep 4, 2026

@martyngigg martyngigg Sep 8, 2026

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I should have spotted this the last time I looked but we need to keep the old model files in warehouses around too. The production pipelines are still using them so we can't delete them until we've fully switched over. For this and the other pull requests we'll just need copies rather than moves of the files.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks for putting these files back.

As these are the old pipelines their content needs to remain unchanged. They still use the old ingest approach that changes the column names to lower case. Essentially this pull request and the future ones should not contain any changes in the warehouses directory.

The simplest way to get everything in the warehouses directory back to what it looks like on main is to run these commands

git fetch
git checkout origin/main warehouses

from the root of the repository and then commit and push the results.

@ambolt314
ambolt314 requested a review from martyngigg September 8, 2026 11:08

@martyngigg martyngigg left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Looks good. The old pipelines still work as needed for production.

The full elt run facility_ops accelerator.opralogweb works locally when the environment variables are set.

@ambolt314
ambolt314 merged commit 2d3021f into main Sep 8, 2026
2 checks passed
@ambolt314
ambolt314 deleted the 458a_port_opralogweb_to_elt branch September 8, 2026 15:27
ambolt314 added a commit that referenced this pull request Sep 8, 2026
<!---
Please write the PR title following the conventions at
https://www.conventionalcommits.org/en/v1.0.0/
Example:
fix(elt-common): Fix bug in package
-->

### Summary
Move `OPRALOGWEB` staging tables from `warehouses` to `elt-pipelines`

<!--- Describe the change below, including rationale and design
decisions -->

Ref #458 . <!-- and fix #xxxx or close #xxxx xor resolves #xxxx. One
line per issue fixed. -->
<!-- alternative
 *There is no associated issue.*
-->
ambolt314 added a commit that referenced this pull request Sep 9, 2026
<!---
Please write the PR title following the conventions at
https://www.conventionalcommits.org/en/v1.0.0/
Example:
fix(elt-common): Fix bug in package
-->

Move `OPRALOGWEB` staging tables from `warehouses` to `elt-pipelines`

<!--- Describe the change below, including rationale and design
decisions -->

Ref #458 . <!-- and fix #xxxx or close #xxxx xor resolves #xxxx. One
line per issue fixed. -->
<!-- alternative
 *There is no associated issue.*
-->
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.

2 participants