ovirt: Add machine-config-operator presubmit

[rgolangh]

Depends-on: #4340

[kikisdeliveryservice]

waiting for #4340 to merge. looks fine otherwise.

/hold

[rgolangh]

On Mon, 6 Jan 2020 at 20:32, Kirsten ***@***.***> wrote: waiting for #4340 <#4340> to merge. looks fine otherwise.

It is other way round, this PR is necessary for #4340 to be green to merge. /hold

…

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#6533?email_source=notifications&email_token=ABGBYHBFNEHE22P6K64WFWLQ4N2MBA5CNFSM4KBVQEH2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEIGKSHY#issuecomment-571255071>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/ABGBYHGHY6YUISE36PUQYP3Q4N2MBANCNFSM4KBVQEHQ> .

[kikisdeliveryservice]

wait.. now im confused... is it post holiday haze? :)

This PR is adding ovirt tests to MCO right? But AFAICT openshift/installer#1948 which is what adds ovirt to installer isn't merged yet no?

So wouldn't this PR depend on #1948? We can't run any tests until installer can install ovirt... no?

Looking closely, not sure why installer e2e would depend on mco e2e (via #4340 )

Can you clarify so I understand?

[rgolangh]

You are 100% right, I was confusing this with other mao PR. This is my evening haze :)

…

On Tue, 7 Jan 2020 at 02:00, Kirsten ***@***.***> wrote: wait.. now im confused... is it post holiday haze? :) This PR is adding ovirt tests to MCO right? But AFAICT openshift/installer#1948 <openshift/installer#1948> which is what adds ovirt to installer isn't merged yet no? So wouldn't this PR depend on #1948 <#1948>? We can't run any tests until installer can install ovirt... no? Looking closely, not sure why installer e2e would depend on mco e2e (via #4340 <#4340> ) Can you clarify so I understand? — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#6533?email_source=notifications&email_token=ABGBYHG4F6HG6H2GFCHWUHLQ4PA2RA5CNFSM4KBVQEH2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEIHGK3Y#issuecomment-571368815>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/ABGBYHBAIEL7Y3NDNHVD6JTQ4PA2RANCNFSM4KBVQEHQ> .

[rgolangh]

/retest

[rgolangh]

@kikisdeliveryservice #4340 is merged

[runcom]

/approve
/lgtm

[openshift-ci-robot]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: rgolangh, runcom

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details

Needs approval from an approver in each of these files:

• ci-operator/jobs/openshift/machine-config-operator/OWNERS [runcom]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[rgolangh]

/hold cancel

[openshift-ci-robot]

@rgolangh: Updated the job-config-master configmap in namespace ci at cluster default using the following files:

• key openshift-machine-config-operator-master-presubmits.yaml using file ci-operator/jobs/openshift/machine-config-operator/openshift-machine-config-operator-master-presubmits.yaml

Details

In response to this:

Depends-on: #4340

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.