Skip to content
This repository has been archived by the owner on May 6, 2022. It is now read-only.

Bump bundler version for jekyll and set fix version for jekyll #2794

Merged
merged 1 commit into from
May 8, 2020

Conversation

mszostok
Copy link
Contributor

@mszostok mszostok commented May 8, 2020

Problem desc

Currently, our pipelines are failing at make docs step, see: https://travis-ci.org/github/kubernetes-sigs/service-catalog/jobs/684782097.

The reason is that we are using jekyll/jekyllwith the latest tag and in the newer version the bundler was updated to 2.1.4.

Updating builder to 2.1.4 failed on netlify.com: https://app.netlify.com/sites/svc-cat/deploys/5eb5a7188c79b800070db33f

8:38:48 PM: Warning: the running version of Bundler (2.0.1) is older than the version that created the lockfile (2.1.4). We suggest you upgrade to the latest version of Bundler by running `gem install bundler`.

Unfortunately, I do not have access to Paul Morie's team/svc-cat so I just selected the 2.0.1 as a fixed version.

Changes proposed in this PR:

  • regenerate the gemfile.lock with bundler version 2.0.1 which satisfy the newest Jekyll and netlify.com
  • change docker image from jekyll/jekyll to jekyll/jekyll:4.0, so the fixed version is used instead of the latest tag (4.0 is the latest release)

Steps used to fix:

  • rm Gemfile.lock
  • bundle install

@k8s-ci-robot k8s-ci-robot added cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. size/M Denotes a PR that changes 30-99 lines, ignoring generated files. labels May 8, 2020
@k8s-ci-robot k8s-ci-robot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label May 8, 2020
@mszostok
Copy link
Contributor Author

mszostok commented May 8, 2020

/test pull-service-catalog-test-integration

1 similar comment
@mszostok
Copy link
Contributor Author

mszostok commented May 8, 2020

/test pull-service-catalog-test-integration

@MHBauer
Copy link
Contributor

MHBauer commented May 8, 2020

/approve

@k8s-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: MHBauer, mszostok

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

@MHBauer
Copy link
Contributor

MHBauer commented May 8, 2020

lol
/lgtm

@k8s-ci-robot k8s-ci-robot added the lgtm Indicates that a PR is ready to be merged. label May 8, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
approved Indicates a PR has been approved by an approver from all required OWNERS files. cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. lgtm Indicates that a PR is ready to be merged. size/M Denotes a PR that changes 30-99 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants