Skip to content

Conversation

irozzo-1A
Copy link
Contributor

@irozzo-1A irozzo-1A commented Sep 25, 2020

What this PR does / why we need it:
Backport of #826 (plus #822 and #794 to fix e2e tests)
Which issue(s) this PR fixes (optional, in fixes #<issue number> format, will close the issue(s) when PR gets merged):
Fixes #

Special notes for your reviewer:

Optional Release Note:

Fix docker installation for CentOS/RHEL 8 
…ermatic#826) * Fix releasever to 7 for RHEL and CentOS in yum docker repository * Fix e2e tests
@kubermatic-bot kubermatic-bot added release-note Denotes a PR that will be considered when it comes time to generate release notes. dco-signoff: yes Denotes that all commits in the pull request have the valid DCO signoff message. sig/cluster-management Denotes a PR or issue as being assigned to SIG Cluster Management. labels Sep 25, 2020
@kubermatic-bot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: irozzo-1A

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:

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

@kubermatic-bot kubermatic-bot added approved Indicates a PR has been approved by an approver from all required OWNERS files. size/M Denotes a PR that changes 30-99 lines, ignoring generated files. labels Sep 25, 2020
@irozzo-1A irozzo-1A changed the title Fix releasever to 7 for RHEL and CentOS in yum docker repository (#826) [v1.14] Fix releasever to 7 for RHEL and CentOS in yum docker repository (#826) Sep 25, 2020
@xrstf
Copy link
Contributor

xrstf commented Sep 25, 2020

/retest

@kubermatic-bot kubermatic-bot added the lgtm Indicates that a PR is ready to be merged. label Sep 25, 2020
@kubermatic-bot
Copy link
Contributor

LGTM label has been added.

Git tree hash: 35c955723a7dec5cf59479995ade095e1c453dec

* Disable CoreOS E2E tests on DigitalOcean * Disable CoreOS E2E tests on GCE
@kubermatic-bot kubermatic-bot removed the lgtm Indicates that a PR is ready to be merged. label Sep 25, 2020
Signed-off-by: Moath Qasim <moad.qassem@gmail.com> Signed-off-by: Moath Qasim <moad.qassem@gmail.com>
Copy link
Member

@kron4eg kron4eg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/retest

@kubermatic-bot kubermatic-bot added the lgtm Indicates that a PR is ready to be merged. label Sep 25, 2020
@kubermatic-bot
Copy link
Contributor

LGTM label has been added.

Git tree hash: 5167955126707940d42e702804b4beeaa9c61590

@kubermatic-bot kubermatic-bot merged commit c06f2f6 into kubermatic:release/v1.14 Sep 25, 2020
@irozzo-1A irozzo-1A deleted the update-yum-docker-version-v1.14 branch September 25, 2020 14:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. dco-signoff: yes Denotes that all commits in the pull request have the valid DCO signoff message. lgtm Indicates that a PR is ready to be merged. release-note Denotes a PR that will be considered when it comes time to generate release notes. sig/cluster-management Denotes a PR or issue as being assigned to SIG Cluster Management. size/M Denotes a PR that changes 30-99 lines, ignoring generated files.

6 participants