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

Get bastion SSH users from installations instead of hard-coding #1946

Closed
wants to merge 30 commits into from

Conversation

tfussell
Copy link
Contributor

No description provided.

@xh3b4sd
Copy link
Contributor

xh3b4sd commented Oct 10, 2019

What should we do with this PR?

@tfussell
Copy link
Contributor Author

I worked on it in the last sig-releng working hours. It was for this issue https://github.com/giantswarm/giantswarm/issues/6049 I'd like to finish it up at some point, but the 1.15.4 release is the top priority at the moment. I'll try to wrap this up in the next week.

Systemd interface{}
}

ts := oauth2.StaticTokenSource(
Copy link
Contributor

Choose a reason for hiding this comment

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

Check

func newSSHUserList(ctx context.Context, config Config) (string, error) {
var err error
var privateRepo *privaterepo.PrivateRepo
{
c := privaterepo.Config{
Owner: "giantswarm",
Repo: "installations",
Token: env.GithubToken(),
}
privateRepo, err = privaterepo.New(c)
if err != nil {
return "", microerror.Mask(err)
}
}
var sshUserList string
{
content, err := privateRepo.Content(ctx, "default-draughtsman-configmap-values.yaml")
if err != nil {
return "", microerror.Mask(err)
}
sshUserList, err = privaterepo.ContentToSSHUserList(content)
if err != nil {
return "", microerror.Mask(err)
}
}
return sshUserList, nil
}
. I once started to write some abstraction code for fetching information from the installations repo. Let's use this if possible.

@tfussell tfussell changed the title initial commit of new ssh key fetching Get bastion SSH users from installations instead of hard-coding Oct 16, 2019
@tfussell tfussell marked this pull request as ready for review October 16, 2019 14:58
@tfussell tfussell self-assigned this Oct 30, 2019
@tfussell tfussell changed the base branch from master to legacy October 30, 2019 14:20
@tfussell tfussell changed the base branch from legacy to master October 30, 2019 14:24
@tfussell tfussell force-pushed the bastion-users-from-installations branch from e67b959 to c7b5dc2 Compare October 30, 2019 14:25
@tfussell tfussell changed the base branch from master to legacy October 30, 2019 14:26
@calvix
Copy link
Member

calvix commented Oct 31, 2019

just FYI this is for testing/debugging e2e only, we don't create bastions fo TC in real enviroments inclusing test envs like ginger or gauss

@xh3b4sd
Copy link
Contributor

xh3b4sd commented Mar 16, 2020

Is this PR still relevant?

@tfussell
Copy link
Contributor Author

Is this PR still relevant?

I think I'm going down the wrong path with this. I've spent too much time on it already and it's still not working. Also I had an idea for simplifying the approach drastically. I will close this for now.

@tfussell tfussell closed this Mar 18, 2020
@tfussell tfussell deleted the bastion-users-from-installations branch March 18, 2020 18:17
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.

None yet

3 participants