I have found a few additional internal checks that depend on this, so we are not ready to merge this PR yet.
We need to discuss how we can move those checks to not use bazel.

Looks like we will still need to export some of the files or change their visibility so other targets can access them. How should we go about this? @gunan any suggestion?

I think we need some filegroup rules, and new data dependencies on these filegroup rules. Those fixes should be a part of this, as they will be "thinner" replacements to the all_files rules these tests are already depending on.
Could you go through the broken tests and triage the build failures?

@Androbin do you mind adding this suggested fix:
/tmpfs/src/github/tensorflow/tensorflow/core/ops/compat/BUILD:33:1: no such target '//tensorflow/core:ops/ops.pbtxt': target 'ops/ops.pbtxt' not declared in package 'tensorflow/core'; however, a source file of this name exists. (Perhaps add 'exports_files(["ops/ops.pbtxt"])' to tensorflow/core/BUILD?) defined by /tmpfs/src/github/tensorflow/tensorflow/core/BUILD and referenced by '//tensorflow/core/ops/compat:backwards_compatibility_test'

Good catch!

Last build failed because of image_retraining.
Rebased on last successful build and resolved merge conflicts.

Looks like there are a few more:
ERROR: /tmpfs/src/github/tensorflow/tensorflow/examples/image_retraining/BUILD:28:1: no such target '//tensorflow/examples/label_image:data/grace_hopper.jpg': target 'data/grace_hopper.jpg' not declared in package 'tensorflow/examples/label_image'; however, a source file of this name exists. (Perhaps add 'exports_files(["data/grace_hopper.jpg"])' to tensorflow/examples/label_image/BUILD?) defined by /tmpfs/src/github/tensorflow/tensorflow/examples/label_image/BUILD and referenced by '//tensorflow/examples/image_retraining:retrain_test'

Okay, that made bazel nobuild pass locally.

Last build succeeded (256848f).
Had to resolve minor merge conflict (620348f).

Thanks @Androbin! Let's see how the builds go.

We had some infra issue earlier. It should be fine now.

Last build successful (99f5d4e)
Had to resolve minor merge conflict (f6bda40)

Hi @Androbin sorry for the delay, I was thinking of bringing this PR in independently and was waiting for the sync to resolve merge conflicts. Let me give it another shot.

@Androbin I managed to create a pending internal change with your PR, and unfortunately found another internal dependency on all_files. Will let you know once we figure out how we can remove it. Thanks.

@Androbin quick update: the internal dependency issue has been fixed. I'm working on submitting this change internally following this process.

Nagging Assignee @gunan: It has been 14 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

bb582f1 has been submitted. I'm going to close this PR. Thanks @Androbin!