Skip to content

Fix issue with awscli and image building#74

Open
someshkoli wants to merge 16 commits into
deliverybot:masterfrom
headout:master
Open

Fix issue with awscli and image building#74
someshkoli wants to merge 16 commits into
deliverybot:masterfrom
headout:master

Conversation

@someshkoli

@someshkoli someshkoli commented Aug 7, 2022

Copy link
Copy Markdown

Todo: Requires some chore cleanup, lmk once rest of the things look good I'll update those

Fixes:
#56
#66
#73
#49

glopezep and others added 12 commits January 24, 2022 08:39
Signed-off-by: Somesh Koli <somesh.koli@headout.com>
Signed-off-by: Somesh Koli <somesh.koli@headout.com>
Signed-off-by: Somesh Koli <somesh.koli@headout.com>
Signed-off-by: Somesh Koli <somesh.koli@headout.com>
Signed-off-by: Somesh Koli <somesh.koli@headout.com>
Signed-off-by: Somesh Koli <somesh.koli@headout.com>
Signed-off-by: Somesh Koli <somesh.koli@headout.com>
@TomHart

TomHart commented Sep 29, 2022

Copy link
Copy Markdown

I've just tried this and got the error:

 /usr/bin/helm: line 1: syntax error: unexpected ")"

YAML:

            -   name: 'Deploy'
                uses: 'headout/helm@master'
                with:
                    ....etc
                env:
                    KUBECONFIG_FILE: '${{ secrets.KUBECONFIG }}'

Any idea why that might be? It's on a self-hosted Github runner running on a rPi4

@someshkoli

Copy link
Copy Markdown
Author

I've just tried this and got the error:

 /usr/bin/helm: line 1: syntax error: unexpected ")"

YAML:

            -   name: 'Deploy'
                uses: 'headout/helm@master'
                with:
                    ....etc
                env:
                    KUBECONFIG_FILE: '${{ secrets.KUBECONFIG }}'

Any idea why that might be? It's on a self-hosted Github runner running on a rPi4

@TomHart can you turn on debug logs and send details?
Looks like some syntactical error in helm command which might due to bad params

@someshkoli

Copy link
Copy Markdown
Author

Btw have been using this at headout for past 2 months, works flawlessly.

@hinling-blisspoint

Copy link
Copy Markdown

Btw have been using this at headout for past 2 months, works flawlessly.

any reason why this pr is not merged? I am using @someshkoli fork also and it solved the awscli not found problem.

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.

5 participants