Skip to content

implemented kubernetes auth#155

Closed
jogueber wants to merge 2 commits into
hashicorp:masterfrom
jogueber:master
Closed

implemented kubernetes auth#155
jogueber wants to merge 2 commits into
hashicorp:masterfrom
jogueber:master

Conversation

@jogueber

Copy link
Copy Markdown

I felt it was useful to have kubernetes auth for the github action.
This is especially handy if you use self hosted runners that have kubernetes auth with vault.
I also added some basic unit tests for the auth.js

@hashicorp-cla

hashicorp-cla commented Nov 17, 2020

Copy link
Copy Markdown

CLA assistant check
All committers have signed the CLA.

@bguedel

bguedel commented Dec 7, 2020

Copy link
Copy Markdown

I think you might need to run a build?

https://github.com/hashicorp/vault-action/blob/master/dist/index.js#L14085 <--- i believe this is the bit that gets run, but it does not include your changes.

@jogueber

jogueber commented Dec 7, 2020

Copy link
Copy Markdown
Author

how can I run a build?

@bguedel

bguedel commented Dec 7, 2020

Copy link
Copy Markdown

Comment thread action.yml
Comment thread action.yml
Comment thread action.yml Outdated
Comment thread action.yml
roleName:
description: "The role name for the kubernetes authentification"
required: false
tokenPath:

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This should probably have a default /var/run/secrets/kubernetes.io/serviceaccount/token, which for 99% of the use cases the JWT will be found.

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

gonna do that :)

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@jojo19893 Are you still planning on making this change? I can, to get this moving again

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Poke 😄

@terry-yolabs

Copy link
Copy Markdown

I'd love to see this merged into the official hashicorp action. I'm presently using curl to perform a vault login and would feel better if I were able to do this via the officially supported vault-action.

Co-authored-by: Jason O'Donnell <2160810+jasonodonnell@users.noreply.github.com>
@FalconerTC

Copy link
Copy Markdown
Contributor

+1. Would love to see this merged!

@jasonodonnell

Copy link
Copy Markdown
Contributor

#218 implemented this and other features. Now that its merged closing this one.

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.

6 participants