*** pcrews has joined #openstack-ironic | 00:18 | |
openstackgerrit | Yuiko Takada proposed stackforge/ironic-discoverd: Load authenticate token from HTTP header https://review.openstack.org/174231 | 00:19 |
---|---|---|
*** ChuckC_ has joined #openstack-ironic | 00:20 | |
*** mtanino has quit IRC | 00:25 | |
*** ijw has joined #openstack-ironic | 00:28 | |
jnielsen55 | everyone gone home for the day | 00:53 |
jnielsen55 | ? | 00:53 |
jnielsen55 | have a couple of questions if anyone is still around | 00:53 |
*** rloo has quit IRC | 00:56 | |
openstackgerrit | Yuiko Takada proposed stackforge/ironic-discoverd: Use keystonemiddleware auth credentials https://review.openstack.org/170782 | 01:03 |
*** yuanying_ has joined #openstack-ironic | 01:10 | |
cinerama | jnielsen55: it's pretty quiet right now but I also know many people in channel read stuff later, so if you are of the sort who has an always-on IRC client, odds are someone will get to your questions | 01:11 |
openstackgerrit | Yuiko Takada proposed stackforge/ironic-discoverd: Use keystonemiddleware auth credentials https://review.openstack.org/170782 | 01:12 |
*** ijw_ has joined #openstack-ironic | 01:12 | |
*** yuanying has quit IRC | 01:14 | |
*** ijw has quit IRC | 01:16 | |
*** ijw has joined #openstack-ironic | 01:23 | |
*** cdearborn has joined #openstack-ironic | 01:24 | |
*** Sukhdev has quit IRC | 01:24 | |
*** Sukhdev has joined #openstack-ironic | 01:25 | |
*** ijw_ has quit IRC | 01:26 | |
*** zsmithnyc has quit IRC | 01:35 | |
*** BadCub has quit IRC | 01:35 | |
*** aweeks has quit IRC | 01:35 | |
*** zsmithnyc has joined #openstack-ironic | 01:36 | |
*** cppforlife_ has quit IRC | 01:36 | |
openstackgerrit | Summer Barringer proposed openstack/bifrost: Add DHCP documentation https://review.openstack.org/187415 | 01:36 |
* TheJulia smiles | 01:37 | |
*** yuanying_ has quit IRC | 01:37 | |
*** BadCub has joined #openstack-ironic | 01:38 | |
*** yuanying has joined #openstack-ironic | 01:39 | |
*** cppforlife_ has joined #openstack-ironic | 01:40 | |
*** cdearborn has quit IRC | 01:41 | |
*** aweeks has joined #openstack-ironic | 01:42 | |
*** Sukhdev has quit IRC | 01:43 | |
*** ijw has quit IRC | 01:46 | |
*** valkyrja has joined #openstack-ironic | 01:53 | |
TheJulia | greetings valkyrja | 01:55 |
*** alexpilotti has quit IRC | 02:00 | |
*** yuanying_ has joined #openstack-ironic | 02:02 | |
*** zhenguo has joined #openstack-ironic | 02:03 | |
valkyrja | hiya TheJulia | 02:04 |
*** yuanying has quit IRC | 02:05 | |
*** eghobo has quit IRC | 02:06 | |
*** zz_natorious is now known as natorious | 02:25 | |
*** shakamunyi has joined #openstack-ironic | 02:26 | |
*** shakamunyi has quit IRC | 02:33 | |
*** chlong has quit IRC | 02:39 | |
openstackgerrit | Merged openstack/ironic: Improve strictness of iLO test cases error checking https://review.openstack.org/177700 | 03:12 |
*** yuanying_ has quit IRC | 03:16 | |
*** chlong has joined #openstack-ironic | 03:22 | |
openstackgerrit | Merged openstack/python-ironicclient: Refactor resource_fields.py https://review.openstack.org/181985 | 03:30 |
*** mtanino has joined #openstack-ironic | 03:48 | |
*** Nisha has joined #openstack-ironic | 03:55 | |
Nisha | Good Morning ironic | 03:55 |
*** eghobo has joined #openstack-ironic | 04:03 | |
*** Sukhdev has joined #openstack-ironic | 04:21 | |
*** david-lyle has quit IRC | 04:24 | |
*** david-lyle has joined #openstack-ironic | 04:25 | |
*** mtanino has quit IRC | 04:27 | |
*** yuanying has joined #openstack-ironic | 04:28 | |
*** yuanying has quit IRC | 04:29 | |
*** gridinv_ has joined #openstack-ironic | 04:37 | |
*** Nisha has quit IRC | 04:38 | |
*** Nisha has joined #openstack-ironic | 04:43 | |
*** kan_ has joined #openstack-ironic | 04:49 | |
*** pradipta has joined #openstack-ironic | 04:50 | |
*** yuanying has joined #openstack-ironic | 04:50 | |
*** spandhe has quit IRC | 04:55 | |
*** yuanying has quit IRC | 04:58 | |
*** spandhe has joined #openstack-ironic | 04:58 | |
Haomeng | Nisha: morning:) | 05:00 |
Nisha | Haomeng, morning | 05:03 |
*** yuanying has joined #openstack-ironic | 05:06 | |
Haomeng | Nisha: ) | 05:09 |
*** yuanying has quit IRC | 05:10 | |
*** nosnos has joined #openstack-ironic | 05:12 | |
openstackgerrit | Yongli He proposed openstack/bifrost: add pyyalm requirement https://review.openstack.org/187455 | 05:12 |
*** yuanying has joined #openstack-ironic | 05:13 | |
*** gridinv_ has quit IRC | 05:15 | |
*** yuanying has quit IRC | 05:15 | |
*** yuanying has joined #openstack-ironic | 05:21 | |
*** ukalifon has joined #openstack-ironic | 05:22 | |
*** lazy_prince has joined #openstack-ironic | 05:26 | |
*** ukalifon has quit IRC | 05:26 | |
*** achanda has joined #openstack-ironic | 05:30 | |
*** achanda_ has joined #openstack-ironic | 05:37 | |
*** nosnos has quit IRC | 05:37 | |
*** zhenguo has quit IRC | 05:37 | |
*** achanda has quit IRC | 05:38 | |
*** ukalifon has joined #openstack-ironic | 05:59 | |
*** spandhe has quit IRC | 06:04 | |
*** achanda_ has quit IRC | 06:07 | |
*** pas-ha has joined #openstack-ironic | 06:11 | |
*** chenglch has joined #openstack-ironic | 06:13 | |
*** radek_ has joined #openstack-ironic | 06:15 | |
*** yuanying has quit IRC | 06:16 | |
*** yuanying has joined #openstack-ironic | 06:18 | |
*** Sukhdev has quit IRC | 06:21 | |
*** ukalifon has quit IRC | 06:21 | |
*** ukalifon1 has joined #openstack-ironic | 06:22 | |
openstackgerrit | Haomeng,Wang proposed openstack/bifrost: supports redhat yum installation https://review.openstack.org/185294 | 06:23 |
*** yuanying has quit IRC | 06:24 | |
*** viktors|afk is now known as viktors | 06:24 | |
*** ijw has joined #openstack-ironic | 06:26 | |
*** yuanying has joined #openstack-ironic | 06:27 | |
openstackgerrit | Feng Xi Yan proposed openstack/ironic: Add documentation for getting a node's console https://review.openstack.org/176642 | 06:33 |
openstackgerrit | Feng Xi Yan proposed openstack/ironic: Add boot interface in Ironic https://review.openstack.org/166512 | 06:35 |
*** ijw has quit IRC | 06:39 | |
*** eghobo has quit IRC | 06:40 | |
*** yuanying has quit IRC | 06:41 | |
*** yuanying has joined #openstack-ironic | 06:43 | |
*** eghobo has joined #openstack-ironic | 06:47 | |
*** jcoufal has joined #openstack-ironic | 06:48 | |
*** eghobo has quit IRC | 06:48 | |
openstackgerrit | Haomeng,Wang proposed openstack/bifrost: supports redhat yum installation https://review.openstack.org/185294 | 06:49 |
*** jcoufal has quit IRC | 06:50 | |
*** eghobo has joined #openstack-ironic | 06:51 | |
*** dtantsur|afk is now known as dtantsur | 06:51 | |
dtantsur | Morning Ironic | 06:51 |
*** jcoufal has joined #openstack-ironic | 06:52 | |
*** eghobo_ has joined #openstack-ironic | 06:52 | |
*** zhenguo has joined #openstack-ironic | 06:54 | |
*** eghobo has quit IRC | 06:55 | |
*** eghobo_ has quit IRC | 06:56 | |
*** ijw has joined #openstack-ironic | 07:08 | |
*** yuanying has quit IRC | 07:09 | |
*** yuanying has joined #openstack-ironic | 07:10 | |
*** ijw_ has joined #openstack-ironic | 07:10 | |
*** rameshg87 has joined #openstack-ironic | 07:11 | |
*** ijw_ has quit IRC | 07:11 | |
*** ijw_ has joined #openstack-ironic | 07:11 | |
*** ijw has quit IRC | 07:14 | |
*** athomas has joined #openstack-ironic | 07:18 | |
*** saripurigopi has joined #openstack-ironic | 07:25 | |
*** ifarkas has joined #openstack-ironic | 07:31 | |
*** yuanying has quit IRC | 07:33 | |
*** saripurigopi has quit IRC | 07:34 | |
*** saripurigopi has joined #openstack-ironic | 07:41 | |
*** yuanying has joined #openstack-ironic | 07:41 | |
*** yuanying has quit IRC | 07:41 | |
*** yuanying has joined #openstack-ironic | 07:42 | |
*** Nisha has quit IRC | 07:43 | |
dtantsur | Haomeng or another core: can I get your +A on https://review.openstack.org/#/c/186772/ ? I guess 3x +2 should be enough :D | 07:47 |
*** dguerri`away is now known as dguerri | 07:48 | |
Haomeng | dtantsur: sure, and I will take your patch as base to commit the patch to support standanlone ironic:) | 07:48 |
*** jistr has joined #openstack-ironic | 07:48 | |
saripurigopi | anyone seeing this problem while running py34 tests on MAC? fatal error: 'CommonCrypto/CommonKeyDerivation.h' file not found | 07:50 |
Haomeng | dtantsur: dtantsur: +a done:) | 07:51 |
dtantsur | thanks! | 07:55 |
dtantsur | saripurigopi, I'm not sure if anyone here develops on MAC actually... | 07:55 |
*** chlong has quit IRC | 07:56 | |
saripurigopi | dtantsur: okay :-( in general ? | 07:57 |
openstackgerrit | Pratyusha proposed stackforge/proliantutils: Add iSCSI boot support in RIS https://review.openstack.org/187162 | 08:04 |
*** takadayuiko has joined #openstack-ironic | 08:09 | |
*** edcragg has joined #openstack-ironic | 08:12 | |
*** bethelwell has joined #openstack-ironic | 08:15 | |
rameshg87 | saripurigopi: did you run tox on your system ? | 08:16 |
saripurigopi | rameshg87: yes it failed while installing cryptography module with this error. | 08:17 |
*** romainh has joined #openstack-ironic | 08:18 | |
rameshg87 | saripurigopi: may be you are missing some headers that are required to compile cryptography module | 08:18 |
rameshg87 | saripurigopi: what's the error ? | 08:18 |
*** lucasagomes has joined #openstack-ironic | 08:18 | |
saripurigopi | building '_Cryptography_cffi_3d141e96xdb61d8fa' extension | 08:19 |
saripurigopi | /usr/bin/clang -fno-strict-aliasing -fno-common -dynamic -DNDEBUG -g -fwrapv -O3 -Wall -Wstrict-prototypes -arch i386 -arch x86_64 -isysroot /Developer/SDKs/MacOSX10.6.sdk -g -I/Library/Frameworks/Python.framework/Versions/3.4/include/python3.4m -c src/cryptography/hazmat/bindings/__pycache__/_Cryptography_cffi_3d141e96xdb61d8fa.c -o build/temp.macosx-10.6-intel-3.4/src/cryptography/hazmat/bin | 08:19 |
saripurigopi | dings/__pycache__/_Cryptography_cffi_3d141e96xdb61d8fa.o | 08:19 |
saripurigopi | src/cryptography/hazmat/bindings/__pycache__/_Cryptography_cffi_3d141e96xdb61d8fa.c:219:10: fatal error: 'CommonCrypto/CommonKeyDerivation.h' file not found | 08:19 |
saripurigopi | #include <CommonCrypto/CommonKeyDerivation.h> | 08:19 |
saripurigopi | ^ | 08:19 |
saripurigopi | 1 error generated. | 08:19 |
saripurigopi | error: command '/usr/bin/clang' failed with exit status 1 | 08:19 |
rameshg87 | saripurigopi: which system are you in ? | 08:19 |
rameshg87 | i mean ubuntu 14.04 ? | 08:19 |
*** derekh has joined #openstack-ironic | 08:19 | |
saripurigopi | rameshg87: I'm on OSX, 10.10 | 08:20 |
Haomeng | saripurigopi: for such "*.h" file missing issue, I think it should be depends on some c/c++ lib installed | 08:21 |
* rameshg87 has never seen OSX :D | 08:22 | |
saripurigopi | Haomeng: okay | 08:22 |
Haomeng | saripurigopi: I have no MAC env, so just try to understand the issue you raised:) | 08:22 |
Haomeng | saripurigopi: and we can google it for solutions:) - https://www.google.com.hk/search?q=CommonCrypto%2FCommonKeyDerivation.h%27+file+not+found&oq=CommonCrypto%2FCommonKeyDerivation.h%27+file+not+found&aqs=chrome..69i57j69i58.385j0j1&sourceid=chrome&ie=UTF-8 | 08:23 |
saripurigopi | Haomeng: I'm trying, couldn't find the solution yet, so thought someone would've seen this already | 08:23 |
Haomeng | saripurigopi: can you show us the lines - src/cryptography/hazmat/bindings/__pycache__/_Cryptography_cffi_3d141e96xdb61d8fa.c | 08:23 |
*** ndipanov has joined #openstack-ironic | 08:24 | |
Haomeng | saripurigopi: and ls /usr/include/CommonCrypto/ | 08:24 |
saripurigopi | ls /usr/include/CommonCrypto/ | 08:25 |
saripurigopi | CommonCrypto.h CommonCryptoError.h CommonCryptor.h CommonDigest.h CommonHMAC.h CommonKeyDerivation.h CommonRandom.h CommonSymmetricKeywrap.h | 08:25 |
Haomeng | saripurigopi: looks fine, CommonKeyDerivation.h is existing as well | 08:25 |
saripurigopi | Haomeng: yeah | 08:26 |
Haomeng | saripurigopi: so how the c code include this header file? | 08:26 |
Haomeng | saripurigopi: can you show - src/cryptography/hazmat/bindings/__pycache__/_Cryptography_cffi_3d141e96xdb61d8fa.c Line 219 | 08:26 |
Haomeng | saripurigopi: should be - #import <CommonCrypto/CommonKeyDerivation.h> ? | 08:27 |
*** pradipta has quit IRC | 08:29 | |
openstackgerrit | Merged openstack/ironic: ironic-discoverd is being renamed to ironic-inspector https://review.openstack.org/186772 | 08:29 |
saripurigopi | Haomeng: Just checking | 08:29 |
Haomeng | saripurigopi: ok | 08:30 |
Haomeng | saripurigopi: this line - #include <CommonCrypto/CommonKeyDerivation.h> ? | 08:32 |
saripurigopi | Haomeng: yeah | 08:32 |
Haomeng | saripurigopi: looks fine | 08:33 |
saripurigopi | Haomeng: right | 08:33 |
Haomeng | saripurigopi: will search system include folder | 08:33 |
rameshg87 | lucasagomes: hi | 08:34 |
lucasagomes | rameshg87, good morning | 08:34 |
* lucasagomes back from holiday :-) | 08:34 | |
*** bethelwell has quit IRC | 08:34 | |
rameshg87 | lucasagomes: morning :) | 08:34 |
rameshg87 | lucasagomes: may be you are just checking things - but your patch worked for me | 08:34 |
rameshg87 | lucasagomes: required a small change in ironic-python-agent which i have proposed in a patch | 08:35 |
lucasagomes | rameshg87, oh the soft reboot? | 08:35 |
rameshg87 | lucasagomes: yeah, the soft reboot | 08:36 |
lucasagomes | nice! yeah I still have to change the methodology | 08:36 |
lucasagomes | as you suggested you know, to do a power off | 08:36 |
lucasagomes | and power on after | 08:36 |
lucasagomes | soft power off* | 08:36 |
rameshg87 | lucasagomes: yeah that will be good | 08:36 |
rameshg87 | lucasagomes: and if we power off and then power on | 08:36 |
rameshg87 | lucasagomes: it works for pxe_ssh as well as wol :) | 08:36 |
lucasagomes | \o/ | 08:37 |
*** romcheg has joined #openstack-ironic | 08:40 | |
*** ukalifon1 has quit IRC | 08:40 | |
*** takadayuiko has quit IRC | 08:40 | |
Haomeng | saripurigopi: maybe we mised the include env, try to set env var - CFLAGS="-I/usr/include" , and have try again | 08:42 |
saripurigopi | Haomeng: I've tried this before, still see the same error. | 08:43 |
Haomeng | saripurigopi: do we make sure this env-var is enabled, maybe need to be added to your makefile | 08:44 |
*** romcheg1 has joined #openstack-ironic | 08:44 | |
saripurigopi | Haomeng: okay, let me check. | 08:44 |
*** romcheg has quit IRC | 08:44 | |
Haomeng | saripurigopi: to make sure the compile process env has such var to include /usr/include folder as default *.h" header files search path | 08:44 |
Haomeng | saripurigopi: how this "/usr/bin/clang -fno-strict-aliasing -fno-common .." launched? | 08:45 |
Haomeng | saripurigopi: sorry, should be CFLAGS=-I/usr/include | 08:47 |
saripurigopi | Running setup.py install for cryptography | 08:47 |
saripurigopi | Complete output from command /Users/saripurigopi/Dev/kilo/ironic/.tox/py34/bin/python3.4 -c "import setuptools, tokenize;__file__='/private/tmp/pip-build-y8l2pwvc/cryptography/setup.py';exec(compile(getattr(tokenize, 'open', open)(__file__).read().replace('\r\n', '\n'), __file__, 'exec'))" install --record /tmp/pip-6kjnvd11-record/install-record.txt --single-version-externally-managed... | 08:47 |
saripurigopi | ...--compile --install-headers /Users/saripurigopi/Dev/kilo/ironic/.tox/py34/bin/../include/site/python3.4/cryptography: | 08:47 |
saripurigopi | running install | 08:48 |
saripurigopi | running build | 08:48 |
saripurigopi | running build_py | 08:48 |
saripurigopi | running egg_info | 08:48 |
saripurigopi | writing dependency_links to src/cryptography.egg-info/dependency_links.txt | 08:48 |
saripurigopi | writing top-level names to src/cryptography.egg-info/top_level.txt | 08:48 |
saripurigopi | writing entry points to src/cryptography.egg-info/entry_points.txt | 08:48 |
saripurigopi | writing src/cryptography.egg-info/PKG-INFO | 08:48 |
saripurigopi | writing requirements to src/cryptography.egg-info/requires.txt | 08:48 |
saripurigopi | warning: manifest_maker: standard file '-c' not found | 08:48 |
saripurigopi | reading manifest file 'src/cryptography.egg-info/SOURCES.txt' | 08:48 |
saripurigopi | reading manifest template 'MANIFEST.in' | 08:48 |
saripurigopi | no previously-included directories found matching 'docs/_build' | 08:48 |
saripurigopi | warning: no previously-included files matching '*' found under directory 'vectors' | 08:48 |
saripurigopi | writing manifest file 'src/cryptography.egg-info/SOURCES.txt' | 08:48 |
saripurigopi | warning: build_py: byte-compiling is disabled, skipping. | 08:48 |
saripurigopi | running build_ext | 08:48 |
saripurigopi | building '_Cryptography_cffi_3d141e96xdb61d8fa' extension | 08:48 |
Haomeng | saripurigopi: can you paste into paste.openstack.org, and share the url, thank you:) | 08:48 |
Haomeng | saripurigopi: that should be easy to read for us:) | 08:49 |
Haomeng | saripurigopi: working now? | 08:49 |
saripurigopi | Haomeng: sure, still adapting. | 08:49 |
saripurigopi | Haomeng: here is the complete log for tox -epy34 | 08:52 |
Haomeng | saripurigopi: the <*.h> header files search path should be passed by "-I" for c/c++ Compiler, so make sure the compiler process has such "-I/usr/include", should work, it is big i, I = Include | 08:52 |
Haomeng | saripurigopi: ok | 08:52 |
saripurigopi | Haomeng: okay | 08:52 |
*** leopoldj has joined #openstack-ironic | 08:53 | |
Haomeng | saripurigopi: can not get you, working now? | 08:53 |
*** kbyrne has joined #openstack-ironic | 08:53 | |
Haomeng | saripurigopi: understand you try to setup ironic dev env on MAC, right? | 08:53 |
*** achanda has joined #openstack-ironic | 08:54 | |
Haomeng | saripurigopi: will be away for a while and get back to you soon:) | 08:54 |
saripurigopi | Haomeng:yes, I've been using for quite sometime, only seeing this issue with py34 | 08:54 |
saripurigopi | Haomeng: sure | 08:54 |
Haomeng | saripurigopi: ok, got it should be py34 env issue for c/c++ compiler, missing some compiler -I include path env I think | 08:55 |
saripurigopi | Haomeng: I think so. | 08:55 |
*** achanda_ has joined #openstack-ironic | 08:56 | |
*** dtantsur is now known as dtantsur|brb | 08:58 | |
*** ukalifon has joined #openstack-ironic | 09:00 | |
*** achanda has quit IRC | 09:00 | |
openstackgerrit | Zhenguo Niu proposed openstack/ironic-specs: Add nodes tagging support https://review.openstack.org/183971 | 09:01 |
*** ukalifon has quit IRC | 09:12 | |
*** ijw_ has quit IRC | 09:19 | |
*** e0ne has joined #openstack-ironic | 09:20 | |
*** achanda_ has quit IRC | 09:21 | |
openstackgerrit | Zhenguo Niu proposed openstack/ironic-specs: Add nodes tagging support https://review.openstack.org/183971 | 09:30 |
*** Nisha has joined #openstack-ironic | 09:34 | |
saripurigopi | Haomeng: It is working now :-), i'm able to run py34 tests. many thanks. | 09:34 |
pshige | getting a little better | 09:37 |
*** romcheg1 has quit IRC | 09:37 | |
pshige | anyway Tempest is tough ... | 09:38 |
*** romcheg has joined #openstack-ironic | 09:40 | |
*** romcheg1 has joined #openstack-ironic | 09:41 | |
*** romcheg has quit IRC | 09:44 | |
*** rameshg871 has joined #openstack-ironic | 09:46 | |
openstackgerrit | Zhenguo Niu proposed openstack/ironic: Add disk layout check on re-provisioning https://review.openstack.org/180014 | 09:47 |
*** rameshg87 has quit IRC | 09:48 | |
*** rameshg87 has joined #openstack-ironic | 09:49 | |
*** rameshg871 has quit IRC | 09:51 | |
*** Nisha_away has joined #openstack-ironic | 09:51 | |
Haomeng | saripurigopi: great, how did you get it works? | 09:51 |
saripurigopi | added env CFLAGS=-I/usr/include , before tox . | 09:52 |
openstackgerrit | Sam Betts proposed openstack/ironic: Fix flake8 E123/6/7/8 in ironic/api https://review.openstack.org/186458 | 09:52 |
openstackgerrit | Sam Betts proposed openstack/ironic: Fix flake8 E123/6/7/8 in ironic/common https://review.openstack.org/186456 | 09:52 |
openstackgerrit | Sam Betts proposed openstack/ironic: Fix flake8 E123/6/7/8 in ironic /cmd /db and /dhcp https://review.openstack.org/186457 | 09:52 |
openstackgerrit | Sam Betts proposed openstack/ironic: Fix flake8 E123/6/7/8 for ironic/tests/drivers https://review.openstack.org/186450 | 09:52 |
openstackgerrit | Sam Betts proposed openstack/ironic: Fix flake8 E123/6/7/8 in non-driver ironic/tests https://review.openstack.org/186451 | 09:52 |
openstackgerrit | Sam Betts proposed openstack/ironic: Fixes flake8 E123/6/7/8 for ironic/drivers https://review.openstack.org/186454 | 09:52 |
openstackgerrit | Sam Betts proposed openstack/ironic: Remove E123/6/7/8 from flake8 ignore list https://review.openstack.org/186021 | 09:52 |
openstackgerrit | Sam Betts proposed openstack/ironic: Fixes flake8 E123/6/7/8 in ironic/conductor https://review.openstack.org/186455 | 09:52 |
openstackgerrit | Sam Betts proposed openstack/ironic: Fix flake8 E123/6/7/8 for ironic/tests/api https://review.openstack.org/186452 | 09:52 |
openstackgerrit | Sam Betts proposed openstack/ironic: Fixes flake8 E123/6/7/8 for ironic/tests/conductor https://review.openstack.org/186453 | 09:52 |
*** rameshg871 has joined #openstack-ironic | 09:53 | |
*** rameshg87 has quit IRC | 09:53 | |
*** Nisha has quit IRC | 09:54 | |
openstackgerrit | Zhenguo Niu proposed openstack/ironic-specs: Add nodes tagging support https://review.openstack.org/183971 | 09:54 |
*** bethelwell has joined #openstack-ironic | 09:58 | |
*** dtantsur|brb is now known as dtantsur | 10:00 | |
*** achanda has joined #openstack-ironic | 10:00 | |
*** Nisha_away has quit IRC | 10:00 | |
*** Nisha has joined #openstack-ironic | 10:03 | |
Haomeng | saripurigopi: cool:) | 10:06 |
Haomeng | saripurigopi: I guess it can be passed to venv and used by c/c++ compiler | 10:07 |
*** kan_ has quit IRC | 10:09 | |
*** jprovazn has joined #openstack-ironic | 10:10 | |
dtantsur | lucasagomes, hey, seems like we have a problem with localboot | 10:10 |
lucasagomes | dtantsur, hi there, whats up? | 10:10 |
dtantsur | lucasagomes, looks like grub config that is generated for an image works only for bm | 10:10 |
dtantsur | lucasagomes, jprovazn knows more details | 10:10 |
jprovazn | dtantsur: lucasagomes: Hi, it's this BZ: | 10:11 |
jprovazn | https://bugzilla.redhat.com/show_bug.cgi?id=1226097 | 10:11 |
openstack | bugzilla.redhat.com bug 1226097 in rhel-osp-director "rhel-osp-director: The overcloud deployment times out." [High,Assigned] - Assigned to jprovazn | 10:11 |
* lucasagomes looks | 10:11 | |
lucasagomes | hmmmm | 10:12 |
lucasagomes | you think it's generating for the kernel that is currently running? | 10:12 |
jprovazn | the problem is that /etc/default/grub.conf is generated on a machine where images are being built, this grub.conf (cmdline console param) works then on the same machine where it was built | 10:12 |
lucasagomes | (since we generate it from the deploy ramdisk) | 10:12 |
lucasagomes | gotcha | 10:12 |
lucasagomes | I will take a look at it | 10:13 |
jprovazn | but if I try prebuilt images as VMs it fails | 10:13 |
*** e0ne is now known as e0ne_ | 10:13 | |
lucasagomes | we probably can open a bug upstream about it too | 10:13 |
jprovazn | lucasagomes: in prebuilt images, it;s this param which breaks it: | 10:13 |
jprovazn | console=ttyS0,115200 | 10:13 |
jprovazn | so I wonder if we are able to find CMDLINE which works both for VMs and for baremetal | 10:14 |
openstackgerrit | Shivanand Tendulker proposed openstack/ironic: Validate capability in properties and instance_info https://review.openstack.org/174785 | 10:14 |
lucasagomes | oh right it's trying to redirect the console | 10:14 |
lucasagomes | well I tihnk it's not very nice to have console redirect on ttyS0 by default anyway... since the device may be called something else | 10:14 |
lucasagomes | ttyS1 perhaps | 10:14 |
jprovazn | lucasagomes: in tripleo for VMs there is an element 'vm' which overwites this grub.conf | 10:14 |
jprovazn | lucasagomes: but it's used for VMs only | 10:15 |
lucasagomes | I see, I will take a look into it, see if I can come up with something | 10:15 |
jprovazn | lucasagomes: thanks | 10:15 |
lucasagomes | jprovazn, thank you for reporting it | 10:15 |
saripurigopi | Haomeng: okay, not sure how to try that. | 10:15 |
jprovazn | lucasagomes: np, actually Yaniv hit and reported this | 10:16 |
lucasagomes | I see | 10:16 |
jprovazn | s/Yaniv/Alexander/ :) | 10:16 |
*** ijw has joined #openstack-ironic | 10:16 | |
Haomeng | saripurigopi: ok | 10:17 |
Nisha | dtantsur, lucasagomes hi | 10:17 |
lucasagomes | Nisha, hi there | 10:18 |
Nisha | dtantsur, lucasagomes i was POC'ing the https://review.openstack.org/182934. (converting capabilities to dictionary) | 10:18 |
Nisha | lucasagomes, ^^^ | 10:18 |
lucasagomes | nice one, thanks | 10:19 |
*** ijw_ has joined #openstack-ironic | 10:19 | |
Nisha | But i find it requires changes in python-ironicclient too | 10:20 |
Nisha | it will require changes in ironicclient/openstack/common/cliutils.py | 10:20 |
lucasagomes | why is that? | 10:20 |
Nisha | 1. capabilities is not a direct field of the node table | 10:20 |
Nisha | 2. assume i give following command line | 10:21 |
Nisha | without any changes to the ironci client today | 10:21 |
Nisha | ironic node-update $NODE add properties/capabilities={'abc':'def','xyz':'pqr'} | 10:21 |
Nisha | in that case it will just update the last key:value pair | 10:22 |
*** ijw has quit IRC | 10:22 | |
Nisha | it will be updated as " properties | {u'capabilities': u'xyz:pqr'} | 10:23 |
lucasagomes | o.o | 10:23 |
Nisha | this is without any modification to client | 10:23 |
lucasagomes | we have other fields that are dict and updatable | 10:23 |
*** e0ne_ has quit IRC | 10:24 | |
lucasagomes | like instance_info/capabilities is a dict | 10:24 |
lucasagomes | I will test it soon | 10:24 |
*** ijw_ has quit IRC | 10:24 | |
Nisha | can it hold multiple key_value pair when u use node-update | 10:24 |
Nisha | i remember it cant | 10:24 |
lucasagomes | https://github.com/openstack/nova/blob/master/nova/virt/ironic/patcher.py#L98-L100 | 10:24 |
Nisha | ironic node-update 48fcee94-f260-46cd-b865-644c1bb0fce0 add instance_info/capabilities={'abc':'def','xyz':'pqr'} | 10:25 |
* lucasagomes will test, 1 sec | 10:25 | |
lucasagomes | you probably should wrap with "" | 10:25 |
lucasagomes | cause of bash | 10:25 |
Nisha | instance_info | {u'capabilities': u'xyz:pqr'} | 10:26 |
openstackgerrit | Yuriy Zveryanskyy proposed openstack/ironic: Do not save auth token on TFTP server in PXE driver https://review.openstack.org/159819 | 10:26 |
lucasagomes | stack@virtual-machine:~/devstack$ ironic node-update 267e42c8-df07-49f5-bc7f-48b566acb109 add instance_info/capabilities="{'test': 'test', 'foo': 'bar'}" | 10:26 |
lucasagomes | | instance_info | {u'capabilities': u"{'test': 'test', 'foo': 'bar'}"} | | 10:26 |
Nisha | oh u used double quotes | 10:27 |
lucasagomes | tho I which we were able to do: node-update <node> add instance_info/capabilities/key=value | 10:27 |
lucasagomes | that's possible if the field always exist as a dict | 10:27 |
Nisha | thats a string for the capabilities key | 10:27 |
Nisha | yes | 10:27 |
lucasagomes | but if it doesn't exist prior to the update, json-patch (the lib) will not create it | 10:27 |
lucasagomes | will fail with a pointer exception | 10:27 |
Nisha | which json-patch^^^ | 10:28 |
lucasagomes | Nisha, oh yeah you gotta do the oposite cause json differentiate " of ; | 10:28 |
lucasagomes | for ' | 10:28 |
lucasagomes | urgh... of ' | 10:29 |
lucasagomes | stack@virtual-machine:~/devstack$ ironic node-update 267e42c8-df07-49f5-bc7f-48b566acb109 add instance_info/capabilities='{"test": "test", "foo": "bar"}' | 10:29 |
lucasagomes | | instance_info | {u'capabilities': {u'test': u'test', u'foo': u'bar'}} | | 10:29 |
lucasagomes | that's misguided indeed | 10:29 |
*** e0ne has joined #openstack-ironic | 10:29 | |
Nisha | but thats still a string | 10:29 |
Nisha | its not a dictionary | 10:29 |
lucasagomes | it's a dict | 10:30 |
Nisha | instance_info | {u'capabilities': u'{abc:def,xyz:pqr}'} | 10:30 |
lucasagomes | Nisha, invert double and single quotes | 10:30 |
lucasagomes | cause json differentiate then | 10:30 |
lucasagomes | (I know it sucks) | 10:31 |
lucasagomes | so yeah I would +1 to have the client to handle such situations better | 10:31 |
Nisha | hmmm | 10:32 |
lucasagomes | try to parse the right type and convert it appropriately | 10:32 |
TheJulia | cinerama: https://github.com/ansible/ansible/commit/b94e2a1f4ee1631d311f6943f6653c391d5022de broke us, I just asked in the ansible chnanel about it. Seems... wrong to me. | 10:32 |
lucasagomes | Nisha, did you try inverting? | 10:32 |
*** [1]cdearborn has joined #openstack-ironic | 10:33 | |
Nisha | lucasagomes, i was about to propose the capabilities as seperate field in node table so that its updating is also easier when its already has some key=value pairs assigned | 10:33 |
lucasagomes | hmm | 10:33 |
*** ukalifon has joined #openstack-ironic | 10:33 | |
lucasagomes | it sounds like we better make it more usable for all the dict fields | 10:33 |
lucasagomes | by having the client to parse stuff more elegantly | 10:34 |
lucasagomes | having a separated field sounds overkill no? | 10:34 |
TheJulia | cinerama: of course, if I try to lock to ansible stable-1.9... segfault *boom* on shade pip step | 10:34 |
Nisha | ironci takes care of existing key/value pairs and not the client, as i understand | 10:35 |
lucasagomes | right, yeah sorry, perhaps the API should parse stuff better | 10:36 |
lucasagomes | we have custom types for dicts and all | 10:37 |
lucasagomes | we can just improve that parsing | 10:37 |
*** dguerri is now known as dguerri`away | 10:37 | |
Nisha | yes i know...i was trying that stuff too...ok let me try and if that works then this can be done | 10:37 |
lucasagomes | ack | 10:38 |
lucasagomes | thanks | 10:38 |
TheJulia | cinerama: bug opened in bifrost project, going to try pushing up a change to move the job over to stable-1.9 | 10:38 |
*** ifarkas has quit IRC | 10:41 | |
*** yuanying has quit IRC | 10:43 | |
*** alexpilotti has joined #openstack-ironic | 10:46 | |
openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: API: Get a subset of fields from Ports and Chassis https://review.openstack.org/183549 | 10:50 |
openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: API: Get a subset of fields from Nodes https://review.openstack.org/183491 | 10:50 |
*** chenglch has quit IRC | 10:54 | |
*** achanda_ has joined #openstack-ironic | 10:56 | |
*** achanda has quit IRC | 10:56 | |
*** viktors has quit IRC | 10:57 | |
openstackgerrit | Sam Betts proposed stackforge/ironic-discoverd: Add SSL/TLS Support https://review.openstack.org/186843 | 11:04 |
*** ifarkas has joined #openstack-ironic | 11:16 | |
*** e0ne has quit IRC | 11:19 | |
*** ijw has joined #openstack-ironic | 11:19 | |
*** pelix has joined #openstack-ironic | 11:21 | |
*** ijw_ has joined #openstack-ironic | 11:23 | |
*** e0ne has joined #openstack-ironic | 11:24 | |
*** e0ne is now known as e0ne_ | 11:25 | |
*** ijw has quit IRC | 11:25 | |
*** [1]cdearborn has quit IRC | 11:26 | |
*** ijw_ has quit IRC | 11:27 | |
*** achanda_ has quit IRC | 11:28 | |
*** e0ne_ is now known as e0ne | 11:30 | |
openstackgerrit | Julia Kreger proposed openstack/bifrost: Lock Bifrost to Ansible stable-1.9 https://review.openstack.org/187532 | 11:30 |
TheJulia | cinerama: ^^^ | 11:31 |
openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Update the config drive doc to remove deprecated value https://review.openstack.org/187534 | 11:35 |
*** Nisha has quit IRC | 11:40 | |
openstackgerrit | Merged openstack/ironic: Updated from global requirements https://review.openstack.org/186931 | 11:42 |
*** lucasagomes is now known as lucas-hungry | 11:43 | |
*** dprince has joined #openstack-ironic | 11:48 | |
openstackgerrit | SHIGEMATSU Mitsuhiro proposed openstack/ironic: Secure boot support for pxe_ilo driver https://review.openstack.org/154808 | 11:53 |
*** romainh has quit IRC | 11:54 | |
*** romainh1 has joined #openstack-ironic | 11:54 | |
*** zhenguo has quit IRC | 11:57 | |
*** radek_ has quit IRC | 11:59 | |
openstackgerrit | Yuriy Zveryanskyy proposed openstack/ironic: Do not save auth token on TFTP server in PXE driver https://review.openstack.org/159819 | 12:00 |
TheJulia | Stepping away to get some breakfast and visit the dentist. | 12:01 |
*** radek_ has joined #openstack-ironic | 12:03 | |
*** ifarkas has quit IRC | 12:04 | |
*** e0ne is now known as e0ne_ | 12:04 | |
*** dkehn__ has joined #openstack-ironic | 12:05 | |
*** dkehn has quit IRC | 12:08 | |
*** achanda has joined #openstack-ironic | 12:09 | |
*** thiagop has quit IRC | 12:11 | |
*** albertoffb has joined #openstack-ironic | 12:13 | |
*** e0ne_ has quit IRC | 12:14 | |
openstackgerrit | Dmitry Tantsur proposed stackforge/ironic-discoverd: Update from global requirements https://review.openstack.org/187544 | 12:17 |
*** jistr is now known as jistr|class | 12:17 | |
*** saripurigopi has quit IRC | 12:18 | |
*** ijw has joined #openstack-ironic | 12:18 | |
*** radek_ has quit IRC | 12:20 | |
*** achanda has quit IRC | 12:20 | |
*** achanda has joined #openstack-ironic | 12:21 | |
*** jistr|class is now known as jistr | 12:22 | |
*** ijw has quit IRC | 12:23 | |
*** achanda has quit IRC | 12:25 | |
* rameshg871 wonders if dtantsur doesn't have a bot to do the update | 12:26 | |
dtantsur | rameshg871, my patch to requirements repo was opened for ~ 3 months... | 12:27 |
dtantsur | now I wip'ed it due to rename | 12:27 |
rameshg871 | oh okay | 12:27 |
*** ifarkas has joined #openstack-ironic | 12:34 | |
sambetts | dtantsur: ping, I'm looking at taking up this https://bugs.launchpad.net/ironic-discoverd/+bug/1460689, but I would like to discuss where best it would be to modify the response | 12:35 |
openstack | Launchpad bug 1460689 in ironic-inspector "/v1/continue should return node UUID to support IPA ramdisks" [High,Triaged] | 12:35 |
*** e0ne has joined #openstack-ironic | 12:36 | |
dtantsur | lemme see | 12:36 |
sambetts | dtantsur: It seems like the main body of the response is generated by _process_node | 12:36 |
dtantsur | sambetts, somewhere https://github.com/stackforge/ironic-discoverd/blob/master/ironic_inspector/process.py#L164-L167 | 12:36 |
dtantsur | right | 12:37 |
sambetts | dtantsur: my other question would be, the format of the response, I could either add the UUID as another parameter in the current dict or like { UUID: <current dict> } what are your thoughts | 12:38 |
dtantsur | sambetts, not sure why, we're processing one node at time | 12:39 |
dtantsur | so just adding a "node_uuid" (or just "uuid"?) field should be enough | 12:39 |
dtantsur | * adding to existing dict | 12:39 |
sambetts | dtantsur: awesome :) just wanted to clarify, thats what I thought but then I doubted myself haha | 12:40 |
dtantsur | heh :) | 12:40 |
openstackgerrit | SHIGEMATSU Mitsuhiro proposed openstack/ironic-python-agent: Make sure sysrq functions are enabled in reboot.sh https://review.openstack.org/187111 | 12:41 |
* rameshg871 goes home | 12:41 | |
*** rameshg871 has left #openstack-ironic | 12:41 | |
*** viktors has joined #openstack-ironic | 12:43 | |
*** dkehn__ is now known as dkehn | 12:44 | |
openstackgerrit | SHIGEMATSU Mitsuhiro proposed openstack/ironic-specs: iPXE to use Swift Temporary URLs https://review.openstack.org/181920 | 12:48 |
*** athomas has quit IRC | 13:04 | |
*** [1]cdearborn has joined #openstack-ironic | 13:11 | |
openstackgerrit | Merged stackforge/ironic-discoverd: Update from global requirements https://review.openstack.org/187544 | 13:14 |
*** ukalifon has quit IRC | 13:17 | |
*** ijw has joined #openstack-ironic | 13:18 | |
*** ChuckC_ has quit IRC | 13:22 | |
*** ijw has quit IRC | 13:23 | |
openstackgerrit | Sam Betts proposed stackforge/ironic-discoverd: Add node UUID to response from /v1/continue https://review.openstack.org/187575 | 13:23 |
*** athomas has joined #openstack-ironic | 13:26 | |
*** Nisha has joined #openstack-ironic | 13:27 | |
*** lucas-hungry is now known as lucasagomes | 13:29 | |
*** dguerri`away is now known as dguerri | 13:31 | |
*** dkehn is now known as dkehnx | 13:33 | |
*** ukalifon1 has joined #openstack-ironic | 13:34 | |
*** dkehnx has quit IRC | 13:35 | |
*** lazy_prince has quit IRC | 13:37 | |
*** ErickHeinrich has joined #openstack-ironic | 13:41 | |
*** ErickHeinrich has quit IRC | 13:41 | |
NobodyCam | good morning (UGT) Ironicers | 13:42 |
lucasagomes | NobodyCam, good ugt morning | 13:47 |
NobodyCam | hey hey lucasagomes | 13:47 |
NobodyCam | how are you doing this day | 13:47 |
lucasagomes | yesterday was holiday here, I tried to arrive on time for the meeting but | 13:47 |
lucasagomes | I was out and didn't make it | 13:47 |
NobodyCam | NP | 13:47 |
NobodyCam | :) | 13:47 |
lucasagomes | NobodyCam, all is well, how are things over there? | 13:47 |
NobodyCam | getting better.. :) getting things ready for our trip north :) | 13:48 |
*** mtanino has joined #openstack-ironic | 13:51 | |
sambetts | Morning NobodyCam o/ | 13:53 |
NobodyCam | morning sambetts | 13:54 |
NobodyCam | :) | 13:54 |
openstackgerrit | Jarrod Johnson proposed stackforge/pyghmi: Implement OEM Inventory components https://review.openstack.org/187592 | 13:56 |
NobodyCam | sambetts: how are things going? I forgot what side of the world do you live? | 13:57 |
sambetts | NobodyCam: Good thanks, wet and windy, but I'm in the south east of England so what can I expect :P | 13:59 |
NobodyCam | ahh :) thats right | 13:59 |
NobodyCam | last time I was in Bristol it was cold and rainy too so that seems par for the course :) | 14:00 |
sambetts | :-P pretty much, although we're meant to have some good weather this weekend, so I'm looking forward to that, where abouts are you based if you don't mind me asking? | 14:02 |
NobodyCam | currently we in Palm Springs Ca (southern California) getting ready to head north to Washington state | 14:04 |
NobodyCam | for the summier | 14:04 |
NobodyCam | summer even | 14:04 |
NobodyCam | getting too hot down here | 14:04 |
NobodyCam | was like 41.6 (for you) this last weekend here | 14:05 |
lucasagomes | sambetts, good ugt morning | 14:05 |
* NobodyCam keeps reading UGT as UGH :-p | 14:06 | |
sambetts | NobodyCam: I bet, I lived in San Jose in 2012, and it got up to 100F mid summmer, it was hooot | 14:06 |
sambetts | when did we start using UGT? | 14:07 |
sambetts | :-P | 14:07 |
NobodyCam | ya this is area is low desert so we get just that much warmer | 14:07 |
NobodyCam | about a week or so ago | 14:07 |
NobodyCam | someone shared http://www.total-knowledge.com/~ilya/mips/ugt.html | 14:07 |
NobodyCam | and we all liked it | 14:07 |
dtantsur | oh great | 14:08 |
dtantsur | good ugt morning, NobodyCam! | 14:08 |
NobodyCam | hehehe good (UGT) morning dtantsur ...: | 14:08 |
NobodyCam | :) | 14:08 |
sambetts | haha | 14:08 |
*** rloo has joined #openstack-ironic | 14:09 | |
trown | lol, it seems like the UGT could be implied | 14:09 |
trown | also, good [UGT] morning | 14:10 |
NobodyCam | mornign trown :) | 14:10 |
NobodyCam | okay time to refill coffee cup and then to file several late reports :-p | 14:11 |
*** jistr is now known as jistr|mtg | 14:16 | |
openstackgerrit | Yuriy Zveryanskyy proposed openstack/ironic: Do not preserve partitions after ephemeral https://review.openstack.org/182688 | 14:16 |
*** ijw has joined #openstack-ironic | 14:18 | |
*** radek_ has joined #openstack-ironic | 14:19 | |
openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Prevent node delete based on provision, not power, state https://review.openstack.org/186361 | 14:22 |
openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Remove bogus conditional from node_update https://review.openstack.org/186386 | 14:22 |
*** ijw has quit IRC | 14:23 | |
*** ukalifon2 has joined #openstack-ironic | 14:23 | |
jnielsen55 | good morning all... question, I am using Neutron with OVS and vlans. Do I need to change my entire network config to flat to get the baremetal machines talking with the DHCP service? | 14:24 |
*** ukalifon1 has quit IRC | 14:25 | |
NobodyCam | morning jnielsen55: we have several folks working on Neutron intragration right now. It is one of our goals for this cycle. | 14:25 |
NobodyCam | I would point you to some of the rackspace folk who have a work-around in production | 14:26 |
NobodyCam | you can see more information about this here: https://etherpad.openstack.org/p/liberty-ironic-network-isolation | 14:27 |
jnielsen55 | Thanks NobodyCam do you think this integeration will be done for liberty or further down the road? | 14:27 |
BadCub | morning folks | 14:28 |
NobodyCam | morning BadCub :) | 14:29 |
sambetts | o/ BadCub | 14:29 |
NobodyCam | jnielsen55: It it one of our top goals for this cycle .. so I think it will land :) | 14:29 |
BadCub | mornin NobodyCam sambetts :) | 14:29 |
jnielsen55 | very cool, thanks! | 14:30 |
*** jistr|mtg is now known as jistr | 14:30 | |
NobodyCam | :) | 14:31 |
*** bethelwell has quit IRC | 14:35 | |
*** rameshg87 has joined #openstack-ironic | 14:39 | |
*** saripurigopi_ has joined #openstack-ironic | 14:39 | |
*** zz_jgrimm is now known as jgrimm | 14:43 | |
TheJulia | cinerama: wrt the ansible issues, seems like they have er | 14:44 |
TheJulia | merged a fix | 14:44 |
*** jprovazn has quit IRC | 14:44 | |
*** Nisha has quit IRC | 14:45 | |
*** coolsvap|afk has quit IRC | 14:45 | |
*** jnielsen55 has quit IRC | 14:45 | |
*** lazy_prince has joined #openstack-ironic | 14:46 | |
lazy_prince | 0/ | 14:47 |
NobodyCam | morning TheJulia and lazy_prince | 14:48 |
TheJulia | good morning | 14:48 |
*** coolsvap|afk has joined #openstack-ironic | 14:48 | |
openstackgerrit | Ramakrishnan G proposed openstack/ironic-specs: Add generic RAID configuration spec for liberty https://review.openstack.org/173214 | 14:51 |
openstackgerrit | Ramakrishnan G proposed openstack/ironic-specs: Implement Zapping States https://review.openstack.org/185122 | 14:51 |
openstackgerrit | Ramakrishnan G proposed openstack/ironic-specs: Add inband RAID configuration spec for liberty https://review.openstack.org/173218 | 14:51 |
*** athomas has quit IRC | 14:52 | |
*** erelwell has joined #openstack-ironic | 14:52 | |
*** erelwell is now known as bethelwell | 14:52 | |
*** jnielsen55 has joined #openstack-ironic | 14:52 | |
rloo | hello and morning, Ironic'ers, TheJulia, NobodyCam, BadCub, lazy_prince, sambetts | 14:53 |
openstackgerrit | Jarrod Johnson proposed stackforge/pyghmi: Implement OEM Inventory components https://review.openstack.org/187592 | 14:53 |
*** killer_prince has joined #openstack-ironic | 14:53 | |
NobodyCam | mornign rloo :) | 14:54 |
rloo | hiya lucasagomes, let me know if you want to discuss 186361 (the HTTP status that is returned) | 14:54 |
lucasagomes | rloo, hi there | 14:54 |
lucasagomes | rloo, will take a look | 14:54 |
*** lazy_prince has quit IRC | 14:54 | |
openstackgerrit | Merged stackforge/ironic-discoverd: Add SSL/TLS Support https://review.openstack.org/186843 | 14:54 |
lucasagomes | rloo, oh, right... Cause I the default return code of InvalidStateRequested is 400. Yeah, actually 400 seems to be the right retunr code for this | 14:55 |
lucasagomes | cause we "abuse" of 409 | 14:55 |
sambetts | Woop thanks dtantsur :D | 14:55 |
sambetts | morning rloo o/ | 14:55 |
lucasagomes | 409 basically is automatically retried by our client, if a node is ACTIVE and one tries to delete it will keep retrying | 14:55 |
rloo | lucasagomes: so is it 'ok' to change it? wondering about backwards compatibility | 14:56 |
rloo | lucasagomes: eg, someone codes for http 409 and now it is changed to 400 | 14:56 |
lucasagomes | rloo, yeah it's bad, I'm thinking | 14:56 |
rloo | lucasagomes: or is it enough to document it | 14:56 |
*** saripurigopi_ has quit IRC | 14:57 | |
cinerama | TheJulia: yay ansible fixing party | 14:57 |
NobodyCam | mornign cinerama | 14:57 |
cinerama | oh also hello everyone | 14:57 |
BadCub | morning rloo :) | 14:57 |
cinerama | hello NobodyCam | 14:57 |
rloo | and the conflict/409 is another issue. eg if I try to create a node with the name of an existing node, the CLI tries 6 times before stopping :-( | 14:57 |
cinerama | hello rloo | 14:57 |
cinerama | hello BadCub | 14:57 |
BadCub | mornin lucasagomes cinerama TheJulia :) | 14:57 |
rloo | hi cinerama | 14:58 |
lucasagomes | rloo, yeah, the api still returns 409 for the instance is associated x.x | 14:58 |
jlvillal | lucasagomes, Thanks for getting the WSME version bumped :) | 14:58 |
lucasagomes | jlvillal, no worries | 14:58 |
TheJulia | cinerama: i've not fully confirmed all is better. After the dentist | 14:58 |
NobodyCam | mornign jlvillal | 14:58 |
*** ukalifon has joined #openstack-ironic | 14:58 | |
jlvillal | NobodyCam, Good morning. And good morning Ironic. | 14:58 |
NobodyCam | :) | 14:58 |
cinerama | TheJulia: yup, i need to go to the dentist in a couple hours as well | 14:59 |
rloo | lucasagomes: I guess the 'conflict' stuff isn't clear to me. In your case, no state was requested. but there is a conflict, cuz it can't be deleted while node is in that state. | 14:59 |
NobodyCam | must the day of dentist visits | 14:59 |
lucasagomes | rloo, usually CONFLICT means another operation is in progress so the request conflicted | 14:59 |
lucasagomes | but the user can retry without changing anything in the request/resource | 15:00 |
rloo | lucasagomes: 'usually'... hmm. We might want to revisit the 409s. | 15:00 |
lucasagomes | where 400 BadRequest (and the reason why I think it's more suitable) the user tried to delete a node in a state that can't be deleted | 15:00 |
lucasagomes | so he have to do some action on it prior to retry | 15:00 |
dtantsur | morning rloo, jlvillal, BadCub, cinerama, and everyone else | 15:01 |
openstackgerrit | John L. Villalovos proposed openstack/ironic: Revert "Add simplegeneric to py34 requirements" https://review.openstack.org/186208 | 15:01 |
lucasagomes | and our client and nova driver retries automatically on 409 | 15:01 |
lucasagomes | so with the current code it will retry to delete a couple of times the node | 15:01 |
lucasagomes | :-/ | 15:01 |
TheJulia | NobodyCam: seems like it :) | 15:01 |
BadCub | mornin dtantsur :) | 15:01 |
rloo | lucasagomes: so it could be 409 if the target_provision_state was set to 'doing something'. | 15:01 |
*** ukalifon2 has quit IRC | 15:01 | |
rloo | lucasagomes: why 409 if an instance is associated with the node? should that be 400 then? | 15:01 |
lucasagomes | rloo, yeah... I think I will keep the 409 for that patch | 15:01 |
lucasagomes | since it involves more thinking about how we better solve it | 15:02 |
lucasagomes | rloo, yeah I think it should be 400 | 15:02 |
lucasagomes | rloo, but that code is really old | 15:02 |
*** hemnafk is now known as hemna | 15:02 | |
lucasagomes | so some assumptions there I don't remember anymore | 15:02 |
rloo | lucasagomes: ok, i was going to open a bug about 409/conflict anyway. will do that soonish. | 15:02 |
lucasagomes | like we couldn't start a deployment if the node wasn't power off before | 15:03 |
lucasagomes | I think that's the reason of the delete | 15:03 |
jnielsen55 | Is it possible to use a external_dhcp server with Ironic and when the networking integeration (support for vlans and tunnels) is completed? | 15:03 |
jnielsen55 | sorry, that question didn't make sense | 15:03 |
jnielsen55 | too early | 15:03 |
*** saripurigopi has joined #openstack-ironic | 15:03 | |
lucasagomes | rloo, ack I will fix that patch | 15:04 |
rloo | thx lucasagomes | 15:04 |
jnielsen55 | Is it possible to use an external dhcp server with Ironic? We would like to continue using vlans and not move to a flat network | 15:04 |
lucasagomes | to keep returning the same as before, so we can think about the 409 as an isolated problem | 15:04 |
lucasagomes | rloo, thank you! | 15:04 |
*** jcoufal has quit IRC | 15:04 | |
rloo | lucasagomes: just goes to show, when I'm lazy and don't do a thorough review, should have caught that sooner... | 15:05 |
lucasagomes | rloo, it's all good. Better late than never | 15:05 |
rloo | lucasagomes: true! | 15:05 |
*** dlpartain has joined #openstack-ironic | 15:05 | |
rloo | lucasagomes: actually, maybe not true. never is good too :) | 15:06 |
lucasagomes | heh | 15:06 |
lucasagomes | yeah depending on the context | 15:06 |
TheJulia | jnielsen55: in standalone mode, it should work just fine | 15:08 |
*** e0ne is now known as e0ne_ | 15:09 | |
TheJulia | jnielsen55: https://review.openstack.org/#/c/187415/ might provide some insight | 15:09 |
*** e0ne_ is now known as e0ne | 15:09 | |
NobodyCam | jnielsen55: also http://docs.openstack.org/developer/ironic/deploy/install-guide.html#using-ironic-as-a-standalone-service | 15:11 |
TheJulia | ++ | 15:11 |
*** yuikotakada has joined #openstack-ironic | 15:12 | |
jnielsen55 | thanks! | 15:13 |
openstackgerrit | Sam Betts proposed openstack/ironic: Fix flake8 E123/6/7/8 in ironic/api https://review.openstack.org/186458 | 15:13 |
openstackgerrit | Sam Betts proposed openstack/ironic: Fix flake8 E123/6/7/8 in ironic/common https://review.openstack.org/186456 | 15:13 |
openstackgerrit | Sam Betts proposed openstack/ironic: Fix flake8 E123/6/7/8 in ironic /cmd /db and /dhcp https://review.openstack.org/186457 | 15:13 |
openstackgerrit | Sam Betts proposed openstack/ironic: Fix flake8 E123/6/7/8 for ironic/tests/drivers https://review.openstack.org/186450 | 15:13 |
openstackgerrit | Sam Betts proposed openstack/ironic: Fix flake8 E123/6/7/8 in non-driver ironic/tests https://review.openstack.org/186451 | 15:13 |
openstackgerrit | Sam Betts proposed openstack/ironic: Fixes flake8 E123/6/7/8 for ironic/drivers https://review.openstack.org/186454 | 15:13 |
openstackgerrit | Sam Betts proposed openstack/ironic: Remove E123/6/7/8 from flake8 ignore list https://review.openstack.org/186021 | 15:13 |
openstackgerrit | Sam Betts proposed openstack/ironic: Fixes flake8 E123/6/7/8 in ironic/conductor https://review.openstack.org/186455 | 15:13 |
openstackgerrit | Sam Betts proposed openstack/ironic: Fix flake8 E123/6/7/8 for ironic/tests/api https://review.openstack.org/186452 | 15:13 |
openstackgerrit | Sam Betts proposed openstack/ironic: Fixes flake8 E123/6/7/8 for ironic/tests/conductor https://review.openstack.org/186453 | 15:13 |
sambetts | sorry for the spam :( | 15:13 |
NobodyCam | ieek | 15:13 |
NobodyCam | :-p | 15:13 |
NobodyCam | lol | 15:13 |
TheJulia | jnielsen55: no problem, let us know if you have any more questions | 15:14 |
*** ijw has joined #openstack-ironic | 15:18 | |
*** dlpartain has quit IRC | 15:18 | |
dtantsur | see you tomorrow | 15:20 |
*** dtantsur is now known as dtantsur|afk | 15:20 | |
sambetts | dtantsur|afk: Cya! | 15:23 |
*** ijw has quit IRC | 15:24 | |
lucasagomes | dtantsur|afk, see ya | 15:27 |
lucasagomes | g'night | 15:27 |
*** krtaylor has quit IRC | 15:27 | |
rloo | hey lucasagomes: maybe we could make the switch409->400 via a microversion: https://bugs.launchpad.net/ironic/+bug/1461140 | 15:28 |
openstack | Launchpad bug 1461140 in Ironic "conflict (HTTP 409) incorrect for some cases" [Undecided,New] | 15:28 |
lucasagomes | rloo, right, yeah but that would need to switch the instance already associated as well right? | 15:28 |
lucasagomes | which I don't wanna to do as part of this patch | 15:29 |
rloo | lucasagomes: oh, don't do it in your patch. that's what the bug is for -- should do 'it all' in one go I think. | 15:29 |
lucasagomes | rloo, perhaps on a following patch we can fix the return codes for all cases in deletion? | 15:29 |
lucasagomes | oh | 15:29 |
lucasagomes | gotcha | 15:29 |
lucasagomes | yeah via microversion seems good | 15:29 |
rloo | lucasagomes: cuz I suspect it is in more places than we know right now. | 15:29 |
lucasagomes | yeah possibly | 15:30 |
openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Prevent node delete based on provision, not power, state https://review.openstack.org/186361 | 15:30 |
openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Remove bogus conditional from node_update https://review.openstack.org/186386 | 15:30 |
*** lazy_prince has joined #openstack-ironic | 15:32 | |
NobodyCam | night dtantsur|afk | 15:33 |
*** killer_prince has quit IRC | 15:35 | |
*** barra204 has joined #openstack-ironic | 15:35 | |
*** shakamunyi has joined #openstack-ironic | 15:36 | |
*** ukalifon3 has joined #openstack-ironic | 15:38 | |
*** ukalifon has quit IRC | 15:39 | |
* lucasagomes brb | 15:40 | |
*** lucasagomes has quit IRC | 15:40 | |
*** lucasagomes has joined #openstack-ironic | 15:42 | |
*** openstackgerrit has quit IRC | 15:42 | |
*** Nisha has joined #openstack-ironic | 15:43 | |
*** openstackgerrit has joined #openstack-ironic | 15:43 | |
*** eghobo has joined #openstack-ironic | 15:44 | |
*** absubram has joined #openstack-ironic | 15:45 | |
rameshg87 | good night everyone | 15:48 |
*** rameshg87 has quit IRC | 15:48 | |
* NobodyCam eats a begal | 15:48 | |
NobodyCam | niht rameshg87 | 15:49 |
NobodyCam | night even | 15:49 |
*** [1]cdearborn has quit IRC | 15:49 | |
trown | NobodyCam: do beagles make good eating? | 15:54 |
NobodyCam | doh | 15:54 |
NobodyCam | lol | 15:54 |
NobodyCam | bagels not beagles!!! | 15:55 |
NobodyCam | lol | 15:55 |
* BadCub does nto think beagles would be a good breakfast item | 15:55 | |
jroll | D: | 15:55 |
* jroll keeps his beagle away from NobodyCam | 15:55 | |
NobodyCam | lol | 15:55 |
*** athomas has joined #openstack-ironic | 15:55 | |
NobodyCam | morning jroll :) | 15:55 |
BadCub | hiya jroll | 15:55 |
jroll | mornin' folks :) | 15:56 |
*** yuikotakada has quit IRC | 15:58 | |
lucasagomes | jroll, BadCub morning! | 16:01 |
jroll | \o | 16:01 |
lucasagomes | rloo, question. Why you think that moving the vendor methods to the deploy (or other interfaces) will change the endpoint? | 16:01 |
lucasagomes | I didn't get that from the sessions | 16:01 |
rloo | lucasagomes: cuz that was the whole idea/diff between vendor_passthru endpoints, wasn't it? | 16:02 |
rloo | lucasagomes: the passthru endpoints were for vendors to do whatever | 16:02 |
lucasagomes | rloo, I think the idea was to leave the vendor interface only for vendor drivers | 16:02 |
lucasagomes | but still using vendor_passthru to expose driver specific methods that are need for certain deployments | 16:02 |
lucasagomes | that would allow us to not having to use those VendorMixIn interfaces | 16:03 |
lucasagomes | but yeah I get what you're saying. The "vendor" in the ednpoint seems misleading | 16:03 |
rloo | lucasagomes: that doesn't make sense to me but maybe it is just me | 16:03 |
lucasagomes | perhaps if we called it "passthru" and "driver_passhtru" | 16:03 |
lucasagomes | that would be easier | 16:03 |
*** krtaylor has joined #openstack-ironic | 16:04 | |
rloo | lucasagomes: hmm, even if we called it 'passthru'. it means we're just allowing some method to be passed through. | 16:04 |
lucasagomes | rloo, avoid mixin helps with the mix and matches for composing drivers, to simplify the matrix | 16:04 |
lucasagomes | rloo, yeah, means it's specific for a driver so we don't have a standard api for that | 16:05 |
rloo | lucasagomes: I thought if we promoted/had a method in DeployInstance, it would be like other 'normal' methods, you'd do /v1/node/<uuid>/NEW-SOMETHING | 16:05 |
*** jistr has quit IRC | 16:05 | |
lucasagomes | right, yeah I think for things like heartbeat and lookup it makes sense once IPA is the default and only deploy ramdisk supported | 16:05 |
lucasagomes | but things for iPXE to auto generate configs still needs to go to a passhtru | 16:05 |
lucasagomes | cause it's a driver specific | 16:06 |
lucasagomes | doesn't make sense for any other driver | 16:06 |
rloo | lucasagomes: maybe i'm mixing up the different passthru methods. | 16:06 |
lucasagomes | right, or I also didn't fully grasp what was the agreement on that session | 16:07 |
rloo | lucasagomes: let me see if i can find any notes on what we were going to promote | 16:07 |
lucasagomes | rloo, ack | 16:07 |
rloo | lucasagomes: in deva's email: | 16:08 |
rloo | we will be promoting the "heartbeat" / "lookup" / | 16:08 |
rloo | "pass_deploy_info" methods, which are currently implemented in | 16:08 |
rloo | VendorPassthru, to become a standard part of the Deploy() interface. | 16:08 |
rloo | It turns out every deploy driver we've got implements this | 16:08 |
rloo | functionality, and promoting common interfaces out of VendorPassthru | 16:08 |
rloo | is one of the reasons that interface exists :) | 16:08 |
rloo | lucasagomes: ugh, pasting isn't the way to go. anyway, my impression was that we were going to promote those three into one/same method in DeployInterface. | 16:09 |
lucasagomes | right, driver interfaces but I don't think we are chaging the API endpoint for those either | 16:09 |
lucasagomes | well we can clarify with devananda when he's around | 16:09 |
rloo | lucasagomes: +1 for clarification! | 16:10 |
lucasagomes | and I'm -1 for the pass_deploy_info... I think we should deprecate the bash ramdisk soonish | 16:10 |
lucasagomes | like this cycle | 16:10 |
* lucasagomes want to know what others think about it | 16:11 | |
*** leopoldj has quit IRC | 16:11 | |
*** dprince has quit IRC | 16:18 | |
*** ijw has joined #openstack-ironic | 16:18 | |
openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Wake-On-Lan Power interface https://review.openstack.org/179078 | 16:19 |
jlvillal | lucasagomes: Did you convince "He-Who-Must-Not-Be-Named" on the requirements version bump? ;) | 16:19 |
lucasagomes | jlvillal, I pinged dhellman in the wsme channel | 16:19 |
lucasagomes | jlvillal, he might be busy and I didn't get an answer but seems that more wsme cores agrees with the bump | 16:20 |
lucasagomes | Julien which +2'd the patch in requirements, myself and cdent | 16:20 |
lucasagomes | so... | 16:20 |
jlvillal | lucasagomes: Great. I did see it get merged | 16:20 |
*** ukalifon3 has quit IRC | 16:20 | |
lucasagomes | I mean we are fixing a bunch of things that needs a bump IMHO. 1) The dependency, 2) serving static files, 3) Some ReST stuff like content-type matching that was broken | 16:21 |
lucasagomes | jlvillal, I will re-review ur patch reverting it | 16:21 |
jlvillal | lucasagomes: I didn't know about the rest of the fixes. I just knew about the dependency. | 16:21 |
jlvillal | lucasagomes: I think my patch needs a recheck. Every tempest job fails??? Previous patch worked and all I did was add the 'Depends-On' to a patch which is already merged. | 16:22 |
lucasagomes | yeah, I know cause know I have to review WSME patches heh | 16:22 |
lucasagomes | jlvillal, oh, you might need to add a change to the requirements as well | 16:22 |
lucasagomes | the bot might send a patch to ironic updating it | 16:23 |
lucasagomes | but if your patch already depends on 0.7 you might want to go first and bump it in Ironic as well | 16:23 |
jlvillal | lucasagomes: I don't think it fails the job though on requirements. | 16:23 |
jlvillal | lucasagomes: Unless the patch is changing requirements. | 16:23 |
lucasagomes | right, I have to look at the logs | 16:24 |
*** ijw has quit IRC | 16:24 | |
jlvillal | lucasagomes: I see 'timed out waiting for...' | 16:24 |
lucasagomes | nothing on the ir-{api, cond} logs? | 16:25 |
lucasagomes | or py27 | 16:25 |
* lucasagomes looks | 16:25 | |
jlvillal | lucasagomes: http://logs.openstack.org/08/186208/5/check/check-tempest-dsvm-ironic-pxe_ssh/864dc92/console.html | 16:25 |
jlvillal | lucasagomes: Search for: Failed 24 tests | 16:25 |
lucasagomes | 2015-06-02 16:02:20.360 9112 ERROR oslo_messaging._drivers.impl_rabbit NoSuchOptError: no such option: rpc_response_timeout | 16:25 |
lucasagomes | in the ir-cond logs | 16:25 |
lucasagomes | not sure how is that related tho | 16:26 |
jlvillal | lucasagomes: Me either... | 16:27 |
*** e0ne has quit IRC | 16:29 | |
jlvillal | Does anyone know what Ironic is doing in regards to versioned objects? I didn't see anything in the whiteboard or specs. | 16:32 |
NobodyCam | TheJulia: are these normal? seems an odd patch to error on: http://logs.openstack.org/15/187415/1/check/gate-bifrost/aae3102/console.html#_2015-06-02_14_53_52_014 | 16:32 |
*** Nisha has quit IRC | 16:33 | |
*** dprince has joined #openstack-ironic | 16:36 | |
*** absubram has quit IRC | 16:38 | |
*** absubram has joined #openstack-ironic | 16:38 | |
openstackgerrit | Clif Houck proposed openstack/ironic: Add support for image caching for the agent driver https://review.openstack.org/161832 | 16:42 |
jlvillal | NobodyCam: Do you know about Ironic and versioned objects? What the plan is? | 16:42 |
NobodyCam | humm, i'm trying to recall we talked about at the summit | 16:43 |
jlvillal | NobodyCam: I also vaguely remember something about versioned objects but I forget when and where I heard it :( | 16:44 |
NobodyCam | jlvillal: I'm thinking we waiting for something in oslo to release. | 16:46 |
*** lazy_prince has quit IRC | 16:46 | |
jlvillal | NobodyCam: Okay. I got a question from a co-worker wondering if Ironic was going to use Oslo versioned objects. Or if Ironic already did. | 16:47 |
NobodyCam | jlvillal: I'm not 100% sure (maybe somebody can correct me) but I thought we wanted to use the oslo version | 16:48 |
jlvillal | lucasagomes: ^^^^ Do you remember anything on Oslo versioned objects? | 16:48 |
TheJulia | NobodyCam: looking | 16:49 |
TheJulia | NobodyCam: they are fairly normal from the os that the job is running on. Interesting failure in that it couldn't lock the package database. Interesting... | 16:52 |
rloo | jlvillal: my understanding is that ironic took a copy of the objects from nova, and the oslo version is from the nova version and we should switch to use the oslo version | 16:52 |
jlvillal | rloo:Thanks :) | 16:52 |
rloo | jlvillal: so we are using versioned objects and *hopefully* it will be straightforward to migrate. | 16:52 |
* lucasagomes reads | 16:52 | |
lucasagomes | jlvillal, I didn't go to the session in the summit because it was colliding with something else (I think the API work group) | 16:53 |
rloo | lucasagomes: were you planning on doing the migration? that was something i wanted to ask you cuz you had mentioned it before | 16:53 |
lucasagomes | jlvillal, but I think it's high desirable that Ironic starts using the oslo objects instead of the copy from nova that we currently have in tree | 16:53 |
TheJulia | NobodyCam: so the ansible folks landed some nasty changes that may have additional impact. Looks like the first issue may be resolved, but this is additional. Logging into my test box now to poke/prod/investigate | 16:53 |
*** Nisha has joined #openstack-ironic | 16:54 | |
lucasagomes | rloo, I'm def +1 with the work. But, I haven't registered any spec or anything about it | 16:54 |
lucasagomes | if someone else wants to do, I'm more than glad to review the work | 16:54 |
NobodyCam | TheJulia: :) | 16:54 |
rloo | lucasagomes: shouldn't need a spec. i think a bug is fine. | 16:54 |
lucasagomes | yeah or a bug | 16:54 |
rloo | lucasagomes: unless it is a drastic change | 16:54 |
jlvillal | lucasagomes, rloo, and NobodyCam: Thanks for the info | 16:54 |
lucasagomes | np | 16:54 |
*** diegolp has quit IRC | 16:55 | |
*** ifarkas has quit IRC | 16:56 | |
openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic-specs: Wake-On-Lan (WOL) power driver https://review.openstack.org/187678 | 16:56 |
lucasagomes | a couple of people have to be using the WOL driver in their home labs (got 2 emails bout it), so I stopped arsing about and wrote a spec | 16:57 |
lucasagomes | if folks have a time please take a look ^, it should be straight forward and small | 16:57 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic: Updated from global requirements https://review.openstack.org/187679 | 16:58 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic-python-agent: Updated from global requirements https://review.openstack.org/187680 | 16:58 |
TheJulia | NobodyCam: I'm thinking https://review.openstack.org/#/c/187532/ I'm getting yet another different error now with restarting services with the ansible devel branch. | 16:58 |
lucasagomes | jlvillal, ^ | 16:59 |
lucasagomes | try to add ur patch on top of that see if it works with 0.7 | 16:59 |
jlvillal | lucasagomes: Okay | 17:00 |
openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Wake-On-Lan Power interface https://review.openstack.org/179078 | 17:00 |
* lucasagomes updates the commit message to include the bp | 17:00 | |
*** trown is now known as trown|lunch | 17:01 | |
*** dguerri is now known as dguerri`away | 17:04 | |
TheJulia | So I will <3 any cores that review https://review.openstack.org/#/c/187532/ and https://review.openstack.org/#/c/186582/2 and https://review.openstack.org/#/c/186581/2 | 17:06 |
*** derekh has quit IRC | 17:06 | |
jlvillal | lucasagomes: FYI: I see other Ironic jobs failing all the tempest jobs. For example: https://review.openstack.org/#/c/186450/ | 17:08 |
*** achanda has joined #openstack-ironic | 17:08 | |
lucasagomes | jlvillal, :-( gope our gate is not broken | 17:08 |
lucasagomes | hope | 17:08 |
jlvillal | lucasagomes: Me too. | 17:08 |
jroll | sure looking like it :/ | 17:09 |
*** samueldmq has joined #openstack-ironic | 17:09 | |
samueldmq | hello | 17:10 |
NobodyCam | so TheJulia would you kill me if I asked for a # NOTE to https://review.openstack.org/#/c/187532/1/scripts/env-setup.sh as to why we are pulling stable? | 17:10 |
TheJulia | greetings | 17:10 |
TheJulia | NobodyCam: never | 17:10 |
samueldmq | does ironic use the a policy system as the other services ? | 17:10 |
NobodyCam | good (UGT) morning samueldmq | 17:10 |
jlvillal | lucasagomes: I'm asking over in infra to see if they know anything about tempest jobs. | 17:10 |
samueldmq | NobodyCam, morning | 17:10 |
samueldmq | :) | 17:10 |
TheJulia | NobodyCam: give me 2-3 minutes and I'll push a note up | 17:10 |
openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic-specs: Wake-On-Lan (WOL) power driver https://review.openstack.org/187678 | 17:10 |
lucasagomes | jlvillal, ack | 17:10 |
NobodyCam | samueldmq: like this: https://github.com/openstack/ironic/blob/master/etc/ironic/policy.json | 17:11 |
TheJulia | NobodyCam: make that 3-5... lost connectivity to my work laptop | 17:11 |
lucasagomes | samueldmq, we are an admin-only API | 17:11 |
*** spandhe has joined #openstack-ironic | 17:11 | |
lucasagomes | but yea, we use the policy mechanism | 17:11 |
samueldmq | NobodyCam, lucasagomes I'd like to know if you use oslo.policy | 17:11 |
lucasagomes | (our is just very simple) | 17:11 |
jroll | jlvillal: new oslo.messaging 2 hours ago https://github.com/openstack/oslo.messaging/releases | 17:11 |
lucasagomes | samueldmq, we do | 17:11 |
samueldmq | https://bugs.launchpad.net/nova/+bug/1458945 | 17:11 |
openstack | Launchpad bug 1458945 in murano "Use graduated oslo.policy instead of oslo-incubator code" [High,Confirmed] - Assigned to Ekaterina Chernova (efedorova) | 17:11 |
lucasagomes | requirements.txt:oslo.policy>=0.3.1 # Apache-2.0 | 17:12 |
samueldmq | rloo marked this as 'no longer affects', I'd like to know if you guys already have implemneted support for oslo.policy | 17:12 |
rloo | samueldmq: yup, we're already using oslo.policy | 17:12 |
samueldmq | and if you would have a link for that, so we can keep track | 17:12 |
jlvillal | lucasagomes: Is the oslo.messaging likely causing the gate failure? | 17:12 |
lucasagomes | samueldmq, https://review.openstack.org/#/c/162501/ | 17:12 |
jlvillal | lucasagomes: fungi, said that new images were pushed out about two hours ago. fungi is investigating. | 17:12 |
lucasagomes | jlvillal, I haven't looked deeply on the logs | 17:12 |
lucasagomes | so I don't know, it seems the errors were somehow related to oslo.message | 17:13 |
lucasagomes | but can't say for sure | 17:13 |
samueldmq | lucasagomes, rloo could one of you guys mark that as 'Fix Released' ? | 17:13 |
jlvillal | lucasagomes: from fungi: 'i see a neutron functional job failing on a specific test (UnprivilegedUserMetadataL3AgentTestCase.test_access_to_metadata_proxy)' | 17:14 |
jroll | jlvillal: lucasagomes the option certainly still exists: https://github.com/openstack/oslo.messaging/blob/097fb235b2a2de216d591ea9cf674b7ceeab4b06/oslo_messaging/rpc/client.py#L35 | 17:14 |
jroll | jlvillal: lucasagomes so maybe it moved and we need an import_opt or something | 17:14 |
lucasagomes | samueldmq, ack | 17:14 |
openstackgerrit | Julia Kreger proposed openstack/bifrost: Lock Bifrost to Ansible stable-1.9 https://review.openstack.org/187532 | 17:14 |
samueldmq | lucasagomes, perfect ! thanks | 17:14 |
jroll | jlvillal: lucasagomes actually this looks like an oslo.messaging bug, the usage is in their internals. lame | 17:14 |
TheJulia | NobodyCam: ^^^ | 17:14 |
samueldmq | lucasagomes, rloo have a nice day ! :) | 17:14 |
rloo | samueldmq: done. but it was released in kilo and that isnt' a choice | 17:15 |
lucasagomes | jroll, oh ew... well so people will either fix and release a new version | 17:15 |
lucasagomes | or revert the change idk | 17:15 |
jroll | yeah | 17:15 |
samueldmq | rloo, yeah, that's just a matter of tracking that change on openstack projects, thanks | 17:15 |
lucasagomes | samueldmq, you too | 17:15 |
lucasagomes | samueldmq, it's all good, thanks for that! | 17:16 |
samueldmq | lucasagomes, thanks :) | 17:16 |
lucasagomes | jroll, jlvillal are you guys on top of that problem? | 17:18 |
lucasagomes | it's a bit late here so I will take a break | 17:18 |
jroll | lucasagomes: dims has been pinged | 17:18 |
jroll | yeah don't wait on us :P | 17:18 |
lucasagomes | heh fair | 17:18 |
jlvillal | lucasagomes: I can pay attention. You should leave work! | 17:18 |
*** ijw has joined #openstack-ironic | 17:18 | |
lucasagomes | thanks folks | 17:18 |
jlvillal | lucasagomes: Thank you! | 17:18 |
lucasagomes | so I will call it a day :-) | 17:18 |
jroll | later dude | 17:19 |
lucasagomes | have a good night everyone, see y'all tomorrow | 17:19 |
*** lucasagomes is now known as lucas-dinner | 17:19 | |
openstackgerrit | Ruby Loo proposed openstack/ironic: Update the config drive doc to replace deprecated value https://review.openstack.org/187534 | 17:19 |
sambetts | night lucas-dinner! | 17:23 |
*** ijw has quit IRC | 17:24 | |
NobodyCam | night lucas-dinner | 17:24 |
*** bethelwell has quit IRC | 17:30 | |
*** pas-ha has quit IRC | 17:31 | |
*** e0ne has joined #openstack-ironic | 17:32 | |
*** mtanino has quit IRC | 17:33 | |
jlvillal | jroll: FYI: dhellman is looking into the oslo.messaging issue. I asked the question about it in #openstack-oslo and he responded. | 17:34 |
jroll | jlvillal: sweet. I assume it's something we're doing with it or else nova would also be dead in the water | 17:35 |
jlvillal | jroll: Oh :( | 17:35 |
jroll | I mean, I could be wrong | 17:35 |
jlvillal | jroll: In this one case, I hope you are ;) | 17:35 |
rloo | hey JoshNang, are you around to answer questions about zapping? | 17:36 |
*** romainh1 has left #openstack-ironic | 17:37 | |
openstackgerrit | Julia Kreger proposed openstack/bifrost: Lock Bifrost to Ansible stable-1.9 https://review.openstack.org/187532 | 17:44 |
openstackgerrit | Julia Kreger proposed openstack/bifrost: WIP: Support for disabling DHCP server configuration https://review.openstack.org/187362 | 17:44 |
openstackgerrit | Julia Kreger proposed openstack/bifrost: Make Ansible modules 1.9 compatible https://review.openstack.org/186582 | 17:44 |
openstackgerrit | Julia Kreger proposed openstack/bifrost: Preserve logs in logs/ folder https://review.openstack.org/186581 | 17:44 |
NobodyCam | brb | 17:47 |
*** romcheg1 has quit IRC | 17:47 | |
*** Nisha has quit IRC | 17:50 | |
*** romcheg has joined #openstack-ironic | 17:50 | |
TheJulia | oh git review... why did you rebase everything | 17:51 |
*** pelix has quit IRC | 17:51 | |
openstackgerrit | Doug Hellmann proposed openstack/ironic: test without oslo.messaging 1.12.0 https://review.openstack.org/187699 | 17:51 |
*** e0ne is now known as e0ne_ | 17:52 | |
jlvillal | I have updated the whiteboard that the Ironic gate appears broken. Bug filed: https://bugs.launchpad.net/ironic/+bug/1461182 | 17:53 |
openstack | Launchpad bug 1461182 in oslo.messaging "After oslo.messaging release Ironic gate is broken" [Undecided,New] | 17:53 |
*** dhellmann has joined #openstack-ironic | 17:53 | |
dhellmann | jlvillal: ping? | 17:53 |
jlvillal | dhellmann: pong | 17:54 |
dhellmann | jlvillal: I'm looking at http://logs.openstack.org/08/186208/5/check/check-tempest-dsvm-ironic-pxe_ssh/864dc92/logs/screen-ir-cond.txt.gz?level=INFO and I see some "Successfully deleted node" messages -- do those come as the result of an inbound rpc request? | 17:54 |
*** romcheg1 has joined #openstack-ironic | 17:54 | |
jlvillal | dhellmann: I will defer to jroll or rloo | 17:54 |
dhellmann | I don't really know how ironic works internally, so I'm trying to understand exactly what's failing in those tests | 17:54 |
jroll | dhellmann: yes, it does | 17:54 |
jroll | dhellmann: nearly(?) any conductor action is the result of an rpc call | 17:54 |
dhellmann | so are the error messages from outbound calls? | 17:54 |
*** edcragg has quit IRC | 17:55 | |
*** romcheg has quit IRC | 17:55 | |
jroll | hard to tell, since the traceback isn't including the ironic bits that triggered it | 17:55 |
jroll | dhellmann: might be when trying to respond to a call | 17:56 |
jroll | synchronous call* | 17:56 |
*** e0ne_ is now known as e0ne | 17:57 | |
* dhellmann may not be the best person to debug this one | 17:58 | |
*** romcheg1 has quit IRC | 17:58 | |
dhellmann | yeah, the traceback isn't especially helpful there | 17:58 |
openstackgerrit | Chris Krelle proposed openstack/bifrost: Update test-birfrost to limit log lines https://review.openstack.org/187703 | 17:58 |
NobodyCam | TheJulia: ^^^^^ | 17:59 |
dhellmann | jroll: yeah, looks like it's an issue with sending the reply, based on where it is in the messaging code | 17:59 |
TheJulia | doh! | 17:59 |
NobodyCam | :-p | 17:59 |
TheJulia | looks like I may need to re-stack those changes | 18:00 |
* TheJulia waits and sees a little bit | 18:01 | |
NobodyCam | brb | 18:01 |
TheJulia | yeah, I'm going to have to re-stack them :( | 18:04 |
*** athomas has quit IRC | 18:04 | |
*** ukalifon1 has joined #openstack-ironic | 18:06 | |
TheJulia | NobodyCam: http://logs.openstack.org/32/187532/3/check/gate-bifrost/449de85/console.html#_2015-06-02_18_04_31_008 :( | 18:07 |
jroll | TheJulia: that looks like the same bug the rest of our gate is hitting | 18:08 |
TheJulia | :( | 18:08 |
TheJulia | That is what I was fearing | 18:08 |
jroll | that's a good thing, it means other people are helping track it down :P | 18:09 |
*** openstackgerrit has quit IRC | 18:09 | |
TheJulia | true | 18:10 |
*** trown|lunch is now known as trown | 18:10 | |
*** openstackgerrit has joined #openstack-ironic | 18:10 | |
*** bethelwell has joined #openstack-ironic | 18:11 | |
*** dguerri`away is now known as dguerri | 18:11 | |
TheJulia | Hmm... times out on the patch to deploy the node | 18:12 |
*** mtanino has joined #openstack-ironic | 18:14 | |
TheJulia | http://logs.openstack.org/32/187532/3/check/gate-bifrost/449de85/console.html#_2015-06-02_18_04_31_185 | 18:16 |
openstackgerrit | Doug Hellmann proposed openstack/ironic: WIP: test if having a client creates the option we need https://review.openstack.org/187713 | 18:20 |
NobodyCam | TheJulia: wow: http://logs.openstack.org/03/187703/1/check/gate-bifrost/1dcfdd2/console.html#_2015-06-02_18_04_04_841 | 18:21 |
TheJulia | yeah, that was the other job failing, why I'm pushing for us to go to ansible stable-1.9 since they are making major changes to devel right now | 18:22 |
TheJulia | I told them I'll test and give them feedback, but we need our stuff to generally work | 18:22 |
dhellmann | jroll: I have 2 patches up now to test variations of fixes within ironic (block that version of messaging, and create a client object to register the option). I think we have a bad assumption in the way that option is registered in oslo.messaging, and those should help verify my hypothesis | 18:22 |
dhellmann | jlvillal: ^^ | 18:22 |
jroll | dhellmann: awesome, thanks for digging on this, feel free to poke if you need a +2 :) | 18:23 |
jlvillal | dhellmann: Thanks for looking into this! | 18:23 |
dhellmann | jroll: well, I think the final fix will be in oslo.messaging, but it's easier to test this way because we don't have a job to run your tempest tests against oslo.messaging master | 18:23 |
jroll | dhellmann: ah, I see :) | 18:23 |
dhellmann | we run dsvm jobs, but not all of the various feature tests | 18:24 |
jroll | yeah | 18:24 |
*** thiagop has joined #openstack-ironic | 18:26 | |
clif_h | gate broke? | 18:32 |
TheJulia | yes | 18:34 |
*** ukalifon1 has quit IRC | 18:34 | |
jlvillal | clif_h: Yes :( https://etherpad.openstack.org/p/IronicWhiteBoard | 18:35 |
clif_h | ah, I should look at the whiteboard more often | 18:35 |
clif_h | thank you | 18:35 |
jlvillal | clif_h: I forget all the time :( | 18:35 |
*** achanda has quit IRC | 18:45 | |
jlvillal | JoshNang: Should ironic-python-agent have 'pbr' in requirements.txt? I don't see it in requirements.txt but see pbr being used. | 18:45 |
jlvillal | JoshNang: This failure made me start looking: https://jenkins02.openstack.org/job/gate-ironic-python-agent-pep8/7/ | 18:46 |
dhellmann | jroll, jlvillal : do either of you have a way to run those tempest tests with an oslo.messaging patch? there's no CI job configuration for that, but if you could do it by hand with https://review.openstack.org/187722 that would help | 18:51 |
*** alexpilotti has quit IRC | 18:52 | |
jlvillal | dhellmann: I don't know how to do that. Though I am meeting adam_g later today to learn about tempest. As I am going to take over the functional testing lead role. | 18:53 |
openstackgerrit | Jarrod Johnson proposed stackforge/pyghmi: Implement OEM Inventory components https://review.openstack.org/187592 | 18:53 |
jlvillal | dhellmann: Hopefully after the meeting I will know more. | 18:53 |
dhellmann | jlvillal: ok. I don't know what's needed on a test system for ironic, so I'm not sure how to do it either. I thought one of the ironic devs might already be set up for a test like that. | 18:54 |
*** openstackgerrit has quit IRC | 18:56 | |
adam_g | i dont think you'd need to mess with tempest for that. looks like a simple devstack+ironic install should be broken and non-functional at this point | 18:56 |
*** openstackgerrit has joined #openstack-ironic | 18:56 | |
dhellmann | adam_g: oslo.messaging master has a job to run the app source against proposed changes to ensure we don't break there, but maybe that gate isn't using ironic? | 18:57 |
adam_g | dhellmann, no, i cant imagine it'd be testing ironic. the ironic gate will just use the oslo.messaging release from pip. if it were setup to install that from src we could use the new gerrit cross-repo dependency thing to test it | 18:59 |
adam_g | ill see if i can spin one up locally and poke | 18:59 |
dhellmann | adam_g: cool, thanks. theoretically this would be covered by functional tests inside oslo.messaging, but that test suite isn't very big yet | 18:59 |
adam_g | dhellmann, not knowing anything about the bug yet, would this be better solved by a failing test case being added to oslo.messaging and working there? or is the root cause still unknown? | 19:01 |
dhellmann | adam_g: the root cause isn't clear; I *think* that patch in oslo.messaging should fix it, and wanted to get it into someone's hands here for testing quickly. If it does, we'll add more tests to oslo.messaging as we land the patch. | 19:02 |
adam_g | dhellmann, cool. spinning something up now, should be able to test | 19:02 |
dhellmann | adam_g: awesome, thanks | 19:03 |
*** achanda has joined #openstack-ironic | 19:03 | |
* devananda skims scrollback | 19:13 | |
devananda | if I started up ironic-api & ironic-conductor in a venv, should that be broken // able to duplicate the issue? | 19:13 |
adam_g | devananda, yeah, probably | 19:18 |
*** cdearborn has joined #openstack-ironic | 19:21 | |
jlvillal | dhellmann, Your patch: https://review.openstack.org/#/c/187713/ Did work | 19:22 |
dhellmann | jlvillal: ok, that gives me faith that the patch to oslo.messaging will also fix the issue, though it's going to be more difficult to test that one. adam_g is working on that | 19:22 |
devananda | adam_g: ok - so it seems to work fine | 19:23 |
dhellmann | devananda: the issue is within the conductor itself. I think ironic-conductor may be the only service that doesn't also use a Client object, so the option that's defined in oslo.messaging's client code but used in the server-side code isn't registered | 19:23 |
devananda | I'm using tip of master for ironic's code | 19:24 |
devananda | oslo.messaging==1.12.0 | 19:24 |
dhellmann | devananda: are you saying the messaging lib patch works, or that conductor works without it? | 19:24 |
devananda | I'm trying to reproduce the problem right now, and have not yet reproduced it | 19:24 |
dhellmann | k | 19:24 |
cdearborn | hey guys - trying to troubleshoot ironic-discoverd issue. ramdisk exits to shell. 400 error, "/run/initramfs/rdsosreport.txt does not exist". It's working on 2 out of 3 nodes. Any thoughts? | 19:25 |
jlvillal | cdearborn, dtantsur|afk is the expert on that. Not sure who is is that is online. | 19:25 |
jlvillal | s/is is/else is/ | 19:26 |
*** achanda has quit IRC | 19:26 | |
cdearborn | jlvillal, thx | 19:26 |
jlvillal | cdearborn, I think he is on Central European Time. Or at least close to that. | 19:27 |
devananda | dhellmann: ok - reproduced it | 19:28 |
cdearborn | jlvillal, k - will try tomorrow AM EDT | 19:28 |
*** romcheg has joined #openstack-ironic | 19:29 | |
devananda | dhellmann: https://review.openstack.org/#/c/187713/1 also fixes it locally | 19:30 |
dhellmann | devananda: yep, though that's even more hacky than https://review.openstack.org/187722 :-) | 19:30 |
dhellmann | it would be good to have a way to test ^^, because if that fixes it we can land it and get a new release out quickly this afternoon | 19:31 |
devananda | dhellmann: I can do a local build and install in my venv | 19:31 |
devananda | dhellmann: also, fwiw, I think ironic-conductor should be creating a Client object. <<< JoshNang, isn't that required for some of the work you did / are doing anyway? | 19:32 |
dhellmann | cool, thanks. If we land that and cut a new release, I can work with sileht tomorrow to add more testing to the lib to prevent a recurrance | 19:32 |
dhellmann | devananda: ok, that's fine, it just shouldn't be needed to reply to messages | 19:32 |
devananda | dhellmann: fair enough :) | 19:33 |
*** mtanino has quit IRC | 19:33 | |
*** romcheg has quit IRC | 19:34 | |
*** elmiko has joined #openstack-ironic | 19:36 | |
devananda | dhellmann: good news - that worked | 19:36 |
devananda | oslo.messaging==1.12.1.dev1 | 19:36 |
elmiko | lucas-dinner: hey, ping me when you get back | 19:36 |
dhellmann | devananda: wonderful, we'll land it and get a release cut asap | 19:36 |
devananda | cheers. I'm gonna get lunch. will be around // ping me here and I can hop back online | 19:37 |
*** bethelwell has quit IRC | 19:38 | |
NobodyCam | enjoy lunch devananda | 19:44 |
rloo | jlvillal: i opened a bug about oslo.versionedobjects https://bugs.launchpad.net/ironic/+bug/1461239 | 19:45 |
openstack | Launchpad bug 1461239 in Ironic "replace objects with oslo.versionedobjects" [Low,New] | 19:45 |
*** romcheg has joined #openstack-ironic | 19:46 | |
*** mtanino has joined #openstack-ironic | 19:48 | |
*** natorious is now known as zz_natorious | 19:53 | |
*** dtantsur|afk is now known as dtantsur | 19:57 | |
dtantsur | cdearborn, I'm sometimes around :) | 19:57 |
*** jasonamyers has left #openstack-ironic | 19:58 | |
dtantsur | cdearborn, missing /run/initramfs/rdsosreport.txt is not an issue, have a look at discoverd service logs first | 19:58 |
*** spandhe has quit IRC | 19:59 | |
jlvillal | rloo: Thanks | 20:03 |
*** samueldmq has quit IRC | 20:05 | |
trown | cdearborn: et al, I am also able to field discoverd questions during US Eastern Time when dtantsur is not on way too late :p | 20:07 |
dtantsur | trown, thanks! | 20:08 |
* dtantsur hides again | 20:08 | |
*** zz_natorious is now known as natorious | 20:10 | |
*** openstackgerrit has quit IRC | 20:10 | |
*** openstackgerrit has joined #openstack-ironic | 20:10 | |
cdearborn | dtantsur, just took a look at the discoverd log & see "InternalServerError: Remote error: DBError (DataError) (1406, "Data too long for column 'extra' at row 1") 'UPDATE nodes SET updated_at=%s, properties=%s, extra=%s WHERE nodes.id = %s'" | 20:11 |
trown | ruh roh | 20:12 |
trown | cdearborn: that is a bug | 20:12 |
cdearborn | the string it's trying to insert is HUGE!!! | 20:12 |
trown | cdearborn: ya, I am to blame for that | 20:12 |
trown | thats what happens when you only test things on virt envs | 20:13 |
cdearborn | trown, is the issue just that the column needs to be wider? | 20:13 |
cdearborn | or a glob or something? | 20:13 |
* dtantsur facepalms | 20:13 | |
trown | cdearborn: that would be a stop gap...really we need to store that in swift | 20:13 |
dtantsur | ++ for swift | 20:13 |
dtantsur | cdearborn, disable 'edeploy' plugin if it's enabled in the conf | 20:14 |
dtantsur | you're unlikely to use it tbh | 20:14 |
cdearborn | trown, dtantsur, would i find that setting in ironic.conf? | 20:17 |
dtantsur | trown, could you create a bug to move edeploy plugin to swift? nice time to do it... (and rename it from edeploy probably) | 20:17 |
dtantsur | cdearborn, in discoverd.conf, likely to be in /etc/ironic-discoverd | 20:18 |
trown | dtantsur: on it already :) | 20:18 |
*** ppal has joined #openstack-ironic | 20:18 | |
cdearborn | dtantsur, trown, just remove "edeploy" from processing_hooks? | 20:19 |
dtantsur | yep, and restart | 20:19 |
trown | dtantsur, I have been looking at more actual baremetal envs recently, and seeing how much is in the extra column...was pretty sure this was going to be a problem | 20:20 |
trown | that is without even running the extended benchmarks | 20:21 |
cdearborn | dtantsur, trown, would you like any info from the log before I do too much? | 20:21 |
trown | cdearborn: no need | 20:21 |
trown | I put the DB error in the bug for posterity sake | 20:22 |
cdearborn | trown, aok - can you send me the bug number when you have a min? | 20:22 |
trown | https://bugs.launchpad.net/ironic-discoverd/+bug/1461252 | 20:22 |
openstack | Launchpad bug 1461252 in ironic-inspector "edeploy plugin puts too much data in Ironic extra column" [Undecided,New] | 20:22 |
*** dprince has quit IRC | 20:22 | |
cdearborn | trown, thx! | 20:23 |
trown | cdearborn: np | 20:23 |
*** ppal has quit IRC | 20:23 | |
*** ndipanov has quit IRC | 20:24 | |
dtantsur | thnx | 20:24 |
trown | lol at ironic.rocks | 20:28 |
*** bethelwell has joined #openstack-ironic | 20:28 | |
NobodyCam | ya love pixie on that site | 20:29 |
openstackgerrit | Thiago Paiva Brito proposed openstack/ironic-specs: OneView Driver for Ironic https://review.openstack.org/187762 | 20:31 |
cinerama | i <3 it!! | 20:31 |
openstackgerrit | Thiago Paiva Brito proposed openstack/ironic-specs: OneView Driver for Ironic https://review.openstack.org/187762 | 20:33 |
lucas-dinner | elmiko, hey there | 20:36 |
elmiko | lucas-dinner: hey | 20:36 |
elmiko | lucas-dinner: i was just reaching out to all the api-wg liaisons to share a couple links and touch base with all the projects. | 20:36 |
elmiko | you got a minute or two to chat? | 20:37 |
lucas-dinner | elmiko, oh nice, yeah sure | 20:37 |
elmiko | lucas-dinner: awesome, so here are two links; the first is to a set of liaison responsibilities that the nova team has proposed, and the second is a merge guideline for the working group | 20:38 |
elmiko | https://wiki.openstack.org/wiki/Nova/APIWGLiaisons | 20:38 |
elmiko | https://review.openstack.org/#/c/186836/ | 20:38 |
elmiko | nothing urgent is needed on these, i'm just trying to get around and do a little outreach from the wg =) | 20:38 |
openstackgerrit | Thiago Paiva Brito proposed openstack/ironic-specs: OneView Driver for Ironic https://review.openstack.org/187762 | 20:39 |
lucas-dinner | elmiko, right, so this model is being tested by the nova folks right? And if it works well other projects might start adapting it | 20:39 |
elmiko | also, we will probably start using some of the merge guideline to freeze proposed guidelines before they go live to get a final round of comments from liaisons and PTLs if necessary. | 20:39 |
elmiko | lucas-dinner: yea | 20:39 |
lucas-dinner | elmiko, I will read it through. Maybe we can start doing some (or all) of it in Ironic this cycle too | 20:39 |
elmiko | ideally, i think we should incorporate their suggestions in the api-wg docs | 20:39 |
lucas-dinner | gotcha | 20:39 |
elmiko | lucas-dinner: awesome, it's nothing hardcore, just some sensible ideas about how liaisons can increase their interactions. | 20:40 |
*** eghobo has quit IRC | 20:40 | |
lucas-dinner | elmiko, thanks for the links I will read it and vote on the #186836 | 20:40 |
elmiko | or when necessary, delegate to other wg members | 20:40 |
elmiko | lucas-dinner: thanks for your time, enjoy dinner! | 20:40 |
*** spandhe has joined #openstack-ironic | 20:40 | |
lucas-dinner | elmiko, thank you for reaching out | 20:41 |
*** radek_ has quit IRC | 20:41 | |
lucas-dinner | elmiko, btw, we don't have an IRC channel for the API workgroup right? | 20:41 |
elmiko | lucas-dinner: we do, openstack-api | 20:41 |
elmiko | =) | 20:41 |
lucas-dinner | ohh | 20:41 |
* lucas-dinner joins | 20:41 | |
lucas-dinner | sorry for that | 20:41 |
elmiko | np, that's partially why i'm making the rounds ;) | 20:41 |
lucas-dinner | elmiko, thanks for that! /me joined | 20:42 |
*** krtaylor has quit IRC | 20:44 | |
*** eghobo has joined #openstack-ironic | 20:48 | |
openstackgerrit | Jarrod Johnson proposed stackforge/pyghmi: Implement OEM Inventory components https://review.openstack.org/187592 | 20:50 |
*** romcheg has quit IRC | 20:51 | |
*** elmiko has left #openstack-ironic | 20:53 | |
*** dtantsur has left #openstack-ironic | 21:02 | |
*** dtantsur has joined #openstack-ironic | 21:02 | |
openstackgerrit | Thiago Paiva Brito proposed openstack/ironic-specs: OneView Driver for Ironic https://review.openstack.org/187762 | 21:02 |
*** albertoffb has quit IRC | 21:03 | |
*** shakamunyi_ has joined #openstack-ironic | 21:03 | |
*** trown is now known as trown|outttypeww | 21:03 | |
dtantsur | devananda, if tc didn't reach my patch today, does it mean it gets delayed one week more? :( | 21:03 |
devananda | dtantsur: yes | 21:03 |
dtantsur | damn... | 21:03 |
devananda | dtantsur: remind me, link? | 21:04 |
dtantsur | devananda, https://review.openstack.org/#/c/185442/ | 21:04 |
*** shakamunyi__ has joined #openstack-ironic | 21:04 | |
devananda | dtantsur: it has almost enough to pass -- probably worth pinging ttx to get it approved now anyway | 21:05 |
*** shakamunyi__ is now known as tylerdurden | 21:05 | |
*** tylerdurden has quit IRC | 21:08 | |
*** shakamunyi_ has quit IRC | 21:08 | |
openstackgerrit | Jarrod Johnson proposed stackforge/pyghmi: Implement OEM Inventory components https://review.openstack.org/187592 | 21:09 |
mrda | Morning | 21:10 |
*** e0ne has quit IRC | 21:11 | |
*** krtaylor has joined #openstack-ironic | 21:11 | |
dtantsur | mrda, using this change to say good morning to you :) | 21:12 |
dtantsur | so g'morning :) | 21:12 |
dtantsur | I now I go to bed finally | 21:12 |
dtantsur | devananda, thanks for pinging people! | 21:12 |
mrda | dtantsur: have a nice sleep! | 21:12 |
NobodyCam | morning mrda | 21:12 |
*** dtantsur is now known as dtantsur|afk | 21:13 | |
devananda | dtantsur|afk: o/5 | 21:15 |
dtantsur|afk | :) | 21:16 |
* dtantsur|afk is now really afk | 21:16 | |
NobodyCam | bbiab running to drop off rental car | 21:22 |
*** thiagop has quit IRC | 21:32 | |
*** hblixt has joined #openstack-ironic | 21:34 | |
dhellmann | devananda: we have that fix for oslo.messaging rechecking, and I'll do a release this evening after it lands -- I had to fix an issue with the py34 jobs before it would work | 21:42 |
devananda | heh, np | 21:42 |
* dhellmann wishes ubuntu would release the updated package of 3.4 | 21:43 | |
*** dguerri is now known as dguerri`away | 21:51 | |
gabriel-bezerra | Hi all. | 21:57 |
gabriel-bezerra | Why is it that the node name attribute must be a hostname compliant string? | 21:58 |
jroll | gabriel-bezerra: because we like weird arbitrary rules? | 21:59 |
gabriel-bezerra | Is it used as an actual hostname anywhere? | 21:59 |
jroll | not today, no | 21:59 |
jroll | we had to make some sort of rules on it, this one seemed reasonable | 21:59 |
*** lucas-dinner has quit IRC | 22:00 | |
gabriel-bezerra | so it would not be a hard deal if we'd proposed to change that, would it? | 22:00 |
jroll | I don't know | 22:01 |
jroll | what's the use case? | 22:01 |
*** jgrimm is now known as zz_jgrimm | 22:01 | |
gabriel-bezerra | our discovery service could bring a name that is not hostname compliant from our CMDB | 22:02 |
*** MattMan has quit IRC | 22:02 | |
gabriel-bezerra | it has less sctrict rules about naming physical hardware | 22:02 |
gabriel-bezerra | nowadays we do not bring the name, but I bet it would be a good improvement in ux | 22:03 |
*** cppforlife_ has quit IRC | 22:04 | |
*** edcragg has joined #openstack-ironic | 22:07 | |
*** absubram has quit IRC | 22:07 | |
*** aweeks has quit IRC | 22:08 | |
mrda | gabriel-bezerra: we wanted the name to allow hostnames | 22:12 |
mrda | gabriel-bezerra: but it appears that restricting it to hostname RFC-compliance might be too too restrictive | 22:12 |
mrda | gabriel-bezerra: see https://bugs.launchpad.net/ironic/+bug/1434376 | 22:14 |
openstack | Launchpad bug 1434376 in Ironic "Node name rule relaxation" [Low,Confirmed] - Assigned to Lucas Alvares Gomes (lucasagomes) | 22:14 |
*** romcheg has joined #openstack-ironic | 22:15 | |
gabriel-bezerra | mrda: good to see there is already work being done on it. | 22:15 |
gabriel-bezerra | mrda: I'll talk to Lucas then to see how it is going. | 22:16 |
mrda | gabriel-bezerra: my preference is that whatever we change the regex to, it should _allow_ hostnames, but also support whatever other use case people find useful | 22:17 |
gabriel-bezerra | mrda: ++ | 22:17 |
*** Ng has quit IRC | 22:18 | |
gabriel-bezerra | mrda, jroll: thank you for the quick and thoughtful response | 22:18 |
mrda | np | 22:20 |
jroll | I'm inclined to just say: [0-9a-zA-Z\-_\.]+{1,255} | 22:20 |
jroll | or something to that effect | 22:21 |
mrda | that's a good start | 22:21 |
jroll | that is, alphanum + - _ . | 22:21 |
*** BadCub has quit IRC | 22:21 | |
jroll | + as in "in addition to", not allow + | 22:21 |
mrda | just depends if we want `~!@#$%^&*()-_=+[{]}\|;:'",<.>/? as well | 22:22 |
jroll | gah, no | 22:22 |
jroll | lol | 22:22 |
mrda | hey, once the kimono is opened... | 22:22 |
mrda | string.letters + string.digits + string.punctuation | 22:24 |
openstackgerrit | Julia Kreger proposed openstack/bifrost: Lock Bifrost to Ansible stable-1.9 https://review.openstack.org/187532 | 22:30 |
openstackgerrit | Julia Kreger proposed openstack/bifrost: Make Ansible modules 1.9 compatible https://review.openstack.org/186582 | 22:30 |
openstackgerrit | Julia Kreger proposed openstack/bifrost: Preserve logs in logs/ folder https://review.openstack.org/186581 | 22:31 |
openstackgerrit | Julia Kreger proposed openstack/bifrost: Update test-birfrost to limit log lines https://review.openstack.org/187703 | 22:31 |
*** [1]cdearborn has joined #openstack-ironic | 22:33 | |
* TheJulia is just re-basing changes | 22:33 | |
*** Sukhdev has joined #openstack-ironic | 22:35 | |
*** cppforlife_ has joined #openstack-ironic | 22:38 | |
openstackgerrit | Julia Kreger proposed openstack/bifrost: Add pip to requirements list https://review.openstack.org/187068 | 22:42 |
*** cdearborn has quit IRC | 22:48 | |
*** aweeks has joined #openstack-ironic | 22:50 | |
openstackgerrit | Julia Kreger proposed openstack/bifrost: WIP: Support for disabling DHCP server configuration https://review.openstack.org/187362 | 22:54 |
*** Ng has joined #openstack-ironic | 23:03 | |
*** BadCub has joined #openstack-ironic | 23:05 | |
*** BadCub has quit IRC | 23:09 | |
*** aweeks has quit IRC | 23:09 | |
dhellmann | devananda: I just tagged oslo.messaging 1.13.0, which includes the fix for ironic | 23:14 |
NobodyCam | dhellmann: woo hoo TY | 23:14 |
dhellmann | NobodyCam: the package is still building, but it should show up quickly | 23:14 |
NobodyCam | :) sweet | 23:15 |
*** BadCub has joined #openstack-ironic | 23:16 | |
NobodyCam | dhellmann: worth us pinning to that ver? currently we pin to >= 1.8.0 | 23:17 |
NobodyCam | https://github.com/openstack/ironic/blob/master/requirements.txt#L39 | 23:18 |
*** aweeks has joined #openstack-ironic | 23:19 | |
*** natorious is now known as zz_natorious | 23:20 | |
*** edcragg has quit IRC | 23:22 | |
*** chlong has joined #openstack-ironic | 23:22 | |
*** romcheg has quit IRC | 23:26 | |
*** naohirot has joined #openstack-ironic | 23:42 | |
*** bradjones has quit IRC | 23:43 | |
*** bradjones has joined #openstack-ironic | 23:45 | |
*** Haomeng|2 has joined #openstack-ironic | 23:52 | |
*** ijw has joined #openstack-ironic | 23:54 | |
*** Haomeng has quit IRC | 23:55 | |
*** barra204 has quit IRC | 23:57 | |
*** shakamunyi has quit IRC | 23:57 | |
*** rloo has quit IRC | 23:58 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!