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

Jacques/docker compose docs #246

Merged
merged 10 commits into from
Sep 16, 2024
Merged

Jacques/docker compose docs #246

merged 10 commits into from
Sep 16, 2024

Conversation

jverre
Copy link
Collaborator

@jverre jverre commented Sep 15, 2024

Details

Updated the Opik documentation with the new self-hosting instructions using the docker-compose approach rather than the Opik installer.

Before this PR is merged, we will need to merge the docker compose PR and the opik login PR.

@jverre jverre requested a review from a team as a code owner September 15, 2024 17:40
@liyaka
Copy link
Contributor

liyaka commented Sep 15, 2024

@jverre please see the changes in my PR for docker-compose - moved to separate folder and added README

@jverre
Copy link
Collaborator Author

jverre commented Sep 15, 2024

Thanks @liyaka, yep once the other PR is merged I'll update the doc

Copy link
Collaborator

@andrescrz andrescrz left a comment

Choose a reason for hiding this comment

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

Please see overlapping changes.

docker-compose.yaml Outdated Show resolved Hide resolved
nginx_default_local.conf Outdated Show resolved Hide resolved
sdks/python/src/opik/cli.py Show resolved Hide resolved
@jverre jverre merged commit 46012fa into main Sep 16, 2024
@jverre jverre deleted the jacques/docker-compose-docs branch September 16, 2024 19:19
dsblank pushed a commit that referenced this pull request Oct 4, 2024
* add docker compose and simple nginx confi for it

* Updated Opik documentation with new deployment options

* [NA] Add info logs (#242)

* [OPIK-91] [UX improvements] Implement the Experiment configuration section in the compare page (#252)

* Update docs

* Removed images

* Update to self-host documentation

---------

Co-authored-by: liyaka <[email protected]>
Co-authored-by: Andres Cruz <[email protected]>
Co-authored-by: Thiago dos Santos Hora <[email protected]>
Co-authored-by: andrii.dudar <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants