chore: re-generate when config or Dockerfile is changed

JoeWang1127 commented 2 months ago

In this PR:

Re-generate common protos and iam if generation config or Dockerfile is changed.

I use #3145 to verify the change.

diegomarquezp commented 2 months ago

LGTM. IIUC this is more generic than only allowing changes in the Dockerfile. I think this can allow modifications coming from anything that alters the output of the generation: postprocessing, gapic generator, Dockerfile, etc. And it's great to allow this btw, but maybe the title/description should be something like "re-generate when config or generation workflow has changed"?

From internal discussion with @JoeWang1127, this is wrong. There was a modification on what would allow the generation to start made in https://github.com/googleapis/sdk-platform-java/commit/74f20a253e65d9d4253541c4c8250d75abe8d0b8. So only the Dockerfile or the config yaml would trigger this.