Do not skip acceptance tests if the main Dockerfile is changed.

[pmacik]

Signed-off-by: Pavel Macík [email protected]

Changes

Currently the skip detector misses the main Dockerfile as in case of #1166. This PR:

• Includes the main Dockerfile
• Changes the filter to match files exactly (instead of using .startsWith method as for directories)

The skipping filter for acceptance tests with OpenShift (CI) is covered by openshift/release#29427

Submitter Checklist

As the author of this PR, please check off the items in this checklist:

• Docs
  included if any changes are user facing
• Tests
  included if any functionality added or changed. For bugfixes please include tests that can catch regressions
• All acceptance test scenarios included in the PR which verifies a bugfix or a requested feature reported by a non-member are tagged with @external-feedback tag.
• Follows the commit message standard

[pmacik]

/hold

[codecov]

Codecov Report

Merging #1168 (527e9ad) into master (130d04a) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #1168   +/-   ##
=======================================
  Coverage   58.08%   58.08%           
=======================================
  Files          33       33           
  Lines        2839     2839           
=======================================
  Hits         1649     1649           
  Misses       1029     1029           
  Partials      161      161           

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 130d04a...527e9ad. Read the comment docs.

[pmacik]

/unhold

[baijum]

/lgtm

[baijum]

/approve

[openshift-ci]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: baijum

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

The pull request process is described here

Needs approval from an approver in each of these files:

• OWNERS [baijum]

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

[pmacik]

/retest

[pmacik]

/retest

[pmacik]

/retest

[pmacik]

/retest

[pmacik]

/retest

[pmacik]

/retest

[openshift-ci]

@pmacik: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name	Commit	Details	Required	Rerun command
ci/prow/4.11-acceptance	3e497dc	link	true	/test 4.11-acceptance

Full PR test history. Your PR dashboard.

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. I understand the commands that are listed here.

[openshift-ci]

New changes are detected. LGTM label has been removed.

[pmacik]

Rebased to latest master for the acceptance tests for the latest feature (#1161)