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

Add node registration entry alias example #231

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

kfox1111
Copy link
Contributor

@kfox1111 kfox1111 commented Nov 3, 2021

No description provided.

@netlify
Copy link

netlify bot commented Nov 3, 2021

✔️ Deploy Preview for spiffe ready!

🔨 Explore the source changes: 0a55a95

🔍 Inspect the deploy log: https://app.netlify.com/sites/spiffe/deploys/6182f1b7b4ad0c0008c1d654

😎 Browse the preview: https://deploy-preview-231--spiffe.netlify.app

Copy link
Member

@evan2645 evan2645 left a comment

Choose a reason for hiding this comment

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

Thank you @kfox1111! This looks good to me

@sanderson042 is the formatting alright?

@evan2645
Copy link
Member

evan2645 commented Nov 3, 2021

@kfox1111 would you mind adding a DCO to this commit and force-pushing back to the branch? It can be added with git flag -s. DCO is required for all SPIFFE projects

https://github.com/spiffe/spiffe.io/pull/231/checks?check_run_id=4096274127

https://en.wikipedia.org/wiki/Developer_Certificate_of_Origin

@kfox1111
Copy link
Contributor Author

kfox1111 commented Nov 3, 2021

Yeah. I can do that... I had just used the github interface to quickly prepare the change. Its a lot more complicated to do the signoff. Oh well.

@kfox1111
Copy link
Contributor Author

kfox1111 commented Nov 3, 2021

Ok. Should be fixed.

For example:

```
spire-server entry create -node -spiffeID spiffe://devvm.local/mynodealias -selector tpm:pub_hash:xxxxx
Copy link
Member

@evan2645 evan2645 Nov 4, 2021

Choose a reason for hiding this comment

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

@sanderson042 what do you think about a more widely used selector here? Perhaps aws_iid:tag:<TAG_NAME>:<TAG_VALUE>

Copy link
Member

Choose a reason for hiding this comment

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

Or something :)

Copy link
Member

Choose a reason for hiding this comment

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

@kfox1111 heya 👋 just wanted to say that Evan's suggestion looks good to me, and since this is supposed to be an example we could even leave the templating stuff out and use something like aws_iid:tag:tier:frontend, we can then get this merged in 💪

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.

3 participants