Add owner reference to update-check job so that it points to Mattermost resource

[mattermost]

Add owner reference to update-check job so that it points to Mattermost resource

[Szymongib]

Hey @imskr, there were some issues with scripts installing tooling. You can rebase with master and try again.

[imskr]

Can I work on this @Szymongib ?

[Szymongib]

Sure @imskr :)
Let me know if you have any questions.

[imskr]

@Szymongib When setting up this repo locally and running command make build-image kind-load-image deploy I get following error:

go: added sigs.k8s.io/yaml v1.2.0
mv: rename /Users/shubhamkumar/go/src/github.com/mattermost/mattermost-operator/bin/controller-gen to /Users/shubhamkumar/go/src/github.com/mattermost/mattermost-operator/bin/controller-gen-v0.7.0: No such file or directory
make: *** [/Users/shubhamkumar/go/src/github.com/mattermost/mattermost-operator/bin/controller-gen] Error 1

[imskr]

@Szymongib I have a question?

1. Can we directly call LaunchMattermostUpdateJob function here ? so that the owner reference gets added

[Szymongib]

When setting up this repo locally and running command make build-image kind-load-image deploy I get following error:

Does the bin directory exists? You might want to try creating it, but not sure if that will help, will need to look into it.

Can we directly call LaunchMattermostUpdateJob function here ? so that the owner reference gets added

I do not think so. The Create function is called in multiple different places for multiple different objects, we would not like to lunch a job each time it is called. You could however use the ResourceHelpers Create function to create and update job instead of directly using the Client here

[imskr]

When setting up this repo locally and running command make build-image kind-load-image deploy I get following error:

Does the bin directory exists? You might want to try creating it, but not sure if that will help, will need to look into it.

No it doesn't inside root of repo

Can we directly call LaunchMattermostUpdateJob function here ? so that the owner reference gets added

I do not think so. The Create function is called in multiple different places for multiple different objects, we would not like to lunch a job each time it is called. You could however use the ResourceHelpers Create function to create and update job instead of directly using the Client here

Thanks

[imskr]

Thanks @Szymongib