<tt><font size=2>Ruslan Kamaldinov <rkamaldinov@mirantis.com> wrote
on 08/24/2014 12:30:04 PM:<br>
<br>
> From: Ruslan Kamaldinov <rkamaldinov@mirantis.com></font></tt>
<br><tt><font size=2>> To: "OpenStack Development Mailing List
(not for usage questions)" <br>
> <openstack-dev@lists.openstack.org>, </font></tt>
<br><tt><font size=2>> Date: 08/24/2014 12:32 PM</font></tt>
<br><tt><font size=2>> Subject: Re: [openstack-dev] [heat] "heat.conf.sample
is not up to date"</font></tt>
<br><tt><font size=2>> <br>
> On Sun, Aug 24, 2014 at 11:10 AM, Mike Spreitzer <mspreitz@us.ibm.com>
wrote:<br>
> > What I really need to know is what to do when committing a change
that<br>
> > really does require a change in the sample configuration file.
Of course I<br>
> > tried running generate_sample.sh, but `tox -epep8` still complains.
What is<br>
> > the right procedure to get a correct sample committed? BTW,
I am doing the<br>
> > following admittedly risky thing: I run DevStack, and make my
changes in<br>
> > /opt/stack/heat/.<br>
> <br>
> Mike,<br>
> <br>
> It seems that you have oslo.messaging installed from master (default<br>
> behavior of Devstack), while heat.sample.config is built for<br>
> oslo.messaging v 1.3.1. As a quick workaround I'd suggest to downgrade<br>
> oslo.messaging to 1.3.1 in pep8 virtual environment:<br>
> $ cd /opt/stack/heat<br>
> $ source .tox/pep8/bin/activate<br>
> $ pip install oslo.messaging==1.3.1<br>
> $ deactivate<br>
> $ tox -e pep8 # should pass now<br>
</font></tt>
<br><tt><font size=2>In that same VM in which I ran DevStack, I also made
an independent clone of heat in ~/code/heat; my original email quoted a
failure from that clone, hoping that it might be easier to debug (being
a simpler scenario). Below is a typescript showing me try again there,
including a trial of what Mathieu Gagné suggested (which has some kind
of command line syntax error, and did nothing). You will see that
I investigated and found that in this case tox's venv contained oslo.messaging
version 1.3.1, so no adjustment about that was needed. Yet it still
failed.</font></tt>
<br>
<br><tt><font size=2>ubuntu@mjs-dstk-821a:~/code/heat$ rm -rf .tox</font></tt>
<br>
<br><tt><font size=2>ubuntu@mjs-dstk-821a:~/code/heat$ tox -evenv bash
./tools/config/generate_sample.sh -b . -p heat -o etc/heat</font></tt>
<br><tt><font size=2>usage: tox [-h] [--version] [-v] [--showconfig] [-l]
[-c CONFIGFILE]</font></tt>
<br><tt><font size=2> [-e envlist]
[--notest] [--sdistonly] [--installpkg PATH]</font></tt>
<br><tt><font size=2> [--develop]
[--set-home] [-i URL] [-r] [--result-json PATH]</font></tt>
<br><tt><font size=2> [--hashseed
SEED] [--force-dep REQ] [--sitepackages]</font></tt>
<br><tt><font size=2> [--skip-missing-interpreters]</font></tt>
<br><tt><font size=2> [args [args
...]]</font></tt>
<br><tt><font size=2>tox: error: unrecognized arguments: -b . -p heat -o
etc/heat</font></tt>
<br>
<br><tt><font size=2>ubuntu@mjs-dstk-821a:~/code/heat$ tox -epep8</font></tt>
<br><tt><font size=2>pep8 create: /home/ubuntu/code/heat/.tox/pep8</font></tt>
<br><tt><font size=2>pep8 installdeps: -r/home/ubuntu/code/heat/requirements.txt,
-r/home/ubuntu/code/heat/test-requirements.txt</font></tt>
<br><tt><font size=2>pep8 develop-inst: /home/ubuntu/code/heat</font></tt>
<br><tt><font size=2>pep8 runtests: PYTHONHASHSEED='0'</font></tt>
<br><tt><font size=2>pep8 runtests: commands[0] | flake8 heat bin/heat-api
bin/heat-api-cfn bin/heat-api-cloudwatch bin/heat-engine bin/heat-manage
contrib</font></tt>
<br><tt><font size=2>pep8 runtests: commands[1] | /home/ubuntu/code/heat/tools/config/check_uptodate.sh</font></tt>
<br><tt><font size=2>--- /tmp/heat.UnHAZD/heat.conf.sample
2014-08-25 04:06:07.648849999 +0000</font></tt>
<br><tt><font size=2>+++ etc/heat/heat.conf.sample
2014-08-24 06:53:51.540849999 +0000</font></tt>
<br><tt><font size=2>@@ -159,10 +159,6 @@</font></tt>
<br><tt><font size=2> # Size of RPC connection pool. (integer value)</font></tt>
<br><tt><font size=2> #rpc_conn_pool_size=30</font></tt>
<br><tt><font size=2> </font></tt>
<br><tt><font size=2>-# Modules of exceptions that are permitted to be
recreated</font></tt>
<br><tt><font size=2>-# upon receiving exception data from an rpc call.
(list value)</font></tt>
<br><tt><font size=2>-#allowed_rpc_exception_modules=oslo.messaging.exceptions,nova.exception,cinder.exception,exceptions</font></tt>
<br><tt><font size=2>-</font></tt>
<br><tt><font size=2> # Qpid broker hostname. (string value)</font></tt>
<br><tt><font size=2> #qpid_hostname=heat</font></tt>
<br><tt><font size=2> </font></tt>
<br><tt><font size=2>@@ -301,15 +297,6 @@</font></tt>
<br><tt><font size=2> # Heartbeat time-to-live. (integer value)</font></tt>
<br><tt><font size=2> #matchmaker_heartbeat_ttl=600</font></tt>
<br><tt><font size=2> </font></tt>
<br><tt><font size=2>-# Host to locate redis. (string value)</font></tt>
<br><tt><font size=2>-#host=127.0.0.1</font></tt>
<br><tt><font size=2>-</font></tt>
<br><tt><font size=2>-# Use this port to connect to redis host. (integer
value)</font></tt>
<br><tt><font size=2>-#port=6379</font></tt>
<br><tt><font size=2>-</font></tt>
<br><tt><font size=2>-# Password for Redis server (optional). (string value)</font></tt>
<br><tt><font size=2>-#password=<None></font></tt>
<br><tt><font size=2>-</font></tt>
<br><tt><font size=2> # Size of RPC greenthread pool. (integer value)</font></tt>
<br><tt><font size=2> #rpc_thread_pool_size=64</font></tt>
<br><tt><font size=2> </font></tt>
<br><tt><font size=2>@@ -1229,6 +1216,22 @@</font></tt>
<br><tt><font size=2> #hash_algorithms=md5</font></tt>
<br><tt><font size=2> </font></tt>
<br><tt><font size=2> </font></tt>
<br><tt><font size=2>+[matchmaker_redis]</font></tt>
<br><tt><font size=2>+</font></tt>
<br><tt><font size=2>+#</font></tt>
<br><tt><font size=2>+# Options defined in oslo.messaging</font></tt>
<br><tt><font size=2>+#</font></tt>
<br><tt><font size=2>+</font></tt>
<br><tt><font size=2>+# Host to locate redis. (string value)</font></tt>
<br><tt><font size=2>+#host=127.0.0.1</font></tt>
<br><tt><font size=2>+</font></tt>
<br><tt><font size=2>+# Use this port to connect to redis host. (integer
value)</font></tt>
<br><tt><font size=2>+#port=6379</font></tt>
<br><tt><font size=2>+</font></tt>
<br><tt><font size=2>+# Password for Redis server (optional). (string value)</font></tt>
<br><tt><font size=2>+#password=<None></font></tt>
<br><tt><font size=2>+</font></tt>
<br><tt><font size=2>+</font></tt>
<br><tt><font size=2> [matchmaker_ring]</font></tt>
<br><tt><font size=2> </font></tt>
<br><tt><font size=2> #</font></tt>
<br><tt><font size=2>check_uptodate.sh: heat.conf.sample is not up to date.</font></tt>
<br><tt><font size=2>check_uptodate.sh: Please run /home/ubuntu/code/heat/tools/config/generate_sample.sh.</font></tt>
<br><tt><font size=2>ERROR: InvocationError: '/home/ubuntu/code/heat/tools/config/check_uptodate.sh'</font></tt>
<br><tt><font size=2>pep8 runtests: commands[2] | /home/ubuntu/code/heat/tools/requirements_style_check.sh
requirements.txt test-requirements.txt</font></tt>
<br><tt><font size=2>pep8 runtests: commands[3] | bash -c find heat -type
f -regex '.*\.pot?' -print0|xargs -0 -n 1 msgfmt --check-format -o /dev/null</font></tt>
<br><tt><font size=2>_________________
__________________ summary ____
_________________________
_______</font></tt>
<br><tt><font size=2>ERROR: pep8: commands failed</font></tt>
<br>
<br><tt><font size=2>ubuntu@mjs-dstk-821a:~/code/heat$ . .tox/pep8/bin/activate</font></tt>
<br>
<br><tt><font size=2>(pep8)ubuntu@mjs-dstk-821a:~/code/heat$ python</font></tt>
<br><tt><font size=2>Python 2.7.6 (default, Mar 22 2014, 22:59:56) </font></tt>
<br><tt><font size=2>[GCC 4.8.2] on linux2</font></tt>
<br><tt><font size=2>Type "help", "copyright", "credits"
or "license" for more information.</font></tt>
<br><tt><font size=2>>>> system</font></tt>
<br><tt><font size=2>Traceback (most recent call last):</font></tt>
<br><tt><font size=2> File "<stdin>", line 1, in
<module></font></tt>
<br><tt><font size=2>NameError: name 'system' is not defined</font></tt>
<br><tt><font size=2>>>> import sys</font></tt>
<br><tt><font size=2>>>> print sys.path</font></tt>
<br><tt><font size=2>['', '/home/ubuntu/code/heat', '/home/ubuntu/code/heat/.tox/pep8/lib/python2.7',
'/home/ubuntu/code/heat/.tox/pep8/lib/python2.7/plat-x86_64-linux-gnu',
'/home/ubuntu/code/heat/.tox/pep8/lib/python2.7/lib-tk', '/home/ubuntu/code/heat/.tox/pep8/lib/python2.7/lib-old',
'/home/ubuntu/code/heat/.tox/pep8/lib/python2.7/lib-dynload', '/usr/lib/python2.7',
'/usr/lib/python2.7/plat-x86_64-linux-gnu', '/usr/lib/python2.7/lib-tk',
'/home/ubuntu/code/heat/.tox/pep8/local/lib/python2.7/site-packages', '/home/ubuntu/code/heat/.tox/pep8/lib/python2.7/site-packages']</font></tt>
<br><tt><font size=2>>>> </font></tt>
<br>
<br><tt><font size=2>(pep8)ubuntu@mjs-dstk-821a:~/code/heat$ find .tox/pep8
-name "*messaging*"</font></tt>
<br><tt><font size=2>.tox/pep8/bin/oslo-messaging-zmq-receiver</font></tt>
<br><tt><font size=2>.tox/pep8/lib/python2.7/site-packages/oslo.messaging-1.3.1-py2.7-nspkg.pth</font></tt>
<br><tt><font size=2>.tox/pep8/lib/python2.7/site-packages/oslo.messaging-1.3.1.dist-info</font></tt>
<br><tt><font size=2>.tox/pep8/lib/python2.7/site-packages/qpid/messaging</font></tt>
<br><tt><font size=2>.tox/pep8/lib/python2.7/site-packages/qpid/tests/messaging</font></tt>
<br><tt><font size=2>.tox/pep8/lib/python2.7/site-packages/kombu/messaging.pyc</font></tt>
<br><tt><font size=2>.tox/pep8/lib/python2.7/site-packages/kombu/messaging.py</font></tt>
<br><tt><font size=2>.tox/pep8/lib/python2.7/site-packages/kombu/tests/test_messaging.pyc</font></tt>
<br><tt><font size=2>.tox/pep8/lib/python2.7/site-packages/kombu/tests/test_messaging.py</font></tt>
<br><tt><font size=2>.tox/pep8/lib/python2.7/site-packages/oslo/messaging</font></tt>
<br><tt><font size=2>.tox/pep8/lib/python2.7/site-packages/oslo/messaging/notify/_impl_messaging.pyc</font></tt>
<br><tt><font size=2>.tox/pep8/lib/python2.7/site-packages/oslo/messaging/notify/_impl_messaging.py</font></tt>
<br><tt><font size=2>(pep8)ubuntu@mjs-dstk-821a:~/code/heat$ </font></tt>
<br>
<br><tt><font size=2>Thanks,</font></tt>
<br><tt><font size=2>Mike</font></tt>