{"action":"created","issue":{"url":"https://api.github.com/repos/cloudfoundry-incubator/disaster-recovery-acceptance-tests/issues/37","repository_url":"https://api.github.com/repos/cloudfoundry-incubator/disaster-recovery-acceptance-tests","labels_url":"https://api.github.com/repos/cloudfoundry-incubator/disaster-recovery-acceptance-tests/issues/37/labels{/name}","comments_url":"https://api.github.com/repos/cloudfoundry-incubator/disaster-recovery-acceptance-tests/issues/37/comments","events_url":"https://api.github.com/repos/cloudfoundry-incubator/disaster-recovery-acceptance-tests/issues/37/events","html_url":"https://github.com/cloudfoundry-incubator/disaster-recovery-acceptance-tests/pull/37","id":349184537,"node_id":"MDExOlB1bGxSZXF1ZXN0MjA3MzYwNzEz","number":37,"title":"Update drats to use new credhub permissions model","user":{"login":"athornton2012","id":22150282,"node_id":"MDQ6VXNlcjIyMTUwMjgy","avatar_url":"https://avatars3.githubusercontent.com/u/22150282?v=4","gravatar_id":"","url":"https://api.github.com/users/athornton2012","html_url":"https://github.com/athornton2012","followers_url":"https://api.github.com/users/athornton2012/followers","following_url":"https://api.github.com/users/athornton2012/following{/other_user}","gists_url":"https://api.github.com/users/athornton2012/gists{/gist_id}","starred_url":"https://api.github.com/users/athornton2012/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/athornton2012/subscriptions","organizations_url":"https://api.github.com/users/athornton2012/orgs","repos_url":"https://api.github.com/users/athornton2012/repos","events_url":"https://api.github.com/users/athornton2012/events{/privacy}","received_events_url":"https://api.github.com/users/athornton2012/received_events","type":"User","site_admin":false},"labels":[{"id":659153460,"node_id":"MDU6TGFiZWw2NTkxNTM0NjA=","url":"https://api.github.com/repos/cloudfoundry-incubator/disaster-recovery-acceptance-tests/labels/scheduled","name":"scheduled","color":"BFD4F2","default":false}],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":4,"created_at":"2018-08-09T15:43:06Z","updated_at":"2018-08-27T18:27:05Z","closed_at":null,"author_association":"MEMBER","pull_request":{"url":"https://api.github.com/repos/cloudfoundry-incubator/disaster-recovery-acceptance-tests/pulls/37","html_url":"https://github.com/cloudfoundry-incubator/disaster-recovery-acceptance-tests/pull/37","diff_url":"https://github.com/cloudfoundry-incubator/disaster-recovery-acceptance-tests/pull/37.diff","patch_url":"https://github.com/cloudfoundry-incubator/disaster-recovery-acceptance-tests/pull/37.patch"},"body":"- Ensure that the credhub fixture app has permission to create\r\ncredentials\r\n- Ensure that the credential created by the credhub fixture app is not\r\ntied to the instance guid\r\n- Update the ci task to require a client and client secret that have\r\ncredhub write_acl permissions\r\n\r\nSigned-off-by: Chhavi Kankaria <ckankaria@pivotal.io>\r\n\r\n## Checklist\r\n\r\n- [X] What component are you testing?\r\nRuntime CredHub\r\n\r\n- [X] Have you created a `TestCase` and added it to the list of cases to be run?\r\nWe modified an existing test case\r\n\r\n- [X] Have you added an `include_<testcase-name>` property and any other new properties to the sample integration_config.json? We modified the ci task to include the credhub admin client name and client secret.\r\n\r\n- [X] Have you manually validated your `TestCase` against a deployed Cloud Foundry? If so, which version? we have validated against cf-deployment (waiting for branch to be merged https://github.com/cloudfoundry/cf-deployment/pull/580)\r\n\r\n- [X] Does this change rely on a particular version of CF release?\r\nThis change relies on a version of cf-deployment in which https://github.com/cloudfoundry/cf-deployment/pull/580 is merged. \r\n\r\n- [X] Are you available for a cross-team pair to help troubleshoot your PR?\r\nYes.\r\n\r\n- [X] Have you submitted a pull request to modify the cf-deployment [backup and restore opsfile](https://github.com/cloudfoundry/cf-deployment/blob/master/operations/experimental/enable-backup-restore.yml) to add a backup job and properties where appropriate? \r\nYes https://github.com/cloudfoundry/cf-deployment/pull/580\r\n"},"comment":{"url":"https://api.github.com/repos/cloudfoundry-incubator/disaster-recovery-acceptance-tests/issues/comments/416321660","html_url":"https://github.com/cloudfoundry-incubator/disaster-recovery-acceptance-tests/pull/37#issuecomment-416321660","issue_url":"https://api.github.com/repos/cloudfoundry-incubator/disaster-recovery-acceptance-tests/issues/37","id":416321660,"node_id":"MDEyOklzc3VlQ29tbWVudDQxNjMyMTY2MA==","user":{"login":"chhhavi","id":2661095,"node_id":"MDQ6VXNlcjI2NjEwOTU=","avatar_url":"https://avatars1.githubusercontent.com/u/2661095?v=4","gravatar_id":"","url":"https://api.github.com/users/chhhavi","html_url":"https://github.com/chhhavi","followers_url":"https://api.github.com/users/chhhavi/followers","following_url":"https://api.github.com/users/chhhavi/following{/other_user}","gists_url":"https://api.github.com/users/chhhavi/gists{/gist_id}","starred_url":"https://api.github.com/users/chhhavi/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/chhhavi/subscriptions","organizations_url":"https://api.github.com/users/chhhavi/orgs","repos_url":"https://api.github.com/users/chhhavi/repos","events_url":"https://api.github.com/users/chhhavi/events{/privacy}","received_events_url":"https://api.github.com/users/chhhavi/received_events","type":"User","site_admin":false},"created_at":"2018-08-27T18:27:05Z","updated_at":"2018-08-27T18:27:05Z","author_association":"MEMBER","body":"@ljfranklin Do you want to write this in a way where it's compatible with both PAS 2.3 and but also the latest cf deployment which has CredHub 2.0?\r\n\r\n "}}
{"actor":{"display_login":"chhhavi"}}