Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add CI/CD for tenzir/zeek Docker image #1

Merged
merged 1 commit into from
Aug 25, 2021
Merged

Conversation

dominiklohmann
Copy link
Member

@dominiklohmann dominiklohmann commented Aug 19, 2021

This PR adds the missing CI pieces and automation to publish to Dockerhub.

@dominiklohmann dominiklohmann added the feature New functionality label Aug 19, 2021
@dominiklohmann dominiklohmann requested a review from a team August 19, 2021 11:05
@dominiklohmann dominiklohmann force-pushed the story/ch27482/ci branch 14 times, most recently from e83a33d to 1f6263a Compare August 19, 2021 14:51
@dominiklohmann dominiklohmann force-pushed the story/ch27482/ci branch 2 times, most recently from e04799b to d0ccc36 Compare August 19, 2021 15:23
@mavam
Copy link
Member

mavam commented Aug 19, 2021

This now fails with the following error:

Installing "zeek/j-gras/zeek-af_packet-plugin".error: incomplete installation, the follow packages failed to be installed:
#10 15.60   zeek/j-gras/zeek-af_packet-plugin (2.1.2)
#10 15.60 
#10 15.60 Failed installing "zeek/j-gras/zeek-af_packet-plugin": package build_command failed, see log in /opt/zeek/var/lib/zkg/logs/zeek-af_packet-plugin-build.log
#10 15.62 === STDERR ===
#10 15.62 CMake Error at CMakeLists.txt:22 (message):
#10 15.62   Kernel headers not found.
#10 15.62 
#10 15.62 
#10 15.62 === STDOUT ===

How come your fix to install linux-headers-generic has no effect?

@dominiklohmann dominiklohmann force-pushed the story/ch27482/ci branch 7 times, most recently from 155b593 to 4cdc94e Compare August 20, 2021 11:00
@dominiklohmann dominiklohmann force-pushed the story/ch27482/ci branch 2 times, most recently from fe302d9 to 31882c0 Compare August 25, 2021 11:01
@dominiklohmann dominiklohmann force-pushed the story/ch27482/ci branch 8 times, most recently from 13e8530 to 828bb13 Compare August 25, 2021 12:33
Copy link
Member

@mavam mavam left a comment

Choose a reason for hiding this comment

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

Good to see you got it finally working!

@dominiklohmann dominiklohmann merged commit 151c4ac into master Aug 25, 2021
@dominiklohmann dominiklohmann deleted the story/ch27482/ci branch August 25, 2021 14:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature New functionality
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants