Skip to content

Remote builder secrets#1017

Merged
alexellis merged 2 commits intomasterfrom
remote_builder_secrets
Mar 23, 2026
Merged

Remote builder secrets#1017
alexellis merged 2 commits intomasterfrom
remote_builder_secrets

Conversation

@alexellis
Copy link
Member

Description

Remote builder secrets

Motivation and Context

  • Adds support for sealing secrets for use with faas-cli

How Has This Been Tested?

  • Tested e2e and saw with a Dockerfile that captured a build
    time secret - the published image showed the value was captured
    as expected.

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

README/cobra CLI help updated with examples of the new commands.

The getMissingTemplates call was performed within an if/else
but was identical so could be moved up one level higher.

Signed-off-by: Alex Ellis (OpenFaaS Ltd) <alexellis2@gmail.com>
@alexellis alexellis force-pushed the remote_builder_secrets branch from c32e7eb to e0b0c1a Compare March 23, 2026 18:27
@reviewfn

This comment has been minimized.

@reviewfn

This comment has been minimized.

* Adds support for sealing secrets for use with faas-cli
* Tested e2e and saw with a Dockerfile that captured a build
time secret - the published image showed the value was captured
as expected.

Signed-off-by: Alex Ellis (OpenFaaS Ltd) <alexellis2@gmail.com>
@alexellis alexellis force-pushed the remote_builder_secrets branch from e0b0c1a to bce0593 Compare March 23, 2026 18:29
@reviewfn
Copy link

reviewfn bot commented Mar 23, 2026

An error occurred while generating the code review file. Please try again.

AI agent details.

Agent processing time: 3m19.409s
Environment preparation time: 3.564s
Total time from webhook: 3m27.886s

@alexellis alexellis merged commit 4889ec1 into master Mar 23, 2026
3 checks passed
@alexellis alexellis deleted the remote_builder_secrets branch March 23, 2026 18:33
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.

1 participant