Skip to content
This repository has been archived by the owner on Dec 15, 2022. It is now read-only.

cannot build the Types: got type "map[string]string" for field "Data", only types "string" and "*string" supported as sensitive #208

Closed
ytsarev opened this issue Jan 24, 2022 · 2 comments · Fixed by #218
Assignees
Labels
bug Something isn't working

Comments

@ytsarev
Copy link
Member

ytsarev commented Jan 24, 2022

What happened?

The attempt to generate rancher2_secret resource in the context of https://github.com/AbsaOSS/provider-jet-rancher failed with

make generate
16:39:29 [ .. ] verify dependencies have expected content
all modules verified
16:39:34 [ OK ] go modules dependencies verified
16:39:34 [ .. ] go generate darwin_amd64
panic: cannot generate crd for resource rancher2_secret: cannot build types for Secret: cannot build the Types: got type "map[string]string" for field "Data", only types "string" and "*string" supported as sensitive

goroutine 1 [running]:
github.com/crossplane/terrajet/pkg/pipeline.Run(0xc000160600, 0xc000832570, 0x28)
	/Users/xxx/absa/provider-jet-rancher/.work/pkg/pkg/mod/github.com/turkenh/[email protected]/pkg/pipeline/run.go:76 +0x1db3
main.main()
	/Users/xxx/absa/provider-jet-rancher/cmd/generator/main.go:48 +0xad
exit status 2
apis/generate.go:32: running "go": exit status 1

How can we reproduce it?

Clone https://github.com/AbsaOSS/provider-jet-rancher, remove limiting WithIncludeList from config/provider.go and run make generate

@ytsarev ytsarev added the bug Something isn't working label Jan 24, 2022
@ulucinar
Copy link
Collaborator

ulucinar commented Jan 24, 2022

Hi @ytsarev,
@sergenyalcin is working on the related issue #100. Please see this comment on that issue.

@sergenyalcin, are you planning to add support for map types in your open PR?

ytsarev added a commit to AbsaOSS/provider-jet-rancher that referenced this issue Jan 24, 2022
`rancher2_secret` is skipped because of
crossplane/terrajet#208

Signed-off-by: Yury Tsarev <[email protected]>
@sergenyalcin
Copy link
Member

Hi @ytsarev, @sergenyalcin is working on the related issue #100. Please see this comment on that issue.

@sergenyalcin, are you planning to add support for map types in your open PR?

Hi @ulucinar yes, I am working to add support for map types in my PR.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants