New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Install the latest filesystem creation packages #3904

Open
wants to merge 1 commit into
base: master
from

Conversation

Projects
None yet
4 participants
@earlruby
Copy link
Contributor

earlruby commented Dec 17, 2018

This PR ensures that the e2fsprogs and xfsprogs packages are installed on all Kubernetes nodes and that the packages are the latest versions. It also ensures that the nodes can create XFS filesystems when necessary, since not all distros install xfsprogs by default.

  • e2fsprogs - ext2/ext3/ext4 file system utilities
  • xfsprogs - Utilities for managing the XFS filesystem
Install the latest filesystem creation packages
This PR ensures that the e2fsprogs and xfsprogs packages are
installed on all Kubernetes nodes and that the packages are
the latest versions. It also ensures that the nodes can
create XFS filesystems when necessary, since not all distros
install xfsprogs by default.

e2fsprogs - ext2/ext3/ext4 file system utilities
xfsprogs - Utilities for managing the XFS filesystem
@k8s-ci-robot

This comment has been minimized.

Copy link
Contributor

k8s-ci-robot commented Dec 17, 2018

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: earlruby
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: ant31

If they are not already assigned, you can assign the PR to them by writing /assign @ant31 in a comment when ready.

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

@k8s-ci-robot k8s-ci-robot requested review from chadswen and mirwan Dec 17, 2018

@earlruby

This comment has been minimized.

Copy link
Contributor

earlruby commented Dec 17, 2018

I installed Rook on my Kubernetes cluster and made it the default provider of block storage. I then reinstalled Prometheus via Helm so that Prometheus would use Rook for persistent storage. This failed since the Prometheus Helm Chart requests an XFS file system, and Kubespray nodes don't install xfsprogs.

Since any Helm chart that uses a persistence provider that runs on the nodes (e.g. the many variants of Rook, local storage, others) may request ext3, ext4, or XFS filesystems, it seems like a good idea to make sure that the e2fsprogs and xfsprogs packages are always installed.

@earlruby

This comment has been minimized.

Copy link
Contributor

earlruby commented Dec 17, 2018

/assign @ant31

@earlruby

This comment has been minimized.

Copy link
Contributor

earlruby commented Jan 11, 2019

/assign @chadswen

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment