Skip to content

Conversation

@jakedoublev
Copy link
Contributor

@jakedoublev jakedoublev commented Jan 31, 2024

  1. Adds full CLI/handlers CRUD for namespaces
  2. updates go.mod to use local modules for opentdf-v2-poc for ease of development until schema rework is merged with main in github
  3. comments out attributes work so far until schema rework is brought in so the build/run does not fail

Addresses #27

@alkalescent
Copy link
Contributor

alkalescent commented Jan 31, 2024

Getting an error when installing or running:

pkg/handlers/namespaces.go:4:2: no required module provides package github.com/opentdf/opentdf-v2-poc/sdk/namespaces; to add it:
	go get github.com/opentdf/opentdf-v2-poc/sdk/namespaces

@jakedoublev
Copy link
Contributor Author

jakedoublev commented Jan 31, 2024

Getting any error when installing or running:

pkg/handlers/namespaces.go:4:2: no required module provides package github.com/opentdf/opentdf-v2-poc/sdk/namespaces; to add it:
	go get github.com/opentdf/opentdf-v2-poc/sdk/namespaces

@suchak1
Yes, unfortunately that is expected because of how Go resolves modules while the namespaces work lives on a branch other than main. This workaround allows you to have the opentdf-v2-poc repo in the same level directory on your machine, and if you run git fetch, then switch to branch policy-config-changes or another branch off of that secondary trunk containing the schema rework, you will be able to run go get github.com/opentdf/opentdf-v2-poc/sdk/namespaces and have it be found.

This is unfortunately the downstream impact of consuming services on a specific branch in another repo while they're unmerged.

It's the equivalent of having your package.json version for a dependency pointing to a local filepath.

@alkalescent
Copy link
Contributor

Gotcha, I was able to run the CLI while having that branch checked out

@jakedoublev jakedoublev merged commit d96ab22 into main Feb 1, 2024
@jakedoublev jakedoublev deleted the policy-config-changes/namespaces branch February 1, 2024 21:58
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.

3 participants