Run builds in travis on xenial

sensu / sensu-go-chef

Chef Library Cookbook for Sensu Go

https://sensu.io

MIT License

11 stars 22 forks source link

Run builds in travis on xenial #75

Closed webframp closed 4 years ago

webframp commented 4 years ago

Switching to xenial since it's a newer base and has support for the addons needed to run chef integration testing.

bionic doesn't yet appear to be added to the approved list of resources for the apt addon:

https://github.com/travis-ci/apt-source-safelist/tree/master/keys

Additionally switched package from chefdk to chef-workstation to stay up to date with what's maintained. Same tools, different name.

Purpose

Update the base platform used for travis builds.

Known Compatibility Issues

None

webframp commented 4 years ago

Saw failures like this: https://travis-ci.org/sensu/sensu-go-chef/jobs/624832661

Small attempt to fix these

webframp commented 4 years ago

@majormoses Don't know why appveyor started failing here

majormoses commented 4 years ago

is rspec not in chef-workstation? https://travis-ci.org/sensu/sensu-go-chef/jobs/624854826#L257

webframp commented 4 years ago

is rspec not in chef-workstation? https://travis-ci.org/sensu/sensu-go-chef/jobs/624854826#L257

My fault, just used wrong bin path

webframp commented 4 years ago

@cwjohnston or @derekgroh maybe can comment on windows failures here. I don't think I can do much about those.

derekgroh commented 4 years ago

Thinking a bit more on #59 and this change, could we move testing to Travis.ci for both platforms? Anyone have experience with breaking down the logic for Travis.ci now that Windows support is more supported?

webframp commented 4 years ago

@derekgroh I'm open to that option and doing some of that work if it's agreeable.

Think it's reasonable we do that instead of merging this as is and just ignoring appveyor failures?