-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
[docker-base]: Build supervisor package from source in order to update to v3.3.2 #777
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,28 @@ | ||
; supervisor config file | ||
|
||
[unix_http_server] | ||
file=/var/run/supervisor.sock ; (the path to the socket file) | ||
chmod=0700 ; sockef file mode (default 0700) | ||
|
||
[supervisord] | ||
logfile=/var/log/supervisor/supervisord.log ; (main log file;default $CWD/supervisord.log) | ||
pidfile=/var/run/supervisord.pid ; (supervisord pidfile;default supervisord.pid) | ||
childlogdir=/var/log/supervisor ; ('AUTO' child log dir, default $TEMP) | ||
|
||
; the below section must remain in the config file for RPC | ||
; (supervisorctl/web interface) to work, additional interfaces may be | ||
; added by defining them in separate rpcinterface: sections | ||
[rpcinterface:supervisor] | ||
supervisor.rpcinterface_factory = supervisor.rpcinterface:make_main_rpcinterface | ||
|
||
[supervisorctl] | ||
serverurl=unix:///var/run/supervisor.sock ; use a unix:// URL for a unix socket | ||
|
||
; The [include] section can just contain the "files" setting. This | ||
; setting can list multiple files (separated by whitespace or | ||
; newlines). It can also contain wildcards. The filenames are | ||
; interpreted as relative to this file. Included files *cannot* | ||
; include files themselves. | ||
|
||
[include] | ||
files = /etc/supervisor/conf.d/*.conf |
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -2,6 +2,7 @@ | |
|
||
DOCKER_BASE = docker-base.gz | ||
$(DOCKER_BASE)_PATH = $(DOCKERS_PATH)/docker-base | ||
$(DOCKER_BASE)_DEPENDS += $(SUPERVISOR) | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggest not include built packages into docker-base. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. it is used by every docker so I think it makes sense to be included in the docker-base. |
||
|
||
ifeq ($(SONIC_CONFIG_DEBUG),y) | ||
GDB = gdb | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,5 @@ | ||
# supervisor package | ||
|
||
SUPERVISOR = python-supervisor_3.3.2-1_all.deb | ||
$(SUPERVISOR)_SRC_PATH = $(SRC_PATH)/supervisor | ||
SONIC_MAKE_DEBS += $(SUPERVISOR) |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,22 @@ | ||
.ONESHELL: | ||
SHELL = /bin/bash | ||
.SHELLFLAGS += -e | ||
|
||
MAIN_TARGET = python-supervisor_3.3.2-1_all.deb | ||
|
||
$(addprefix $(DEST)/, $(MAIN_TARGET)): $(DEST)/% : | ||
# Remove any stale files | ||
rm -rf ./supervisor | ||
|
||
# Clone isc-dhcp repo | ||
git clone git://github.com/Supervisor/supervisor.git | ||
pushd ./supervisor | ||
git checkout -f 3.3.2 | ||
|
||
# Build Python and Debian package | ||
python setup.py --command-packages=stdeb.command bdist_deb | ||
popd | ||
|
||
# Move the newly-built .deb packages to the destination directory | ||
mv ./supervisor/deb_dist/$* $(DEST)/ | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
These packages should be defined as supervisor's runtime dependencies, following other sonic component Makefile. #Closed
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
use _RDEPENDS
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"_RDEPENDS" is only used for installing runtime dependencies that we build, correct? The packages I am installing here are standard packages in the Debian repo.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
in line 58, you have apt-get -y install -f that help to fix the dependency, so you probably do notneed line 38.
In reply to: 125733195 [](ancestors = 125733195)