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

livepeer-log: remove livepeer-log #695

Closed

Conversation

iameli-streams
Copy link
Contributor

@iameli-streams iameli-streams commented Oct 12, 2023

Depends on livepeer/mistserver#2.

One commit just removes livepeer-log and all of its dependencies. The other handles livepeer-victoria-metrics -j and livepeer-vmagent -j - previously livepeer-log injected their usage information but now we make use of a Bash wrapper script (at scripts/livepeer-vmagent) to handle -j and then invoke the vmagent binary itself, now present at ./bin/lp-vmagent.

@iameli-streams iameli-streams changed the title victoria-metrics: migrate to dedicated ./scripts livepeer-log: remove livepeer-log Oct 12, 2023
@iameli-streams iameli-streams force-pushed the eli/remove-livepeer-log branch from aa9e431 to 190dda1 Compare October 12, 2023 00:41
@iameli-streams iameli-streams marked this pull request as ready for review October 12, 2023 00:41
@@ -70,6 +70,8 @@ RUN apt update && apt install -yqq \
"$(if [ "$BUILD_TARGET" != "stripped" ]; then echo "gdb"; fi)" \
&& rm -rf /var/lib/apt/lists/*

# Most of ./scripts is for livepeer-in-a-box except livepeer-vmagent, which is used in production in Catalyst proper
ADD ./scripts/livepeer-vmagent /usr/local/bin
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Note to self: need to add this script to the uploaded bundle too...

@iameli iameli mentioned this pull request Oct 18, 2023
@iameli
Copy link
Contributor

iameli commented Oct 18, 2023

Closing in favor of #707

@iameli iameli closed this Oct 18, 2023
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.

3 participants