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

[memgraph-2-17 < ] Debugging memgraph #765

Merged
merged 15 commits into from
May 22, 2024
Merged

Conversation

Josipmrden
Copy link
Contributor

@Josipmrden Josipmrden commented May 14, 2024

Description

Instructions on how to debug Memgraph when in RelWithDebInfo mode with the tools inside the container.

Pull request type

Please check what kind of PR this is:

  • Fix or improvement of an existing page
  • New documentation page, release related

Related PRs and issues

PR this doc page is related to:
@MarkoBarisic tag the PR when you add linux-perf, gdb and procps to the RelWithDebInfo image

Checklist:

  • Check all content with Grammarly
  • Perform a self-review of my code
  • Make corresponding changes to the rest of the documentation (consult with the DX team)
  • The build passes locally
  • My changes generate no new warnings or errors
  • Add a corresponding label
  • If release-related, add a product and version label (update to docs when we start releasing relwithdebinfo images)
  • If release-related, add release note on product PR (update to docs when we start releasing relwithdebinfo images)

@Josipmrden Josipmrden self-assigned this May 14, 2024
@Josipmrden Josipmrden requested a review from kgolubic as a code owner May 14, 2024 13:10
Copy link

vercel bot commented May 14, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
documentation ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 22, 2024 11:08am

@Josipmrden
Copy link
Contributor Author

@kgolubic can you check why my image at the bottom does not show in the dev environment?

@Josipmrden Josipmrden added priority: medium (missing info) An additional information can be helpful or interesting, but the absence is not disruptive customer labels May 14, 2024
@Josipmrden
Copy link
Contributor Author

Josipmrden commented May 14, 2024

@kgolubic need 2 more links and then I'm done, i've put them in todo in the content

@kgolubic
Copy link
Contributor

kgolubic commented May 15, 2024

@kgolubic can you check why my image at the bottom does not show in the dev environment?

The images need to go to /public/pages/ folder, not directly in the folder where the page is located. I've moved the image to the right location.

@Josipmrden, once you are done, I'll take over this this PR. There are some additional things that need to be done:

@kgolubic kgolubic changed the base branch from main to memgraph-2-17 May 16, 2024 14:54
@kgolubic kgolubic changed the base branch from memgraph-2-17 to main May 16, 2024 14:58
@kgolubic kgolubic changed the base branch from main to memgraph-2-17 May 22, 2024 11:00
Copy link
Contributor

@kgolubic kgolubic left a comment

Choose a reason for hiding this comment

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

Everything looks OK.

@kgolubic kgolubic merged commit b3d6414 into memgraph-2-17 May 22, 2024
2 checks passed
@kgolubic kgolubic deleted the debugging-memgraph branch May 22, 2024 11:13
kgolubic added a commit that referenced this pull request May 23, 2024
* Memgraph 2.17 docs

* Docs

* Zoned datetime data type (#723)

* Document how zoned datetime is supported

* Fix formatting

* Update the Differences in Cypher implementation page

* Update min/max functions

* Revert "Zoned datetime data type (#723)" (#763)

This reverts commit 16c8157.

* [Memgraph 2.17 < ]Update Kubernetes docs  (#773)

* Init docs for kubernetes.

* Fixes.

* Update page.

* Update.

* Fix.

* Update pages/getting-started/install-memgraph/kubernetes.mdx

Co-authored-by: Katarina Supe <61758502+katarinasupe@users.noreply.github.com>

* Update kubernetes.mdx

---------

Co-authored-by: Katarina Supe <61758502+katarinasupe@users.noreply.github.com>
Co-authored-by: Kruno Golubic <46486712+kgolubic@users.noreply.github.com>

* [memgraph-2.17 < ] --restore-replication-state-startup default to true (#762)

* Replication changes

* Update config docs

---------

Co-authored-by: kgolubic <kgolubic@gmail.com>

* Memgrpah 2.17 release notes (#679)

* Initial commit

* Update release notes

* Update release notes

* Update styling

* Fix typo

* Update RN

* Update release notes

* Update RN

* Update Release notes

* Add HA changes for 2.17 (#749)

* add '#1932 discard request leadership on follower' changes from megraph

* add user other user actions from '#1981' PR

* add part for env flags

---------

Co-authored-by: kgolubic <kgolubic@gmail.com>

* new: Add Lab 2.14 release notes (#779)

Co-authored-by: Kruno Golubic <46486712+kgolubic@users.noreply.github.com>

* Fix the Cypher examples for ZonedDateTime (#782)

* Create if not exists 2.17 (#780)

* Added description for if not exists for users and roles

* Added changes to description for if not exists for users and roles

* Update role-based-access-control.mdx

* Update users.mdx

---------

Co-authored-by: Kruno Golubic <46486712+kgolubic@users.noreply.github.com>

* Update release notes

* Update RN

* [memgraph-2-17 < ] Debugging memgraph (#765)

* Debugging memgraph

* Updated documentation

* Add Docker Compose

* Add one more todo

* Grammarly

* Fix image path

* Move to new location

* Update help-center.md

* Fix typo (#783)

* Debugging image

* Remove TODO mark

---------

Co-authored-by: kgolubic <kgolubic@gmail.com>
Co-authored-by: Kruno Golubic <46486712+kgolubic@users.noreply.github.com>

* Update release notes

* Update direct download  links

* Bolt routing (#777)

* Update RN

* Update RN

* Add bolt server to the output (#728)

* Update RN

* Update RN

* Data recovery on startup (#793)

* Add flags

* Update flags and remove callout

* Add callout

* Update RN

* Update release notes

* Update values for Vertex (#791)

* Update

* Update Delta values

* Fix typo

* Update code

* Update values

* Update one

* Update calculations

---------

Co-authored-by: Ante Pušić <ante.pusic@memgraph.io>
Co-authored-by: Ante Javor <ante.javor@memgraph.io>
Co-authored-by: Katarina Supe <61758502+katarinasupe@users.noreply.github.com>
Co-authored-by: Andi <andi8647@gmail.com>
Co-authored-by: Antonio Filipovic <61245998+antoniofilipovic@users.noreply.github.com>
Co-authored-by: Toni <toni.lastre@memgraph.io>
Co-authored-by: tonijurjevic96 <168409767+tonijurjevic96@users.noreply.github.com>
Co-authored-by: Josipmrden <josip.mrden@memgraph.io>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
customer priority: medium (missing info) An additional information can be helpful or interesting, but the absence is not disruptive
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants