feat(cmd): add auth:whoami --all #118

bacongobbler · 2016-07-07T22:29:40Z

~~requires changes in deis/controller-sdk-go before it is considered feature-complete.~~

closes #54
requires deis/controller-sdk-go#36

deis-bot · 2016-07-07T22:29:42Z

@Joshua-Anderson and @arschles are potential reviewers of this pull request based on my analysis of git blame information. Thanks @bacongobbler!

arschles · 2016-07-07T22:30:55Z

So far, lgtm. Feel free to ping when you want a final review!

bacongobbler · 2016-07-07T22:57:24Z

done, ready for final review once I hook this up to e2e

bacongobbler · 2016-07-07T23:23:11Z

ah, forgot to glide up. One sec

bacongobbler · 2016-07-11T15:37:34Z

This PR keeps hitting a github merge issue:

> git checkout -f af935fbc557652ab3f46302f611704f4541a1f70
[BFA] Scanning build for known causes...
[BFA] No failure causes found
[BFA] Done. 0s

GitHub has been notified of this commit’s build result

hudson.plugins.git.GitException: Could not merge AnyObjectId[aec1656679a52459b57579b715c3dcc00c7e0143]
    at org.jenkinsci.plugins.gitclient.CliGitAPIImpl$3.execute(CliGitAPIImpl.java:590)
    at org.jenkinsci.plugins.github_branch_source.GitHubSCMSource$MergeWith.decorateRevisionToBuild(GitHubSCMSource.java:667)
    at hudson.plugins.git.GitSCM.determineRevisionToBuild(GitSCM.java:998)
    at hudson.plugins.git.GitSCM.checkout(GitSCM.java:1087)
    at org.jenkinsci.plugins.workflow.steps.scm.SCMStep.checkout(SCMStep.java:109)
    at org.jenkinsci.plugins.workflow.cps.CpsScmFlowDefinition.create(CpsScmFlowDefinition.java:108)
    at org.jenkinsci.plugins.workflow.multibranch.SCMBinder.create(SCMBinder.java:85)
    at org.jenkinsci.plugins.workflow.job.WorkflowRun.run(WorkflowRun.java:206)
    at hudson.model.ResourceController.execute(ResourceController.java:98)
    at hudson.model.Executor.run(Executor.java:410)
Caused by: hudson.plugins.git.GitException: Command "git merge aec1656679a52459b57579b715c3dcc00c7e0143" returned status code 1:
stdout: Auto-merging cmd/ps.go
CONFLICT (content): Merge conflict in cmd/ps.go
Automatic merge failed; fix conflicts and then commit the result.

Which seems to be from aec1656 but I'm not sure why because this PR is up to date. Gonna try closing and re-opening to see what happens.

bacongobbler · 2016-07-11T15:40:27Z

I also cannot retry the job because I hit a server error:

Stack trace

javax.servlet.ServletException: java.lang.NullPointerException
    at org.kohsuke.stapler.Stapler.tryInvoke(Stapler.java:796)
    at org.kohsuke.stapler.Stapler.invoke(Stapler.java:876)
    at org.kohsuke.stapler.MetaClass$11.dispatch(MetaClass.java:380)
    at org.kohsuke.stapler.Stapler.tryInvoke(Stapler.java:746)
    at org.kohsuke.stapler.Stapler.invoke(Stapler.java:876)
    at org.kohsuke.stapler.MetaClass$11.dispatch(MetaClass.java:380)
    at org.kohsuke.stapler.Stapler.tryInvoke(Stapler.java:746)
    at org.kohsuke.stapler.Stapler.invoke(Stapler.java:876)
    at org.kohsuke.stapler.MetaClass$5.doDispatch(MetaClass.java:233)
    at org.kohsuke.stapler.NameBasedDispatcher.dispatch(NameBasedDispatcher.java:58)
    at org.kohsuke.stapler.Stapler.tryInvoke(Stapler.java:746)
    at org.kohsuke.stapler.Stapler.invoke(Stapler.java:876)
    at org.kohsuke.stapler.MetaClass$5.doDispatch(MetaClass.java:233)
    at org.kohsuke.stapler.NameBasedDispatcher.dispatch(NameBasedDispatcher.java:58)
    at org.kohsuke.stapler.Stapler.tryInvoke(Stapler.java:746)
    at org.kohsuke.stapler.Stapler.invoke(Stapler.java:876)
    at org.kohsuke.stapler.MetaClass$5.doDispatch(MetaClass.java:233)
    at org.kohsuke.stapler.NameBasedDispatcher.dispatch(NameBasedDispatcher.java:58)
    at org.kohsuke.stapler.Stapler.tryInvoke(Stapler.java:746)
    at org.kohsuke.stapler.Stapler.invoke(Stapler.java:876)
    at org.kohsuke.stapler.Stapler.invoke(Stapler.java:649)
    at org.kohsuke.stapler.Stapler.service(Stapler.java:238)
    at javax.servlet.http.HttpServlet.service(HttpServlet.java:790)
    at org.eclipse.jetty.servlet.ServletHolder.handle(ServletHolder.java:812)
    at org.eclipse.jetty.servlet.ServletHandler$CachedChain.doFilter(ServletHandler.java:1669)
    at hudson.util.PluginServletFilter$1.doFilter(PluginServletFilter.java:135)
    at hudson.plugins.greenballs.GreenBallFilter.doFilter(GreenBallFilter.java:59)
    at hudson.util.PluginServletFilter$1.doFilter(PluginServletFilter.java:132)
    at hudson.util.PluginServletFilter.doFilter(PluginServletFilter.java:126)
    at org.eclipse.jetty.servlet.ServletHandler$CachedChain.doFilter(ServletHandler.java:1652)
    at hudson.security.csrf.CrumbFilter.doFilter(CrumbFilter.java:49)
    at org.eclipse.jetty.servlet.ServletHandler$CachedChain.doFilter(ServletHandler.java:1652)
    at hudson.security.ChainedServletFilter$1.doFilter(ChainedServletFilter.java:84)
    at hudson.security.UnwrapSecurityExceptionFilter.doFilter(UnwrapSecurityExceptionFilter.java:51)
    at hudson.security.ChainedServletFilter$1.doFilter(ChainedServletFilter.java:87)
    at jenkins.security.ExceptionTranslationFilter.doFilter(ExceptionTranslationFilter.java:117)
    at hudson.security.ChainedServletFilter$1.doFilter(ChainedServletFilter.java:87)
    at org.acegisecurity.providers.anonymous.AnonymousProcessingFilter.doFilter(AnonymousProcessingFilter.java:125)
    at hudson.security.ChainedServletFilter$1.doFilter(ChainedServletFilter.java:87)
    at org.acegisecurity.ui.rememberme.RememberMeProcessingFilter.doFilter(RememberMeProcessingFilter.java:142)
    at hudson.security.ChainedServletFilter$1.doFilter(ChainedServletFilter.java:87)
    at org.acegisecurity.ui.AbstractProcessingFilter.doFilter(AbstractProcessingFilter.java:271)
    at hudson.security.ChainedServletFilter$1.doFilter(ChainedServletFilter.java:87)
    at jenkins.security.BasicHeaderProcessor.doFilter(BasicHeaderProcessor.java:93)
    at hudson.security.ChainedServletFilter$1.doFilter(ChainedServletFilter.java:87)
    at org.acegisecurity.context.HttpSessionContextIntegrationFilter.doFilter(HttpSessionContextIntegrationFilter.java:249)
    at hudson.security.HttpSessionContextIntegrationFilter2.doFilter(HttpSessionContextIntegrationFilter2.java:67)
    at hudson.security.ChainedServletFilter$1.doFilter(ChainedServletFilter.java:87)
    at hudson.security.ChainedServletFilter.doFilter(ChainedServletFilter.java:76)
    at hudson.security.HudsonFilter.doFilter(HudsonFilter.java:171)
    at org.eclipse.jetty.servlet.ServletHandler$CachedChain.doFilter(ServletHandler.java:1652)
    at org.kohsuke.stapler.compression.CompressionFilter.doFilter(CompressionFilter.java:49)
    at org.eclipse.jetty.servlet.ServletHandler$CachedChain.doFilter(ServletHandler.java:1652)
    at hudson.util.CharacterEncodingFilter.doFilter(CharacterEncodingFilter.java:82)
    at org.eclipse.jetty.servlet.ServletHandler$CachedChain.doFilter(ServletHandler.java:1652)
    at org.kohsuke.stapler.DiagnosticThreadNameFilter.doFilter(DiagnosticThreadNameFilter.java:30)
    at org.eclipse.jetty.servlet.ServletHandler$CachedChain.doFilter(ServletHandler.java:1652)
    at org.eclipse.jetty.servlet.ServletHandler.doHandle(ServletHandler.java:585)
    at org.eclipse.jetty.server.handler.ScopedHandler.handle(ScopedHandler.java:143)
    at org.eclipse.jetty.security.SecurityHandler.handle(SecurityHandler.java:553)
    at org.eclipse.jetty.server.session.SessionHandler.doHandle(SessionHandler.java:223)
    at org.eclipse.jetty.server.handler.ContextHandler.doHandle(ContextHandler.java:1127)
    at org.eclipse.jetty.servlet.ServletHandler.doScope(ServletHandler.java:515)
    at org.eclipse.jetty.server.session.SessionHandler.doScope(SessionHandler.java:185)
    at org.eclipse.jetty.server.handler.ContextHandler.doScope(ContextHandler.java:1061)
    at org.eclipse.jetty.server.handler.ScopedHandler.handle(ScopedHandler.java:141)
    at org.eclipse.jetty.server.handler.HandlerWrapper.handle(HandlerWrapper.java:97)
    at org.eclipse.jetty.server.Server.handle(Server.java:499)
    at org.eclipse.jetty.server.HttpChannel.handle(HttpChannel.java:311)
    at org.eclipse.jetty.server.HttpConnection.onFillable(HttpConnection.java:257)
    at org.eclipse.jetty.io.AbstractConnection$2.run(AbstractConnection.java:544)
    at winstone.BoundedExecutorService$1.run(BoundedExecutorService.java:77)
    at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1145)
    at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:615)
    at java.lang.Thread.run(Thread.java:745)
Caused by: java.lang.NullPointerException
    at com.chikli.hudson.plugin.naginator.NaginatorRetryAction.doIndex(NaginatorRetryAction.java:51)
    at sun.reflect.GeneratedMethodAccessor2481.invoke(Unknown Source)
    at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
    at java.lang.reflect.Method.invoke(Method.java:606)
    at org.kohsuke.stapler.Function$InstanceFunction.invoke(Function.java:324)
    at org.kohsuke.stapler.Function.bindAndInvoke(Function.java:167)
    at org.kohsuke.stapler.Function.bindAndInvokeAndServeResponse(Function.java:100)
    at org.kohsuke.stapler.IndexDispatcher.dispatch(IndexDispatcher.java:26)
    at org.kohsuke.stapler.Stapler.tryInvoke(Stapler.java:746)
    ... 74 more

bacongobbler · 2016-07-11T15:44:49Z

Looks like we are hitting this bug which was opened only 6 days ago: https://issues.jenkins-ci.org/browse/JENKINS-36438

bacongobbler · 2016-07-21T21:18:50Z

Because the "requires" syntax did not exist before, I manually created a build referencing the sdk PR.

bacongobbler · 2016-07-21T21:19:47Z

closing then re-opening another PR to hopefully resolve this CI issue.

bacongobbler added enhancement in progress labels Jul 7, 2016

bacongobbler added this to the v2.2 milestone Jul 7, 2016

bacongobbler self-assigned this Jul 7, 2016

bacongobbler mentioned this pull request Jul 7, 2016

feat(auth): add auth:whoami endpoint deis/controller-sdk-go#36

Merged

bacongobbler added awaiting review and removed in progress labels Jul 7, 2016

bacongobbler force-pushed the 54-auth-whoami branch from d01cbce to 564091c Compare July 7, 2016 22:55

bacongobbler force-pushed the 54-auth-whoami branch from 3a29ceb to 3eca642 Compare July 7, 2016 22:59

bacongobbler mentioned this pull request Jul 7, 2016

feat(cmd): add auth:whoami --all tests deis/workflow-e2e#250

Merged

bacongobbler force-pushed the 54-auth-whoami branch 2 times, most recently from eab3c87 to af935fb Compare July 11, 2016 15:29

bacongobbler closed this Jul 11, 2016

bacongobbler reopened this Jul 11, 2016

deis-admin added in progress and removed awaiting review labels Jul 11, 2016

bacongobbler force-pushed the 54-auth-whoami branch from af935fb to 4c7fd97 Compare July 13, 2016 21:23

bacongobbler removed this from the v2.2 milestone Jul 14, 2016

bacongobbler mentioned this pull request Jul 14, 2016

feat(Jenkinsfile): receive sdk-related params #128

Merged

bacongobbler added awaiting review and removed in progress labels Jul 21, 2016

bacongobbler added this to the v2.3 milestone Jul 21, 2016

feat(cmd): add auth:whoami --all

58e38b9

bacongobbler force-pushed the 54-auth-whoami branch from 4c7fd97 to 58e38b9 Compare July 21, 2016 20:51

bacongobbler closed this Jul 21, 2016

bacongobbler reopened this Jul 21, 2016

deis-admin added in progress and removed awaiting review labels Jul 21, 2016

bacongobbler added awaiting review and removed in progress labels Jul 21, 2016

bacongobbler closed this Jul 21, 2016

deis-admin removed the awaiting review label Jul 21, 2016

bacongobbler mentioned this pull request Jul 21, 2016

feat(cmd): add auth:whoami --all #145

Merged

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

feat(cmd): add auth:whoami --all #118

feat(cmd): add auth:whoami --all #118

bacongobbler commented Jul 7, 2016 •

edited

Loading

deis-bot commented Jul 7, 2016

arschles commented Jul 7, 2016

bacongobbler commented Jul 7, 2016

bacongobbler commented Jul 7, 2016

bacongobbler commented Jul 11, 2016

bacongobbler commented Jul 11, 2016

bacongobbler commented Jul 11, 2016

bacongobbler commented Jul 21, 2016

bacongobbler commented Jul 21, 2016

feat(cmd): add auth:whoami --all #118

feat(cmd): add auth:whoami --all #118

Conversation

bacongobbler commented Jul 7, 2016 • edited Loading

deis-bot commented Jul 7, 2016

arschles commented Jul 7, 2016

bacongobbler commented Jul 7, 2016

bacongobbler commented Jul 7, 2016

bacongobbler commented Jul 11, 2016

bacongobbler commented Jul 11, 2016

bacongobbler commented Jul 11, 2016

bacongobbler commented Jul 21, 2016

bacongobbler commented Jul 21, 2016

bacongobbler commented Jul 7, 2016 •

edited

Loading