-
Notifications
You must be signed in to change notification settings - Fork 8
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
Fixes #33 #43 : Create ps4-controller role #39
Conversation
first error :
+ error fixed when git is installed with `sudo apt install git` |
Error :
+ error fixed, 'est' variable should be `dest: /home/pi/ds4drv` |
+ Fixes : added 'virtualenv_command: /usr/bin/python3 -m venv' |
~ Not really fixed but this error can be because missing a / in chdir |
~ Not fixed : OHHHH yess , it search to install setup.py trought pythons modules, or setup.py is a local file
might be because i've removed this line : `extra_args: '-e'`
+ Fixed in d817964 commit |
This role seen good, except the |
No working on a raspberry pi you need at least on this branch a default environment file in pi@raspberrypi:~ $ sudo systemctl status ds4drv.service
● ds4drv.service - ds4drv daemon
Loaded: loaded (/etc/systemd/system/ds4drv.service; enabled; vendor preset: enabled)
Active: failed (Result: resources)
Apr 17 10:40:50 raspberrypi systemd[1]: ds4drv.service: Failed to load environment files: No such file or directory
Apr 17 10:40:50 raspberrypi systemd[1]: ds4drv.service: Failed to run 'start' task: No such file or directory
Apr 17 10:40:50 raspberrypi systemd[1]: ds4drv.service: Failed with result 'resources'.
Apr 17 10:40:50 raspberrypi systemd[1]: Failed to start ds4drv daemon.
pi@raspberrypi:~ $ cat /etc/systemd/system/ds4drv.service
[Unit]
Description=ds4drv daemon
Requires=bluetooth.service
After=bluetooth.service
[Service]
EnvironmentFile=/etc/donkeycar/ds4drv.env
ExecStart=sudo /home/pi/env/bin/ds4drv --controller-limit 1 --led 00CED1
Restart=on-abort
[Install]
WantedBy=bluetooth.targetpi@raspberrypi:~ $ cat /etc/donkeycar/ds4drv.env
cat: /etc/donkeycar/ds4drv.env: No such file or directory |
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.
A lot of small bugs, biggest is the missing environment file for the service.
Also don't forget to rebase before submitting for review (change were made yesterday) |
bfe1413
to
1dd44bf
Compare
462f507
to
c2c3ddf
Compare
c2c3ddf
to
6a8052d
Compare
d6e62e5
to
6a8052d
Compare
6a8052d
to
b35b11e
Compare
b35b11e
to
d059c31
Compare
d059c31
to
b35b11e
Compare
My tests :
More details :
|
My tests :
More details :
|
For dhcpd issue we can also increase the timeout : The issue might be related to first-boot only as the raspberry pi tries to do a lot of stuff that might slow down dhcpd. |
My path file for dhcpd service on my personal raspberry pi 3 B+
The content of the service :[Unit]
Description=dhcpcd on all interfaces
Wants=network.target
Before=network.target
[Service]his i
Type=forking
PIDFile=/run/dhcpcd.pid
ExecStart=/usr/lib/dhcpcd5/dhcpcd -q -b
ExecStop=/sbin/dhcpcd -x
[Install]
WantedBy=multi-user.target
Alias=dhcpcd5.service The content of the service if we want to restart the service on failure and wait his ip( I hope ) [Unit]
Description=dhcpcd on all interfaces
Wants=network.target
Before=network.target
Restart=always
[Service]his i
Type=forking
PIDFile=/run/dhcpcd.pid
ExecStart=/usr/lib/dhcpcd5/dhcpcd -w -q -t 0 %I
ExecStop=/sbin/dhcpcd -x
[Install]
WantedBy=multi-user.target
Alias=dhcpcd5.service
Fixed in c617978 commit |
build with commits passed !! |
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.
I'll make the small changes and start a build, so I can test it tonight.
Test report
Last 2 errors aren't directly related to the issue we can resolve them on #44 More detailspython manage.py drive --js issueI noticed 2 errors :
|
I'm approving this PR, errors will be fixed in #46 |
Fixes #33 #43