Skip to content
This repository has been archived by the owner on Dec 15, 2020. It is now read-only.

Update test Docker container versions #2239

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Update test Docker container versions #2239

wants to merge 1 commit into from

Conversation

zwass
Copy link
Contributor

@zwass zwass commented May 25, 2020

Previously these test Docker containers used
https://github.com/kolide/docker-osquery, which no longer seems to be
maintained and has very old osquery versions (2.7.0). Migrate to using
containers from the new https://github.com/dactivllc/docker-osquery
project. This brings Fleet to testing with current (4.3.0) osquery on an
expanded set of operating systems.

Previously these test Docker containers used
https://github.com/kolide/docker-osquery, which no longer seems to be
maintained and has very old osquery versions (2.7.0). Migrate to using
containers from the new https://github.com/dactivllc/docker-osquery
project. This brings Fleet to testing with current (4.3.0) osquery on an
expanded set of operating systems.
@@ -3,7 +3,7 @@ version: '2'

services:
ubuntu14-osquery:
image: "kolide/osquery:${KOLIDE_OSQUERY_VERSION}"
image: "dactiv/osquery:4.3.0-ubuntu14.04"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Given that this is in the kolide org, it may make more sense to fix kolide/osquery? Thoughts

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If you would like to maintain kolide/osquery, that's fine with me. The important bit IMO is that we are developing with new versions of osquery.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@directionless do you think Kolide can take on maintaining kolide/osquery?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe. Want to help? I can give you permission on both the git side, and the docker side...

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants