[GitHub] [beam] jaketf commented on pull request #11959: refactor HCLS IO ITs to support stores in other projects

2020-07-08 Thread GitBox
jaketf commented on pull request #11959: URL: https://github.com/apache/beam/pull/11959#issuecomment-655672949 @aaltay @pabloem checks seem good now. LMK if this needs anything else This is an automated message from the

[GitHub] [beam] jaketf commented on pull request #11959: refactor HCLS IO ITs to support stores in other projects

2020-07-08 Thread GitBox
jaketf commented on pull request #11959: URL: https://github.com/apache/beam/pull/11959#issuecomment-655632442 Run Java PostCommit This is an automated message from the Apache Git Service. To respond to the message, please

[GitHub] [beam] jaketf commented on pull request #11959: refactor HCLS IO ITs to support stores in other projects

2020-07-07 Thread GitBox
jaketf commented on pull request #11959: URL: https://github.com/apache/beam/pull/11959#issuecomment-655219973 Run Java PostCommit This is an automated message from the Apache Git Service. To respond to the message, please

[GitHub] [beam] jaketf commented on pull request #11959: refactor HCLS IO ITs to support stores in other projects

2020-07-07 Thread GitBox
jaketf commented on pull request #11959: URL: https://github.com/apache/beam/pull/11959#issuecomment-655202173 retest this please This is an automated message from the Apache Git Service. To respond to the message, please

[GitHub] [beam] jaketf commented on pull request #11959: refactor HCLS IO ITs to support stores in other projects

2020-07-07 Thread GitBox
jaketf commented on pull request #11959: URL: https://github.com/apache/beam/pull/11959#issuecomment-655145391 I have some level of access but I need additional permissions to modify IAM This is an automated message from the

[GitHub] [beam] jaketf commented on pull request #11959: refactor HCLS IO ITs to support stores in other projects

2020-07-02 Thread GitBox
jaketf commented on pull request #11959: URL: https://github.com/apache/beam/pull/11959#issuecomment-653207804 ultimately we need to make sure service-[project_number]@gcp-sa-healthcare.iam.gserviceaccount.com has Storage Object Viewer on the buckets we are importing from.

[GitHub] [beam] jaketf commented on pull request #11959: refactor HCLS IO ITs to support stores in other projects

2020-07-02 Thread GitBox
jaketf commented on pull request #11959: URL: https://github.com/apache/beam/pull/11959#issuecomment-653188130 I was able to reproduce this failure. It occurs when using the import method and [falling back to using the temp

[GitHub] [beam] jaketf commented on pull request #11959: refactor HCLS IO ITs to support stores in other projects

2020-07-02 Thread GitBox
jaketf commented on pull request #11959: URL: https://github.com/apache/beam/pull/11959#issuecomment-653182234 My focus has been shifted to other customers. Taking a look at why this test was failing now, will update this thread with a commit to fix or findings.