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

Building tendrl-node-agent package from master branch is broken #634

Closed
dahorak opened this issue Oct 2, 2017 · 2 comments
Closed

Building tendrl-node-agent package from master branch is broken #634

dahorak opened this issue Oct 2, 2017 · 2 comments

Comments

@dahorak
Copy link

dahorak commented Oct 2, 2017

Building tendrl-node-agent package from master branch is broken since Sep 28th[1].
Build log shows following SyntaxError traceback[2]:

Compiling /builddir/build/BUILDROOT/tendrl-node-agent-1.5.2-20170928T124317.1822bc6.x86_64/usr/lib/python2.7/site-packages/tendrl/node_agent/monitoring/collectd/collectors/gluster/tendrl_gluster_brick_disk_stats.py ...
  File "/usr/lib/python2.7/site-packages/tendrl/node_agent/monitoring/collectd/collectors/gluster/tendrl_gluster_brick_disk_stats.py", line 89
    if etcd_ca_cert_file and str(etcd_ca_cert_file) != ""
                                                        ^
SyntaxError: invalid syntax

Which seems to be related to commit 1822bc6.

See also Tendrl/commons/issues/744.

[1] https://ci.centos.org/view/tendrl-build-master/job/tendrl-build-1-master-node-agent/
[2] https://copr-be.cloud.fedoraproject.org/results/tendrl/tendrl/epel-7-x86_64/00609078-tendrl-node-agent/build.log.gz

@rishubhjain
Copy link
Contributor

@dahorak https://ci.centos.org/view/tendrl-build-master/job/tendrl-build-1-master-node-agent/ The build errors are resolved, Please verify

@dahorak
Copy link
Author

dahorak commented Oct 4, 2017

Works as expected.

@dahorak dahorak closed this as completed Oct 4, 2017
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

No branches or pull requests

2 participants