ARO_IMAGE is mandatory now

[mjudeikis]

As a developer, I don't want to care if ARO_IMAGE is set or not if I'm not working on that part of the code. We should have sane defaults.
Currently, after operator merged it is required for devs to set ARO_IMAGE for aro rp to work.

the suggestion was to use:

func (d *dev) AROOperatorImage() string {
	override := os.Getenv("ARO_IMAGE")
        tag := vesiong.GitCommit 
	if override != "" {
		return override
	}
        # in case we running on dirty code without override:
        if tag == "unknown"{
             tag = "master"
        }
	return fmt.Sprintf("%s.azurecr.io/aro:%s", d.acrName, tag)
}

For discussion @asalkeld @jim-minter @ehashman

[asalkeld]

so, we have ARO_IMAGE in env.example, I think this is not a big hassle - it's just a once off change..

[asalkeld]

one option would be to add it to env/secrets - assuming that is purely for developers and not used in CI or any other context

[mjudeikis]

My question is why we cant decide it dynamically if override is not set? What is stopping factor here?