TheHive and Cortex Compose scripts #2652
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
rdettai
changed the title
rdettai
changed the title
rdettai
force-pushed
the
story/sc-37376/thehive-init-revamp
branch
from
c71d775 to
4b4ce3b
Compare
This almost works, however the container that runs the actual query (tenzir/vast-neuron) doesn't find the `input.json` in the expected place (`/job/input/input.json`). I checked that the `input.json` is correctly creatd in the cortex container with proper contents, and when the container is started an actual log message indicates that that very volume is mounted to `/job` on the neuron. ``` thehive-cortex-1 | timeout: 30 minutes thehive-cortex-1 | image : tenzir/vast-neuron thehive-cortex-1 | volume : /opt/cortex/jobs/cortex-job-yfueFYQBc4_qCE3nWixQ-7660690682055716688:/job ``` However, when logging into the neuron /job is empty. I've run out of ideas. It does seem to work for Kaans set of images, maybe some sort of permission problem.
It works when running the neuron independently,
("./vast-cortex-neuron/tests/test service")
but fails when it is executed from Cortex
("Container could not be started")
dispanser
force-pushed
the
story/sc-37376/thehive-init-revamp
branch
from
9412983 to
f6be72a
Compare
rdettai
commented
Nov 7, 2022
|
It will be easier to write some high level tests and documentation once the full usecase is implemented, so I think we can merge this now. |
dispanser
approved these changes
Nov 8, 2022
2 tasks
dominiklohmann
added a commit
that referenced
this pull request
Nov 25, 2022
This is addressing multiple issues that bubbled up after #2652 - There was a `$` missing before `exit_code` - Lint issues - A transient failure in the Cortex init script due to concurrent access to a mounted volume
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This is a complete revamp of the #2532 with a more stable init script.
📝 Reviewer Checklist
Review this pull request by ensuring the following items:
Todo:
replace pyvast with the new bindingsTo be done separately to close this off