Skip to content

Conversation

@myii
Copy link
Contributor

@myii myii commented Mar 23, 2019

  • kitchen-salt 0.0.21 being installed instead of current latest 0.5.0
    • Failing builds since pillars_from_files is not recognised
    • Deprecated pillars-from-files is working but not the right solution
  • Reason: test-kitchen 2.0.0 was released 3 days ago on March 20, 2019
    • kitchen-salt 0.5.0 depends on test-kitchen ~> 1.4
    • kitchen-salt 0.0.21 depends on test-kitchen >= 0, so gets pulled
  • Remove test-kitchen since it is pulled automatically by the other gems
    • Allow the correct version to be installed via. dependency resolution

More detailed explanation on Slack:

Found the problem: Fetching kitchen-salt 0.0.21.

The working CI runs: Fetching kitchen-salt 0.5.0.

Due to this, pillars_from_files is not recognised and reverting to the deprecated pillars-from-files works. But that's not the solution going forwards.

Fixing in the Gemfile...

And here's why it started pulling kitchen-salt 0.0.21 instead of 0.5.0: test-kitchen 2.0.0 was released 3 days ago on March 20, 2019. That version is being pulled by a Gemfile without a stipulated version. But the dependencies of kitchen-salt states test-kitchen ~> 1.4. kitchen-salt 0.0.21 is the "latest" version that "works" with test-kitchen 2.0.0, due to its dependency of test-kitchen >= 0.

* `kitchen-salt 0.0.21` being installed instead of current latest `0.5.0`
  - Failing builds since `pillars_from_files` is not recognised
  - Deprecated `pillars-from-files` is working but not the right solution
* Reason: `test-kitchen 2.0.0` was released 3 days ago on March 20, 2019
  - `kitchen-salt 0.5.0` depends on `test-kitchen ~> 1.4`
  - `kitchen-salt 0.0.21` depends on `test-kitchen >= 0`, so gets pulled
* Remove `test-kitchen` since it is pulled automatically by the other gems
  - Allow the correct version to be installed via. dependency resolution
@myii myii merged commit ae7af65 into saltstack-formulas:master Mar 23, 2019
@myii myii deleted the bug/fix-kitchen-failures branch March 23, 2019 15:45
@saltstack-formulas-travis

🎉 This PR is included in version 1.2.5 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

myii added a commit to myii/systemd-formula that referenced this pull request Mar 24, 2019
* saltstack-formulas/template-formula#81
  - Update `Gemfile` inc. remove `serverspec` gems
* Fix minor issues in config files
myii added a commit to myii/systemd-formula that referenced this pull request Mar 24, 2019
* saltstack-formulas/template-formula#81
  - Update `Gemfile` inc. remove `serverspec` gems
* saltstack-formulas/template-formula#74
  - Replace deprecated `pillars-from-files` with `pillars_from_files`
* Fix minor issues in config files
myii added a commit to myii/systemd-formula that referenced this pull request Mar 24, 2019
* saltstack-formulas/template-formula#81
  - Update `Gemfile` inc. remove `serverspec` gems
* saltstack-formulas/template-formula#74
  - Replace deprecated `pillars-from-files` with `pillars_from_files`
* saltstack-formulas/template-formula#82
  - Use `debian:8` again since `python-systemd` now available
* Fix minor issues in config files
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants