Description of the problem / feature request:

When an action is tagged with no-remote or no-remote-cache, it's not properly using the disk cache. Only with no-cache should it exclude the disk cache as well.

Bugs: what's the simplest, easiest way to reproduce this bug? Please provide a minimal example if possible.

bazel build --disk_cache=/tmp/disk_cache -- //some/cacheable:target
bazel clean
bazel build --disk_cache=/tmp/disk_cache --modify_execution_info='.*=+no-remote-cache' -- //some/cacheable:target

The second build won't check the disk cache.

What operating system are you running Bazel on?

macOS 11.4

What's the output of bazel info release?

4.1.0, but is still broken at HEAD.