-
Notifications
You must be signed in to change notification settings - Fork 19
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
[Feature] Create new configuration abstraction for a Project #61
Comments
Isn't it |
Some thoughts here, @z3z1ma. The mechanism for respecting/enforcing the So, we have a common need for downstream projects to KNOW this configuration value for upstream projects. This means that we either load the upstream project's I recommend re-reading the proposal again and dig deep into how this solution might work! |
Thanks @nicholasyager! I totally get how what your saying will work. I'm saying that So TLDR Speculating the solution is simple enough The implications however are that we are exposing a (in my opinion a hard to miss) gap in the upstream codebase that surprises me was not thought through in this way. |
Cool. This is aligned with my thinking. I'll ponder some reasonable stepping stones that gets loom's configuration working in this direction in a backwards compatible manner. Regarding the complexities of managing plugins using the interfaces provided, I totally understand -- you're commenting on an issue thread for a dbt-core plugin 😉🤣 Let's be charitable to the devs working on dbt-core, and operate with an understanding that they are working under similar resource, timing, and commercial pressures as the people who work for dbt Labs' competitors who happen to leverage |
Agreed. Don't mean to come off the wrong way. I am as much a contributor to OSS as I am a critic to software. They go hand in hand sometimes and have nothing to do with any company as much as it does a collection of python files which comprise the software. I would definitely PR these opinions upstream if I was still using dbt at work lol |
Is your feature request related to a problem? Please describe.
Currently, dbt-loom does not have a good way to leverage a dbt project's
restrict-access
configuration. This config is important for limiting cross-project access toprotected
andprivate
nodes. Since this is an entirely new file to load, we do not currently have a defined path for loading this information.Describe the solution you'd like
I'd like to be able to define a
Project
, and with this project a location for itsdbt_project.yml
file and an associatedmanifest.json
file. This should support all of our existing artifact sources where possible.To configure this, we can introduce a new optional top-level concept of a Project.
Describe alternatives you've considered
restrict-access
configured in thedbt_loom.config.yml
file instead.restruct-access
at all.Additional context
The text was updated successfully, but these errors were encountered: