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

Allow inputting b64 encoded data with kubernetes_config_map_v1_data via binary_data #2467

Open
fullykubed opened this issue Apr 13, 2024 · 1 comment · May be fixed by #2616
Open

Allow inputting b64 encoded data with kubernetes_config_map_v1_data via binary_data #2467

fullykubed opened this issue Apr 13, 2024 · 1 comment · May be fixed by #2616
Assignees

Comments

@fullykubed
Copy link

Description

I propose adding the binary_data field to kubernetes_config_map_v1_data to match the behavior of kubernetes_config_map.

Unlike kubernetes_config_map, the kubernetes_config_map_v1_data does not have a way to the b64 encoded binary_data directly. This is an issue when trying to add certain types of files such as PNGs to confgmaps as terraform file cannot be used without producing the following error:

│ Call to function "file" failed: contents of "./vault.png" are not valid
│ UTF-8; use the filebase64 function to obtain the Base64 encoded contents or
│ the other file functions (e.g. filemd5, filesha256) to obtain file hashing
│ results instead.

Reading the file via filebase64 and then attempting to decode does not work either.

Community Note

  • Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
  • If you are interested in working on this issue or have submitted a pull request, please leave a comment
@fullykubed fullykubed changed the title Allow inputting binary_data encoded data with kubernetes_config_map_v1_data Allow inputting b64 encoded data with kubernetes_config_map_v1_data via binary_data Apr 13, 2024
@JaylonmcShan03 JaylonmcShan03 self-assigned this Oct 30, 2024
@JaylonmcShan03
Copy link
Contributor

@fullykubed Hi, I will be implementing the binary_data field in a few weeks!

@JaylonmcShan03 JaylonmcShan03 linked a pull request Nov 4, 2024 that will close this issue
2 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants