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 debug weviate build and related script to use remote delve #4051

Merged
merged 1 commit into from
Jan 26, 2024

Conversation

reyreaud-l
Copy link
Contributor

What's being changed:

  • Add a new build target in the root Dockerfile to build a debug friendly binary of weaviate
  • Add new docker-compose and debug script to start this new build and connect a remote debugger to it

Review checklist

  • Documentation has been updated, if necessary. Link to changed documentation:
  • Chaos pipeline run or not necessary. Link to pipeline:
  • All new code is covered by tests where it is reasonable.
  • Performance tests have been run or not necessary.

Dockerfile Outdated Show resolved Hide resolved
@weaviate-git-bot
Copy link

Great to see you again! Thanks for the contribution.

beep boop - the Weaviate bot 👋🤖

PS:
Are you already a member of the Weaviate Slack channel?

Copy link

sonarcloud bot commented Jan 22, 2024

Quality Gate Failed Quality Gate failed

Failed conditions

3 Security Hotspots

See analysis details on SonarCloud

@reyreaud-l reyreaud-l merged commit 3f851a6 into master Jan 26, 2024
34 of 35 checks passed
@reyreaud-l reyreaud-l deleted the lre/add-debug-weaviate branch January 26, 2024 09:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants