search

puppetlabs / influxdb

2 stars 22 forks source link

Add Debian 11 support #103

Closed kenyon closed 11 months ago

kenyon commented 1 year ago

Also includes #102.

bastelfreak commented 1 year ago

can you remove the RedHat mocking in https://github.com/puppetlabs/influxdb/blob/main/spec/classes/init_spec.rb#L4 ?

kenyon commented 1 year ago

I saw that and thought it looked wrong too. Removed.

kenyon commented 1 year ago

@bastelfreak maybe there is a better way to handle the different operating systems in the spec tests, but this at least runs tests with the facts for each OS now!

Easiest to review one commit at a time.

kenyon commented 1 year ago

That AlmaLinux failure looks unrelated, let's try running the tests again...

MartyEwings commented 11 months ago

@m0dular we good to merge this?

m0dular commented 11 months ago

I meant to only rerun the Alma test, but accidentally kicked them all off again. I think we should be good to merge this regardless of that test, though.