deis / workflow-cli

The CLI for Deis Workflow
http://deis.com
MIT License
31 stars 43 forks source link

apps:transfer scenario exit code discrepancy #29

Closed vdice closed 8 years ago

vdice commented 8 years ago

e2e has caught an intermittent error where the app is transferred from a testUser to an admin and when a subsequent deis info command is invoked (by testUser) we expect said command to exit with 1, yet 0 is produced:

05:59:51 remote: Done, test-27862817:v2 deployed to Deis        
05:59:51 remote: --->         
05:59:51 remote: Use 'deis open' to view this application in your browser        
05:59:51 remote: --->         
05:59:51 remote: To learn more, use 'deis help' or visit http://deis.io        
05:59:51 remote: [DEBUG]         
05:59:51 remote: running [git gc] in directory /home/git/test-27862817.git        
05:59:51 To ssh://git@deis.10.231.253.119.xip.io:2222/test-27862817.git
05:59:51  * [new branch]      master -> master
05:59:51 $ deis apps:transfer admin
05:59:51 $ deis info -a test-27862817
05:59:51 Transferring test-27862817 to admin... === test-27862817 Application
05:59:51 updated:  2016-03-24T11:59:48UTC
05:59:51 uuid:     a0702efb-ceac-4570-923d-7b71ac7239bd
05:59:51 created:  2016-03-24T11:59:36UTC
05:59:51 url:      test-27862817.10.231.253.119.xip.io
05:59:51 owner:    test-81
05:59:51 id:       test-27862817
05:59:51 
05:59:51 === test-27862817 Processes
05:59:51 --- web:
05:59:51 test-27862817-v2-web-5bjsd up (v2)
05:59:51 
05:59:51 === test-27862817 Domains
05:59:51 test-27862817
05:59:51 
05:59:51 $ deis apps:destroy --app=test-27862817 --confirm=test-27862817
05:59:51 Destroying test-27862817...
05:59:51 Error: 
05:59:51 403 Forbidden
05:59:51 detail: You do not have permission to perform this action.
05:59:51 
05:59:51 done
05:59:51 
05:59:51 • Failure [16.979 seconds]
05:59:51 Apps
05:59:51 /go/src/github.com/deis/workflow-e2e/tests/apps_test.go:204
05:59:51   with a deployed app
05:59:51   /go/src/github.com/deis/workflow-e2e/tests/apps_test.go:174
05:59:51     can transfer the app to another owner [It]
05:59:51     /go/src/github.com/deis/workflow-e2e/tests/apps_test.go:173
05:59:51 
05:59:51     No future change is possible.  Bailing out early after 0.133s.
05:59:51     Expected
05:59:51         <int>: 0
05:59:51     to match exit code:
05:59:51         <int>: 1
05:59:51 
05:59:51     /go/src/github.com/deis/workflow-e2e/tests/apps_test.go:165
05:59:51 ------------------------------

Related logs can be found here

Joshua-Anderson commented 8 years ago

Have you figured this out? I'm pretty confused by this, because this is the line that is responsible for printing that error message, and it seems like it would be impossible for it not to return 1.

bacongobbler commented 8 years ago

ping @vdice ^^

vdice commented 8 years ago

To my knowledge, this issue has not been seen in CI for some time. Closing for now; can re-open if detected.