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

Custom cluster builder tries to retrieve kaniko secret #2768

Closed
tbarrella opened this issue Aug 29, 2019 · 2 comments · Fixed by #2783
Closed

Custom cluster builder tries to retrieve kaniko secret #2768

tbarrella opened this issue Aug 29, 2019 · 2 comments · Fixed by #2783
Assignees
Labels
area/build build/custom kind/bug Something isn't working priority/p1 High impact feature/bug.

Comments

@tbarrella
Copy link
Contributor

Expected behavior

Using a custom builder for in-cluster builds doesn't require a kaniko secret

Actual behavior

If no kaniko secret exists, the build fails

Starting build...
Creating kaniko secret [kaniko-secret]...
FATA[0001] build failed: build failed: setting up pull secret: checking for existing kaniko secret: secrets "kaniko-secret" not found

Information

  • Skaffold version: v0.37.0
  • Operating system: Ubuntu 16.04
  • Contents of skaffold.yaml:
apiVersion: skaffold/v1beta13
kind: Config
build:
  artifacts:
  - image: <repo>
    custom:
      buildCommand: ./build.sh
  cluster: {}

Steps to reproduce the behavior

  1. Use any build.sh
  2. skaffold build
@dgageot dgageot self-assigned this Aug 30, 2019
@dgageot dgageot added build/custom kind/bug Something isn't working labels Aug 30, 2019
@dgageot dgageot assigned dgageot and unassigned dgageot Aug 30, 2019
@balopat balopat added the priority/p1 High impact feature/bug. label Aug 30, 2019
dgageot added a commit to dgageot/skaffold that referenced this issue Sep 2, 2019
Fixes GoogleContainerTools#2768

Signed-off-by: David Gageot <david@gageot.net>
@dgageot dgageot self-assigned this Sep 2, 2019
dgageot added a commit to dgageot/skaffold that referenced this issue Sep 3, 2019
Fixes GoogleContainerTools#2768

Signed-off-by: David Gageot <david@gageot.net>
dgageot added a commit to dgageot/skaffold that referenced this issue Sep 3, 2019
Fixes GoogleContainerTools#2768

Signed-off-by: David Gageot <david@gageot.net>
dgageot added a commit to dgageot/skaffold that referenced this issue Sep 3, 2019
Fixes GoogleContainerTools#2768

Signed-off-by: David Gageot <david@gageot.net>
dgageot added a commit to dgageot/skaffold that referenced this issue Sep 4, 2019
Fixes GoogleContainerTools#2768

Signed-off-by: David Gageot <david@gageot.net>
dgageot added a commit to dgageot/skaffold that referenced this issue Sep 4, 2019
Fixes GoogleContainerTools#2768

Signed-off-by: David Gageot <david@gageot.net>
dgageot added a commit to dgageot/skaffold that referenced this issue Sep 4, 2019
Fixes GoogleContainerTools#2768

Signed-off-by: David Gageot <david@gageot.net>
dgageot added a commit to dgageot/skaffold that referenced this issue Sep 4, 2019
Fixes GoogleContainerTools#2768

Signed-off-by: David Gageot <david@gageot.net>
dgageot added a commit to dgageot/skaffold that referenced this issue Sep 4, 2019
Fixes GoogleContainerTools#2768

Signed-off-by: David Gageot <david@gageot.net>
dgageot added a commit to dgageot/skaffold that referenced this issue Sep 4, 2019
Fixes GoogleContainerTools#2768

Signed-off-by: David Gageot <david@gageot.net>
@tbarrella
Copy link
Contributor Author

Hello, with v0.38.0 the above repro still works for me; I think it's because setDefaultClusterPullSecret ensures that pullSecretName is never ""

Maybe not obvious how to best change since it would be a breaking change...

@tbarrella
Copy link
Contributor Author

Looks like this is tracked at #2888

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/build build/custom kind/bug Something isn't working priority/p1 High impact feature/bug.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants