Skip to content

doc(hubble): add docker-compose to start with server#522

Merged
simon824 merged 2 commits intoapache:masterfrom
aroundabout:doc
Oct 8, 2023
Merged

doc(hubble): add docker-compose to start with server#522
simon824 merged 2 commits intoapache:masterfrom
aroundabout:doc

Conversation

@aroundabout
Copy link
Contributor

Purpose of the PR

  • update the way to quick start server with hubble

Main Changes

  • update the way to quick start server with hubble

Verifying these changes

  • Trivial rework / code cleanup without any test coverage. (No Need)
  • Already covered by existing tests, such as (please modify tests here).
  • Need tests and can be verified as follows:
    • xxx

Does this PR potentially affect the following parts?

  • Nope
  • Dependencies (add/update license info)
  • Modify configurations
  • The public API
  • Other affects (typed here)

Documentation Status

  • Doc - TODO
  • Doc - Done
  • Doc - No Need

@github-actions github-actions bot added the hubble hugegraph-hubble label Sep 19, 2023
Copy link
Member

@imbajin imbajin left a comment

Choose a reason for hiding this comment

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

Also update the instrinstruction in root (enhance the desc)

image

- 8088:8088
```

Then we should follow the [hubble doc](https://hugegraph.apache.org/cn/docs/quickstart/hugegraph-hubble/#3%E5%B9%B3%E5%8F%B0%E4%BD%BF%E7%94%A8%E6%B5%81%E7%A8%8B) to create the graph.
Copy link
Member

Choose a reason for hiding this comment

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

use en doc link to avoid the long encode

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done

```
version: '3'
services:
graph_hubble:
Copy link
Member

@imbajin imbajin Sep 19, 2023

Choose a reason for hiding this comment

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

graph_hubble seems a little confused? (use server?)

and shall we add the preload param here (comment it or not by default)

Copy link
Member

@imbajin imbajin left a comment

Choose a reason for hiding this comment

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

lgtm

@simon824 simon824 merged commit d7733db into apache:master Oct 8, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

hubble hugegraph-hubble

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants