Wednesday, 2017-03-22

*** eglute_s has quit IRC00:14
NobodyCamquestion: anyone ever encountered a case where a conductor appears and then disappears (about every 2 seconds)00:14
*** eglute has joined #openstack-ironic00:15
*** sasha2 has quit IRC00:25
*** Sukhdev has quit IRC00:30
*** Syed__ has quit IRC00:35
openstackgerritAnup Navare proposed openstack/ironic master: Config drive support for Ceph  https://review.openstack.org/40763900:40
*** crushil has joined #openstack-ironic00:41
*** harlowja has joined #openstack-ironic00:42
*** bfournie has quit IRC00:46
*** Lambert has joined #openstack-ironic00:47
*** phuongnh has joined #openstack-ironic00:51
*** rbudden has joined #openstack-ironic00:52
*** tovin07 has quit IRC00:52
*** zhurong has joined #openstack-ironic00:52
*** bfournie has joined #openstack-ironic01:00
*** SerenaFeng has joined #openstack-ironic01:00
*** baoli has joined #openstack-ironic01:02
*** linggao has quit IRC01:04
*** Sukhdev has joined #openstack-ironic01:11
*** priteau has joined #openstack-ironic01:16
*** priteau has quit IRC01:16
*** kjw3 has quit IRC01:43
*** mat128 has joined #openstack-ironic01:47
*** cdearborn has quit IRC01:48
*** sayantani01 has quit IRC01:52
*** sayantani01 has joined #openstack-ironic01:52
*** sayantani01 has quit IRC01:53
*** kjw3 has joined #openstack-ironic01:58
*** rbudden has quit IRC01:58
openstackgerritAnup Navare proposed openstack/ironic master: Updates documentation to install PySqlite3  https://review.openstack.org/44461002:08
*** maratoid_ has quit IRC02:13
*** baoli has quit IRC02:16
*** baoli has joined #openstack-ironic02:19
openstackgerritRuby Loo proposed openstack/ironic master: Add RPC and object version pinning  https://review.openstack.org/40749102:35
openstackgerritRuby Loo proposed openstack/ironic master: Add version column  https://review.openstack.org/41239702:35
openstackgerritRuby Loo proposed openstack/ironic master: Add new dbsync command with first online data migration  https://review.openstack.org/40855602:36
openstackgerritRuby Loo proposed openstack/ironic master: Unit test for checking cross-version migrations compatibility  https://review.openstack.org/41999302:37
*** priteau has joined #openstack-ironic02:37
openstackgerritRuby Loo proposed openstack/ironic master: Add new dbsync command with first online data migration  https://review.openstack.org/40855602:44
openstackgerritRuby Loo proposed openstack/ironic master: Unit test for checking cross-version migrations compatibility  https://review.openstack.org/41999302:45
*** kjw3 has quit IRC02:50
*** rama_y_ has joined #openstack-ironic02:51
*** yatinkarel has joined #openstack-ironic03:00
*** priteau has quit IRC03:01
yatinkarelcan someone help to debug why i cannot create baremetal nodes using ironic?03:01
yatinkarelI had three ironic nodes all in 'available' state. when i fired: openstack server create, all nodes one by one went to 'clean failed' state03:02
yatinkareland in maintenance mode03:02
*** baoli has quit IRC03:03
*** yuanying has quit IRC03:05
*** rama_y_ has quit IRC03:08
Lambertoh, which code are u using?03:11
openstackgerritRuby Loo proposed openstack/ironic master: Rolling upgrade procedure documentation  https://review.openstack.org/41656503:12
*** links has joined #openstack-ironic03:12
*** tovin07 has joined #openstack-ironic03:14
*** rloo has quit IRC03:18
*** Nisha_Agarwal has joined #openstack-ironic03:23
*** aparnav has joined #openstack-ironic03:32
TheJuliayatinkarel: sounds like the deployment is failing and then clean is failing.  Possibly they were started in available state to begin with and never previously went through cleaning?  Due they have a driver_info/deploy_ramdisk and deploy_kernel defined?03:33
TheJuliaNobodyCam: disappears/re-appears how-so.  How are you measuring this?03:33
*** faizy has joined #openstack-ironic03:34
*** harlowja has quit IRC03:35
*** links has quit IRC03:38
*** faizy has quit IRC03:39
*** Nisha_ has joined #openstack-ironic03:42
yatinkarelTheJulia, driver_info            | {u'ssh_port': 22, u'ssh_username': u'ubuntu', u'deploy_kernel':         |03:44
yatinkarel|                        | u'0e301419-9e69-4317-9cd9-62f4057e986b', u'deploy_ramdisk':             |03:44
yatinkarel|                        | u'e0275b7a-5595-40b0-b8ed-40ee762ea1ac', u'ssh_virt_type': u'virsh',    |03:44
yatinkarel|                        | u'ssh_address': u'192.168.200.12', u'ssh_key_filename':                 |03:44
yatinkarel|                        | u'/opt/stack/data/ironic/ssh_keys/ironic_key'}03:44
TheJuliayatinkarel: anything in last_error?  seems like your going to have to dig through your conductor log03:44
*** Nisha_Agarwal has quit IRC03:45
yatinkarelTheJulia, Timeout reached while waiting for callback for node 34317b75-c9f9-402e-abe5-586fc8ffafd503:45
yatinkarelTheJulia, Node 34317b75-c9f9-402e-abe5-586fc8ffafd5 moved to provision state "deleting" from state "deploy failed"; target provision state is "available"03:46
*** Sukhdev has quit IRC03:46
TheJuliayatinkarel: so... the console log for that VM is what your going to need to see.  For some some reason, most likely networking related, IPA was unable to heartbeat to the ironic API endpoint03:47
*** dyasny has quit IRC03:47
yatinkarelTheJulia, VM is in Error state03:47
TheJuliayatinkarel: not the VM in nova, but libvirt03:49
TheJulialocally on the host your running devstack on03:49
yatinkareldo you mean libvirtd logs03:51
yatinkarelTheJulia, I found something in: /opt/stack/data/ironic/logs/node-0_no_ansi_2017-03-21-17\:59\:44.log03:51
yatinkarelMConfiguring (net0 52:54:00:22:42:f7).................. Error 0x040ee119 (http://^Mipxe.org/040ee119)^M^M03:51
yatinkarel^MNo more network devices^M03:51
TheJuliahmmm03:51
TheJuliaany other logs for node-0?03:51
*** mat128 has quit IRC03:52
*** links has joined #openstack-ironic03:52
yatinkarelFile /opt/stack/data/ironic/logs/node-0_console_2017-03-21-17\:59\:44.log03:53
yatinkarelDirectory /opt/stack/data/ironic/logs/deploy_logs is empty03:54
TheJuliaHmm.  Sounds like either networking is wrong somewhere along the way binding the VM to neutron/dnsmasq... or dnsmasq just didn't reply03:55
yatinkarelTheJulia, any way to find that out04:00
yatinkarelTheJulia, it looks related to mine: https://lists.launchpad.net/yahoo-eng-team/msg38900.html04:02
TheJuliayatinkarel: All I can think is to double-check everything networking related, check neutron logs, check dnsmasq logs. Perhaps all before this, check your devstack local.cf file against ironic's examples and restack if you've made any changes04:02
yatinkarelTheJulia, Thanks. Ok i will recheck(BTW i have used local.conf that we use for ironic testing in magnum gate)04:04
yatinkarelAny way to bring nodes from clean failed state to available state04:05
TheJuliayatinkarel: sadly that is exactly what happens when there is any networking failure symptom wise.  https://bugs.launchpad.net/ironic/+bug/1499054 seems to be fairly specific to neutron, but that was a long while back04:05
openstackLaunchpad bug 1499054 in neutron "devstack VMs are not booting" [Critical,Fix released] - Assigned to Kevin Benton (kevinbenton)04:05
yatinkarelTheJulia, Ok will check this, may be i get some pointers04:06
TheJuliayatinkarel: not really... You can take them back to manageable state easily, but you have to go through cleaning again to reach available.  See https://docs.openstack.org/developer/ironic/_images/states.svg04:07
TheJuliayatinkarel: alternatively, delete the node and restack and it should be recreated04:07
TheJuliayatinkarel: Hopefully I've helped :(04:07
yatinkarelTheJulia, Thanks for the pointers, i will try again04:08
*** Nisha_ has quit IRC04:08
*** Nisha_ has joined #openstack-ironic04:09
*** udesale has joined #openstack-ironic04:12
*** Nisha_ has quit IRC04:13
*** yuanying has joined #openstack-ironic04:18
*** fragatin_ has joined #openstack-ironic04:28
*** fragatina has quit IRC04:32
tovin07JayF, hi04:32
*** fragatin_ has quit IRC04:33
*** links has quit IRC04:33
tovin07can you take a look at this https://review.openstack.org/#/c/358142/, it's rebased04:33
*** zhurong has quit IRC04:37
*** Administrator_ has joined #openstack-ironic04:38
*** crushil has quit IRC04:38
*** zhugaoxiao has quit IRC04:41
openstackgerritTuan Luong-Anh proposed openstack/ironic-specs master: Baremetal provisioning in UEFI secure boot mode  https://review.openstack.org/23027404:48
*** links has joined #openstack-ironic04:50
*** gcb has quit IRC04:53
*** rcernin has joined #openstack-ironic05:09
*** rcernin has quit IRC05:09
*** gcb has joined #openstack-ironic05:11
*** faizy has joined #openstack-ironic05:19
*** faizy has quit IRC05:22
*** faizy has joined #openstack-ironic05:22
*** faizy_ has joined #openstack-ironic05:23
*** faizy_ has quit IRC05:27
*** faizy_ has joined #openstack-ironic05:27
*** faizy has quit IRC05:27
*** faizy_ has quit IRC05:30
*** faizy_ has joined #openstack-ironic05:31
*** zhurong has joined #openstack-ironic05:33
*** vikrant has joined #openstack-ironic05:38
*** fragatina has joined #openstack-ironic05:38
*** fragatina has quit IRC05:40
*** fragatina has joined #openstack-ironic05:41
*** fragatina has quit IRC05:42
*** fragatina has joined #openstack-ironic05:43
*** udesale__ has joined #openstack-ironic05:46
*** udesale has quit IRC05:48
*** faizy_ has quit IRC06:07
*** faizy has joined #openstack-ironic06:08
*** faizy has quit IRC06:11
*** faizy has joined #openstack-ironic06:11
*** yatinkarel has quit IRC06:14
*** faizy has quit IRC06:15
*** faizy has joined #openstack-ironic06:15
*** faizy has quit IRC06:18
*** faizy has joined #openstack-ironic06:19
*** faizy has quit IRC06:21
*** faizy has joined #openstack-ironic06:22
*** ChubYann has quit IRC06:25
*** yatinkarel has joined #openstack-ironic06:28
openstackgerritVarun Gadiraju proposed openstack/ironic master: Remove translation of log messages from ironic/dhcp and ironic/cmd  https://review.openstack.org/44764206:29
*** fragatina has quit IRC06:37
openstackgerritBéla Vancsics proposed openstack/ironic master: Use more specific asserts in tests  https://review.openstack.org/43294106:44
*** udesale__ has quit IRC07:16
*** udesale has joined #openstack-ironic07:16
*** e0ne has joined #openstack-ironic07:17
*** jaosorior has joined #openstack-ironic07:21
*** jtomasek has joined #openstack-ironic07:31
*** pcaruana has joined #openstack-ironic07:39
*** rbartal has joined #openstack-ironic07:41
*** venki has quit IRC07:42
*** venki has joined #openstack-ironic07:42
*** jtomasek_ has joined #openstack-ironic07:43
*** jtomasek has quit IRC07:43
thomas_oneillmorning07:44
*** faizy has quit IRC07:54
*** tesseract has joined #openstack-ironic07:55
*** zzzeek has quit IRC08:00
*** udesale__ has joined #openstack-ironic08:00
*** zhurong has quit IRC08:00
*** zzzeek has joined #openstack-ironic08:00
*** amarao has joined #openstack-ironic08:01
*** udesale has quit IRC08:03
*** Serlex has joined #openstack-ironic08:04
*** zhurong has joined #openstack-ironic08:11
*** lifeless_ has joined #openstack-ironic08:13
*** lifeless has quit IRC08:14
pas-hamorning ironic :)08:14
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic master: [devstack] Use global requirements for virtualbmc  https://review.openstack.org/44071508:20
*** amotoki has joined #openstack-ironic08:20
openstackgerritGalyna Zholtkevych proposed openstack/ironic-specs master: API Evolution - ETAG identifiers  https://review.openstack.org/38199108:21
*** venki has quit IRC08:23
*** athomas has joined #openstack-ironic08:25
aparnavmorning ironic08:26
*** mgoddard has joined #openstack-ironic08:32
*** stendulker has joined #openstack-ironic08:35
thomas_oneilldnsmasq should stop responding to dhcp requests for a provisioned/deployed/active node, right?08:44
thomas_oneillusing bifrost, having this problem that when deployment is done and it reboots, pxe responds again on the reboot08:44
thomas_oneillso it gets in a deployment loop lol08:44
*** ralonsoh has joined #openstack-ironic08:45
*** amoralej|off is now known as amoralej08:45
*** yatinkarel has quit IRC08:48
*** e0ne has quit IRC08:56
*** athomas has quit IRC08:56
*** aolwas has joined #openstack-ironic09:00
*** yatinkarel has joined #openstack-ironic09:02
*** mgoddard has quit IRC09:08
*** racedo has joined #openstack-ironic09:13
*** jtomasek_ is now known as jtomasek09:16
*** mvpnitesh has joined #openstack-ironic09:16
openstackgerritDao Cong Tien proposed openstack/python-ironicclient master: Extends driver-list, driver-show supporting new hardware types  https://review.openstack.org/41927409:20
*** kromanenko has joined #openstack-ironic09:23
*** udesale has joined #openstack-ironic09:31
*** yuanying has quit IRC09:31
*** priteau has joined #openstack-ironic09:31
*** derekh has joined #openstack-ironic09:32
*** deray has joined #openstack-ironic09:32
*** udesale__ has quit IRC09:33
*** derekh has quit IRC09:33
*** derekh has joined #openstack-ironic09:34
*** mgoddard has joined #openstack-ironic09:34
mgoddardhi thomas_oneill: did anyone respond about bifrost?09:35
*** lucas-afk is now known as lucasagomes09:35
lucasagomesmorning all09:35
thomas_oneillmgoddard: about what about bifrost?09:35
*** mat128 has joined #openstack-ironic09:36
deraygood morning thomas_oneill, pas-ha aparnav mgoddard lucasagomes and all ironic'ers !09:37
mat128morning everyone09:37
deraymat128, g'morning09:37
thomas_oneillderay: that's great, what are you guys talking about though? do you mean the stuff I typed about an hour ago?09:38
openstackgerritGalyna Zholtkevych proposed openstack/ironic master: Disable guessing content type from extension in resource names.  https://review.openstack.org/40859709:40
openstackgerritGalyna Zholtkevych proposed openstack/ironic master: Ensure dying SOL console on conductor death  https://review.openstack.org/44657109:43
*** gcb has quit IRC09:48
*** zhurong has quit IRC09:52
*** e0ne has joined #openstack-ironic09:54
vdrokgood morning all!09:59
openstackgerritGalyna Zholtkevych proposed openstack/ironic master: Ensure dying SOL console on conductor death  https://review.openstack.org/44657110:02
mgoddardsorry thomas_oneill, missed your response. you asked a question this morning about bifrost & dhcp10:06
openstackgerritVladyslav Drok proposed openstack/ironic master: Revert the move of the logger setup  https://review.openstack.org/44813210:06
thomas_oneillmgoddard: oh I see you quit and then joined, yeah nobody responded.10:06
thomas_oneillif there isn't a solution as part of bifrost/ironic, it'd be easy enough to write some event responder to the state in the API changing and then add/remove iptables rules10:07
thomas_oneillI'm just surprised it doesn't already do that10:07
mgoddardthomas_oneill: ok, I'll give it a shot. Ironic configures the dhcp server to respond with PXE boot options for the duration of the node being deploying/active10:08
mgoddardthomas_oneill: if ironic sets the node to boot from local disk after deployment then it will still need a DHCP lease but should just ignore those options since it will be an OS requesting rather than a PXE client10:10
*** MattMan has quit IRC10:11
mgoddardthomas_oneill: is the node set to boot from the local disk (capabilities/boot_option:local)?10:11
thomas_oneillah was just going to ask what you meant by that10:11
*** MattMan has joined #openstack-ironic10:11
*** yuanying has joined #openstack-ironic10:11
thomas_oneillmgoddard: I'll check, I think I need to redeploy anyway, will take note of the states and capabilities throughout10:11
thomas_oneillty10:11
mgoddardthomas_oneill: np10:13
thomas_oneillalso if I stop dnsmasq and xinetd on my server and reboot my baremetal, the baremetal still manages to dhcp/pxe ... wtf :s10:13
openstackgerritDao Cong Tien proposed openstack/python-ironicclient master: Update OSC baremetal node set/unset supporting dynamic drivers  https://review.openstack.org/43154210:13
thomas_oneilllol10:13
thomas_oneillthis could be a personal problem..10:13
mgoddardthomas_oneil: btw, bifrost by default configures ironic inspector not to manage iptables rules10:13
*** udesale__ has joined #openstack-ironic10:23
openstackgerritVladyslav Drok proposed openstack/ironic master: Do not merge  https://review.openstack.org/44848910:24
openstackgerritVladyslav Drok proposed openstack/ironic master: Do not merge  https://review.openstack.org/44848910:25
*** udesale has quit IRC10:25
*** rook is now known as rook-afk10:26
*** athomas has joined #openstack-ironic10:27
*** rook-afk is now known as rook10:33
*** rook is now known as rook-afk10:35
*** jkilpatr has quit IRC10:37
*** dtantsur|afk is now known as dtantsur10:38
*** racedo has quit IRC10:38
dtantsurmorning ironic10:39
*** venki has joined #openstack-ironic10:42
*** racedo has joined #openstack-ironic10:49
thomas_oneillmorning10:49
*** phuongnh has quit IRC10:51
thomas_oneillthis is my bifrost/playbooks/inventory/group_vars/target http://paste.openstack.org/show/603737/ - there isn't anything wrong with the dib_env_vars bit is there? And the target is the right place? It's not taking, can't log into the box with this image on using the devuser creds, and trying to troubleshoot the failure to stick this static network configuration on too10:53
*** jkilpatr has joined #openstack-ironic10:53
*** udesale__ has quit IRC10:57
thomas_oneillthe env var is in the /httpboot/deployment_image.d/dib-manifests/dib_environment so that's a good sign ...10:57
pas-hathe dib var for password looks legit10:59
mgoddardthomas_oneil: I think it's your dib_elements11:00
mgoddardthomas_oneill: bifrost doesn't include the default elements when you override dib_elements, I had the same issue.11:00
thomas_oneilloh I see11:01
pas-hahttps://github.com/openstack/bifrost/blob/30ab6dd568b82a78b77b3adcfb2fcc1c452b6e17/playbooks/test-bifrost.yaml#L8011:01
pas-haextra_dib_elements is your thing11:01
mgoddardthomas_oneill: you also need vm11:01
pas-hathe default ones are 'vm enable-serial-console simpleinit"11:02
pas-hathe console one is for dib >=2, previously it was 'serial-console'11:03
*** stendulker has quit IRC11:03
mgoddardthomas_oneill, pas-ha: the non-test install.yaml playbook doesn't support extra_dib_elements :( https://github.com/openstack/bifrost/blob/30ab6dd568b82a78b77b3adcfb2fcc1c452b6e17/playbooks/install.yaml#L2511:04
thomas_oneillpas-ha: ah yes, I did come across this yesterday, but checked the doc and the doc suggested differently I think, https://github.com/openstack/bifrost/tree/master/playbooks/roles/bifrost-create-dib-image#role-variables11:04
*** SerenaFeng has quit IRC11:04
mgoddardpas-ha, thomas_oneill: I lied, actually it does :)11:04
thomas_oneillyeah my dib_arguments in the http boot are: -o /httpboot/deployment_image.qcow2 -t qcow2 ubuntu vm enable-serial-console simple-init11:04
thomas_oneilloh devuser isn't there...11:05
thomas_oneillI will add extra_dib_elements to the doc and submit PS11:05
thomas_oneillpas-ha: but yes also another thing I noticed is that if you look at the test-bifrost.yaml vs. what is ran in install.yaml for real, the way dib_os_element is passed is different11:06
thomas_oneillI'm guessing the test playbook is called by zuul, maybe some difference like this could explain why zuul was passing the broken thing from yesterday... is that fixed yet actually? https://bugs.launchpad.net/bifrost/+bug/167464911:07
openstackLaunchpad bug 1674649 in bifrost "AnsibleUndefinedVariable bootstrap" [Undecided,New] - Assigned to Pavlo Shchelokovskyy (pshchelo)11:07
pas-hathomas_oneill: not yet, it is on my todo list for today/tomorrow11:08
pas-hathat's indeed a problem of bifrost that it tests not that is supposed to be used 'in prod'...11:09
pas-has/that/what/11:09
thomas_oneillpas-ha: do you know what the fix is and can it be synopsised in a line on IRC? if it's not too difficult and you can point me in the direction, I could try to do that\11:09
thomas_oneillI'm trying to get my contributions up :p11:09
*** deray has quit IRC11:10
thomas_oneillis the remote for bifrost https://username@review.openstack.org/ironic/bifrost.git or bifrost/bifrost.git ?11:15
pas-hathomas_oneill: might be around https://github.com/openstack/bifrost/blob/master/playbooks/roles/bifrost-keystone-client-config/templates/clouds.yaml.j2#L3 (either I got the condition wrong, or jinja needs all vars to resolve even when it will not render that part, in which case we'd need to split the clouds.yaml template in two)11:15
pas-haopenstack/bifrost11:15
pas-hafeel free to assign the bug to yourself, I've unassigned myself11:16
*** venki has quit IRC11:18
*** venki has joined #openstack-ironic11:19
xavierrgood morning ironic11:19
*** vmud213 has joined #openstack-ironic11:23
sbezverkmgoddard: ping11:23
lucasagomesdtantsur, mgould|afk btw, based on the conversation we had yesterday, do you envision sushy working like: "ss = s.get_system('/redfish/v1/Systems/437XR1138R2')" ?11:24
thomas_oneillhm I can ssh -p 29418 thomas.oneill@review.openstack.org gerrit version, but when I try to gerrit review in the bifrost repository it says could not read remote repository, I don't seem to have read rights11:24
lucasagomesfor the collection, I think we still need some heuristic to get the /Systems or /Nodes or ... collection from the root resource, right ?11:24
dtantsurlucasagomes, seems so. for the collection, yes..11:25
lucasagomesright on11:25
lucasagomescheers11:25
openstackgerritThomas O'Neill proposed openstack/bifrost master: Trivial correction to documentation on creating the DIB image.  https://review.openstack.org/44851411:26
mgoddardsbezverk: hi11:26
thomas_oneillweird... re-cloning fixed it11:27
*** priteau has quit IRC11:27
sbezverkmgoddard: I got baremetal provisioned in kolla-kube in my local test bed, but the same config in the gate is faling. would you be able to help a bit to debug it?11:30
mgoddardsbezverk: that's good news. how can I help?11:32
*** athomas has quit IRC11:33
*** rook-afk is now known as rook11:34
openstackgerritGalyna Zholtkevych proposed openstack/ironic master: Warn user about guessing content type deprecation.  https://review.openstack.org/40859711:35
sbezverkmgoddard: in the gate VM gets images and then get stuck :( could not spot anything obvious in logs11:36
sbezverkmgoddard: Appreciate if you could check logs too, maybe your ironic eye will catch what I have missed?11:36
mgoddardsbezverk: got a link?11:37
aparnavhey mariojv, I have a question related to rescue implementation11:37
sbezverkmgoddard: http://logs.openstack.org/50/441550/108/experimental/gate-kolla-kubernetes-deploy-centos-source-3-ironic-nv/01fb32e/logs/vm-1.log11:37
sbezverkhere is where VM gets stuck ^^^11:38
*** gcb has joined #openstack-ironic11:38
sbezverkmgoddard: hereis the link with all opensatck related logs http://logs.openstack.org/50/441550/108/experimental/gate-kolla-kubernetes-deploy-centos-source-3-ironic-nv/01fb32e/logs/openstack/11:38
aparnavmariojv, Are you planning to add dib changes for rescue support?11:39
sbezverkmgoddard: and here is the gate job log: http://logs.openstack.org/50/441550/108/experimental/gate-kolla-kubernetes-deploy-centos-source-3-ironic-nv/01fb32e/console.html11:39
*** bfournie has quit IRC11:41
*** bfournie has joined #openstack-ironic11:41
*** fxpester has joined #openstack-ironic11:43
openstackgerritAndrey Shestakov proposed openstack/bifrost master: Add missed extra_args to pip command  https://review.openstack.org/44852311:44
*** bfournie has quit IRC11:46
jrollmorning y'all11:46
jrollwhatever plague I got is getting worse, I'll be in and out today, likely more of out11:46
vdrokmorning jroll , get better!11:49
jrollthanks vdrok11:50
*** athomas has joined #openstack-ironic11:51
*** hw_wutianwei has quit IRC11:54
*** hshiina has joined #openstack-ironic11:54
openstackgerritAparna proposed openstack/ironic master: Support hpsum based `update_firmware` for iLO drivers  https://review.openstack.org/42257211:56
thomas_oneillgah wtf :P https://github.com/openstack/bifrost/blob/master/bifrost/inventory.py#L182-L185 - so you can't just set addressing_mode, you have to set node_network_data lolll12:01
*** SerenaFeng has joined #openstack-ironic12:02
dtantsurmorning jroll12:02
dtantsurjroll, get better :(12:02
*** aparnav has quit IRC12:02
thomas_oneillanyone know where I can find what network_node_data i.e. network_data.json should contain/look like?12:03
jrolldtantsur: thanks, I'll be alright12:03
thomas_oneillah found something here http://www.madorn.com/openstack-metadata-types.html12:03
jrollthomas_oneill: here's the spec http://specs.openstack.org/openstack/nova-specs/specs/liberty/implemented/metadata-service-network-info.html#rest-api-impact12:04
thomas_oneillalso, what is the point of all these ipv4_ vars in this part of bifrost if in order to use it we have to set the node_network_data which presumably will contain the same info? https://github.com/openstack/bifrost/tree/master/playbooks/roles/bifrost-configdrives-dynamic12:04
*** fragatina has joined #openstack-ironic12:05
*** ccamacho is now known as ccamacho|lunch12:06
thomas_oneillcould any bifroster provide an example of what node_network_data should look like?12:11
thomas_oneilloh I missed the 'and not ipv4_addr' part... so it should see the ipv4_address and refrain from setting the addressing_mode to dhcp... maybe I've defined these in the wrong place... ignore me for now sorry12:13
lucasagomesdtantsur, mgould|afk rloo: https://review.openstack.org/#/c/448539/12:14
lucasagomestested locally here, seems good I will setup it with devstack just to make sure as well12:14
*** mgould|afk is now known as mgould12:15
mgouldhi ironic!12:15
mgouldjroll: get well soon!12:15
mgouldlucasagomes: looking now12:18
*** dprince has joined #openstack-ironic12:18
lucasagomesalso https://review.openstack.org/#/c/448542/12:18
lucasagomes(just adding sushy to gerritbot)12:18
lucasagomesmgould, thanks12:18
lucasagomesjroll, get better soon man!12:19
*** venki has quit IRC12:19
*** venki has joined #openstack-ironic12:19
*** bfournie has joined #openstack-ironic12:20
*** rbartal has quit IRC12:24
*** hw_wutianwei has joined #openstack-ironic12:29
*** SerenaFeng has quit IRC12:31
*** rbudden has joined #openstack-ironic12:34
*** jaypipes has joined #openstack-ironic12:35
*** SerenaFeng has joined #openstack-ironic12:36
openstackgerritKyrylo Romanenko proposed openstack/ironic master: Add functional API tests for portgroups  https://review.openstack.org/43687812:37
*** lucasagomes is now known as lucas-hungry12:43
*** amoralej is now known as amoralej|lunch12:47
*** priteau has joined #openstack-ironic12:47
*** trown|outtypewww is now known as trown12:50
thomas_oneillhm, the configdrive is there with the network_data.json and it's all correct, but there's no network configuration on any of my interfaces. It's ubuntu xenial with the simple-init element, wouldn't have thought the legacy debian interfaces option would be required, I'll try setting that to true though12:50
mariojvgood morning ironic12:51
thomas_oneillhi12:51
*** milan has joined #openstack-ironic12:52
milanmorning Ironic! :)12:52
mgouldhi milan!12:52
milanhey mgould! :)12:52
*** priteau has quit IRC12:54
*** jcoufal has joined #openstack-ironic12:54
*** pbandark has joined #openstack-ironic13:03
openstackgerritYin Zhongsheng proposed openstack/ironic master: Remove log translations  https://review.openstack.org/44856113:03
*** sasha21 has joined #openstack-ironic13:06
*** Goneri has joined #openstack-ironic13:07
*** rloo has joined #openstack-ironic13:10
*** dyasny has joined #openstack-ironic13:10
*** ccamacho|lunch is now known as ccamacho13:12
*** vmud213 has quit IRC13:14
*** ijw has joined #openstack-ironic13:15
*** ijw has quit IRC13:16
*** ijw has joined #openstack-ironic13:17
*** cdearborn has joined #openstack-ironic13:17
*** ijw has quit IRC13:21
thomas_oneillsimple-init isn't doing anything with my network_data.json, I'm trying for a static network configuration, the network_data.json is there in the /mnt/config13:23
*** artom__ is now known as artom13:24
openstackgerritMichael Turek proposed openstack/ironic master: Remove translator assignments from i18n  https://review.openstack.org/44819613:24
*** venki has quit IRC13:25
*** venki has joined #openstack-ironic13:25
*** Lambert has quit IRC13:26
*** snecklifter has joined #openstack-ironic13:29
*** baoli has joined #openstack-ironic13:30
*** SerenaFeng has quit IRC13:30
*** ipukha has joined #openstack-ironic13:31
*** SerenaFeng has joined #openstack-ironic13:34
*** SerenaFeng has quit IRC13:35
*** SerenaFeng has joined #openstack-ironic13:36
*** lucas-hungry is now known as lucasagomes13:38
*** amoralej|lunch is now known as amoralej13:46
*** crushil has joined #openstack-ironic13:48
*** snecklifter has quit IRC13:48
pas-haironic cores, please merge https://review.openstack.org/#/c/447930/ to unblock stable gates13:49
openstackgerritVladyslav Drok proposed openstack/ironic master: [WIP] Neutron notifications  https://review.openstack.org/44077813:49
*** priteau has joined #openstack-ironic13:49
*** mtanino has joined #openstack-ironic13:50
*** priteau_ has joined #openstack-ironic13:50
*** priteau_ has quit IRC13:51
*** priteau_ has joined #openstack-ironic13:51
*** priteau has quit IRC13:51
*** jcoufal_ has joined #openstack-ironic13:59
*** Zombine has quit IRC14:00
rloopas-ha: thx for the reminder. done.14:01
pas-hathanks ruby :)14:01
*** udesale has joined #openstack-ironic14:02
*** jcoufal has quit IRC14:03
*** oomichi has quit IRC14:04
*** chlong has joined #openstack-ironic14:04
*** venki has quit IRC14:05
*** venki has joined #openstack-ironic14:05
*** oomichi has joined #openstack-ironic14:06
*** priteau_ has quit IRC14:11
*** chlong has quit IRC14:12
*** chlong has joined #openstack-ironic14:13
openstackgerritFellype Cavalcante proposed openstack/ironic master: Shutdown server before change boot order for agent drivers  https://review.openstack.org/40829814:13
openstackgerritGalyna Zholtkevych proposed openstack/ironic master: Ensure dying SOL console on conductor death  https://review.openstack.org/44657114:14
*** dprince has quit IRC14:19
*** links has quit IRC14:21
*** causten has joined #openstack-ironic14:25
*** rpioso has joined #openstack-ironic14:32
*** vikrant has quit IRC14:36
*** dprince has joined #openstack-ironic14:36
*** crushil_ has joined #openstack-ironic14:38
*** crushil has quit IRC14:38
*** crushil_ is now known as crushil14:38
*** athomas has quit IRC14:41
*** baha has joined #openstack-ironic14:44
openstackgerritLucas Alvares Gomes proposed openstack/ironic-specs master: Follow-up patch for the Redfish spec  https://review.openstack.org/44547814:48
*** athomas has joined #openstack-ironic14:50
openstackgerritMerged openstack/ironic master: [devstack] Use global requirements for virtualbmc  https://review.openstack.org/44071514:50
NobodyCamGood Morning Ironic'erd14:51
NobodyCamieek14:51
NobodyCam*Ironic'ers :)14:51
* NobodyCam goes back to coffee14:51
lucasagomesNobodyCam, morning14:51
NobodyCammorning lucasagomes :)14:51
dtantsurmorning NobodyCam14:55
NobodyCamMorning dtantsur :)14:55
TheJuliaGood morning14:57
dtantsurmorning TheJulia14:57
openstackgerritMerged openstack/ironic master: Revert the move of the logger setup  https://review.openstack.org/44813214:58
openstackgerritThomas O'Neill proposed openstack/bifrost master: Trivial correction to documentation on creating the DIB image.  https://review.openstack.org/44851414:59
*** sayantani01 has joined #openstack-ironic14:59
*** g3ek has quit IRC14:59
*** yufei has joined #openstack-ironic15:01
thomas_oneillhi TheJulia mgoddard , responded on that fix, I'm 99% sure dib_elements doesn't actually overwrite dib_elements, have either of you experienced being able to do this in the past?15:03
TheJuliaI'm fairly sure, but it was a LONG time ago15:04
thomas_oneill99% sure = I've just tested and used ansible debug and seeing it not doing anything, I put the debug just before the "Initiate image build" task, my vars, which definitely work for the other dib stuff, are in the target inventory file15:04
mgoddardthomas_oneill: I'm currently using dib_elements15:04
dtantsurTheJulia, do you have ironic-ui support in bifrost? I'm looking for a place to cargo-cult puppet support from :)15:04
mgoddardthomas_oneill: are you using the install.yaml playbook?15:04
TheJuliadtantsur: sadly no, no horizon stuff installed by bifrost15:04
thomas_oneillmgoddard: and it actually overwrites/negates the vm enable-serial-console elements?15:04
thomas_oneillmgoddard: yes15:04
dtantsurah, I see15:04
dtantsurTheJulia, I suspect we don't need most of https://github.com/openstack/puppet-murano/blob/master/manifests/dashboard.pp15:05
TheJuliadtantsur: no, the puppet manifest for the ui should be about half that size ;)15:05
thomas_oneillmgoddard: oh, you just write a lightweight playbook for calling into that role?15:05
dtantsurheh, I see15:05
mgoddardthomas_oneill: yes. where are you setting your overridden value for the variable? I'm using kolla-ansible, which provides an extra-vars file to ansible via -e @<file>, which has highest variable precedence15:06
mgoddardthomas_oneill: no, I'm using install.yaml15:06
dtantsurTheJulia, judging by https://docs.openstack.org/developer/ironic-ui/installation.html it does not have any configuration, right?15:06
thomas_oneillmgoddard: dib_elements is in playbooks/inventory/group_vars/target15:06
*** trown is now known as trown|brb15:07
thomas_oneilland I've made localhost a symlink to target just in case of any issues (not a fan of bifrost 3-way var split fwiw :p)15:07
mgoddardthomas_oneill: right, but rather than modify that file, kolla-ansible writes out an override file15:07
TheJuliadtantsur: It should draw entirely from keystone at present.  I guess we should add some knobs for that... our just wait for people to ask for those knobs15:08
TheJuliadtantsur: at least, if memory serves me right now, its all from keystone15:08
dtantsurnice, less work for me  :)15:08
mgoddardthomas_oneill: https://github.com/openstack/kolla-ansible/blob/master/ansible/roles/bifrost/tasks/bootstrap.yml#L215:08
*** g3ek has joined #openstack-ironic15:08
thomas_oneillah yeah, the dib.yml15:09
thomas_oneillI envy you, I gave up on trying to run bifrost from kolla, was a world of pain for me lol15:09
TheJuliathomas_oneill: (some of that free way split should not be too hard to fix..., it is all use pattern driven though)15:09
*** mtanino has quit IRC15:10
mgoddardthomas_oneill: http://docs.ansible.com/ansible/playbooks_variables.html#variable-precedence-where-should-i-put-a-variable15:10
mgoddardthomas_oneill: play vars > inventory vars15:10
mgoddardthomas_oneill: but extra vars > everything. generally they are very useful for applying 'global' configuration15:11
thomas_oneillso maybe I should do an -e @my_vars_I_insist_on hmmm15:12
thomas_oneillthanks15:12
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic stable/mitaka: [devstack] Use global requirements for virtualbmc  https://review.openstack.org/44055915:12
*** mtanino has joined #openstack-ironic15:13
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic stable/newton: [devstack] Use global requirements for virtualbmc  https://review.openstack.org/44863115:15
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic stable/ocata: [devstack] Use global requirements for virtualbmc  https://review.openstack.org/44863215:16
thomas_oneillseems like the hack is that we're using the same role for the ipa as for the deploy_image15:16
thomas_oneillif they were two different roles they could each have their own normal default vars and then vars in the inventory would override those15:17
TheJuliadtantsur: yeah, from keystone service name baremetal.  There are like 4 knobs hard coded in the server side api interface.  We should get them moved out so they are not in the code, but the defaults seem fine for most cases except where TLS is totally misconfigured at a system leave.15:17
*** udesale has quit IRC15:17
*** zackf has joined #openstack-ironic15:18
TheJuliathomas_oneill: eh, but they both ultimately call the same tool.  I guess it could just be an extra variable passed in to append "ironic-agent" to the list15:18
dtantsurTheJulia, ack, thanks15:18
openstackgerritThomas O'Neill proposed openstack/bifrost master: Trivial correction to documentation on creating the DIB image.  https://review.openstack.org/44851415:21
*** zackf has quit IRC15:22
*** zackf has joined #openstack-ironic15:23
thomas_oneillI'm a fan of the roles all having their own readme's though, that's nice :-)15:24
*** dprince has quit IRC15:24
openstackgerritYin Zhongsheng proposed openstack/ironic master: Remove log translations  https://review.openstack.org/44864215:24
TheJuliaI was originally planning on peeling off the roles into their own git repos, but as time passed, I gradually lost time to do things like that :)15:25
*** dprince has joined #openstack-ironic15:25
mgoddardthomas_oneill, TheJulia: if you want to reuse a role then what I would do would be to make the user-configurable variable in inventory/host/group vars, then set the relevant role variable to the required global variable on each use of the role15:26
*** mat128 has quit IRC15:26
mgoddardnot a great explanation, but essentially have two dib_elements-like variables. one for the OS image, another for ipa.15:27
*** zackf has quit IRC15:27
*** trown|brb is now known as trown15:27
openstackgerritChris Krelle proposed openstack/bifrost master: Add Note that trailing space is required.  https://review.openstack.org/44755515:27
*** chlong has quit IRC15:30
*** yufei has left #openstack-ironic15:31
thomas_oneilloh yeah that's a good shout15:32
mgouldppiela: did you submit your patch to unbreak ironic-ui's JS tests?15:32
*** chlong has joined #openstack-ironic15:34
ppielamgould: Not yet. I have gotten pulled off on other stuff, but will try to get to it later today15:34
*** zackf has joined #openstack-ironic15:38
*** zackf has quit IRC15:41
* TheJulia goes off conferencing :(15:43
*** causten has quit IRC15:46
*** zackf has joined #openstack-ironic15:47
*** causten has joined #openstack-ironic15:47
mgouldppiela: cool, thanks15:50
*** hshiina has quit IRC15:51
*** zackf has quit IRC15:51
*** causten has quit IRC15:51
ppielamgould: Sorry for the delay15:52
mgouldppiela: no problem15:53
*** Serlex has quit IRC15:53
*** SerenaFeng has quit IRC15:54
*** fragatina has quit IRC15:56
*** Sukhdev has joined #openstack-ironic15:57
*** mvpnitesh has quit IRC16:00
*** bcornec has joined #openstack-ironic16:03
joannamorning ironic!16:05
TheJulia Good morning16:05
*** causten has joined #openstack-ironic16:05
crushilmorning joanna16:06
*** harlowja has joined #openstack-ironic16:06
*** Sukhdev_ has joined #openstack-ironic16:06
thomas_oneillTheJulia: you know the node_network_data, should that be like a nested yaml kind of structure, or a json string, or what?16:10
thomas_oneillI know that it should eventually end up being converted to network_data.json which I know the format of, but not the var for bifrost's format D:16:10
TheJuliathomas_oneill: hmmmmm  I don't remember16:11
openstackgerritMario Villaplana proposed openstack/ironic-specs master: Change client default API version  https://review.openstack.org/44866116:11
TheJuliaI feel like the networking is write for a rewrite16:11
thomas_oneillnode_network_data=dict(required=False, type='dict')   # guessing this just means it should be like a var dict thing, same as the dib env vars, I'll give it a go.16:13
mguzhi. has anyone out there deployed esxi 6.x with ironic?16:14
* thomas_oneill throws up on mguz's feet16:14
mguzyeah... i know.16:14
TheJuliamguz: I'm not aware of anyone that has tried...16:14
TheJuliaWell, someone was asking about setting network configuration....16:15
mguzi've done it with a whole disk image but the network gets screwed up as it remembers the mac addr of the system the image was created from16:15
TheJuliaAnd our suggestion was to rip apart the disk image, inject something like glean, and deploy that image16:15
TheJuliaManually strip out the old network config?16:16
TheJuliaIn Esxi 4, I think it was like ini files....16:17
openstackgerritKyrylo Romanenko proposed openstack/python-ironicclient master: Add usage documentation for Baremetal OSC Plugin  https://review.openstack.org/41526116:18
*** vikrant has joined #openstack-ironic16:18
*** baha has quit IRC16:18
mguzTheJulia: yeah, there's a config file that's written and i wanted to preserve it mostly so that it had the correct password etc etc16:20
pas-hamguz: i have a colleague who tried/researched that (deplou esxi thru ironic), but in the end he found vmware autodeploy to be better for this16:20
pas-haand better integrated into whole vmeare ecosystem16:21
mguzpas-ha: pity, i'm trying to gather all our baremetal installs under the openstack api.16:21
pas-haif you are interested ping vnogin in IRC16:21
TheJuliamguz: script an auto editor maybe?16:21
mguzTheJulia: Yeah thinking that or something like that..16:22
TheJuliaM16:23
TheJuliaErr16:23
TheJuliaA number of folks have discussed wanting to be able to do the same since some people also deploy windows via ironic16:23
*** bcornec has quit IRC16:23
mguzTheJulia: we are doing windows deploys ok16:24
mguzTheJulia: we have 2k8r2 2012r2 and 2016 images that work16:24
TheJuliaWoot16:24
mguzTheJulia: esxi can be kickstarted, so my brain tells me it must be possible via ironic also, just haven't figured it out yet :-(16:26
TheJuliaSo... you could hack together specific deploy driver16:26
TheJuliaOr... hmmmm16:26
TheJuliaGlance tftp override Parameters16:27
TheJuliaI'm not sure we ever implemented that16:27
mguzyeah i thought about adding the kernel and ramdisk to glance and booting those somehow. Or maybe a custom deploy_ramdisk16:28
*** vikrant has quit IRC16:29
TheJuliaYeah, not implemented16:29
*** gfhellma has joined #openstack-ironic16:32
TheJuliaHmmmm16:32
*** kromanenko has quit IRC16:32
*** trown is now known as trown|lunch16:32
openstackgerritLucas Alvares Gomes proposed openstack/ironic master: Add redfish driver  https://review.openstack.org/43898216:38
openstackgerritLucas Alvares Gomes proposed openstack/ironic master: Add documentation for the redfish driver  https://review.openstack.org/44253716:38
openstackgerritLucas Alvares Gomes proposed openstack/ironic master: DevStack: Setup a Redfish environment  https://review.openstack.org/44552916:38
*** Sukhdev_ has quit IRC16:40
*** chlong has quit IRC16:45
*** Nisha_Agarwal has joined #openstack-ironic16:48
*** links has joined #openstack-ironic16:49
*** chlong has joined #openstack-ironic16:52
*** baha has joined #openstack-ironic16:52
*** lucasagomes is now known as lucas-brb16:55
*** milan has quit IRC16:57
*** fragatina has joined #openstack-ironic17:00
*** lucas-brb is now known as lucasagomes17:01
*** kencjohnston has joined #openstack-ironic17:01
*** bfournie1 has joined #openstack-ironic17:02
*** sayantani01 has quit IRC17:02
*** sayantani01 has joined #openstack-ironic17:03
*** mat128 has joined #openstack-ironic17:04
*** bfournie has quit IRC17:05
*** causten has quit IRC17:07
anishdtantsur: been waiting on you for a bit on https://review.openstack.org/#/c/401415/, could you take a look ?17:09
dtantsuranish, I will, eventually, but I'd prefer one of Dell folks, rpioso or cdearborn, to take a look first17:11
anishsure, sounds good17:12
*** amarao has quit IRC17:12
*** jkilpatr_ has joined #openstack-ironic17:13
*** jkilpatr has quit IRC17:15
*** e0ne has quit IRC17:16
*** maratoid_ has joined #openstack-ironic17:18
*** gfhellma has quit IRC17:23
*** gfhellma has joined #openstack-ironic17:23
rloohi lucasagomes, wrt the redfish spec, will you be able to handle resources within a different authority as the request URI? (where URI starts with '//')17:27
rloolucasagomes:  s/URI/URI path/17:27
lucasagomesrloo, I wish I could answer that, I mean, I don't have any means to test it17:28
rloolucasagomes: maybe we should explicitly mention that we don't support that now?17:28
lucasagomesI assume that, as they are all part the same collection they will be handled in the same way17:29
lucasagomesbut I don't have any hardware (or mockup) that exposes that17:29
lucasagomesrloo, yeah, that would be safe to say I guess17:29
lucasagomesdef it will be not tested17:29
rloolucasagomes: i don't understand the authority stuff, if the resource patch is a diff authority, can you get that info using the redfish_address?17:29
*** trown|lunch is now known as trown17:30
lucasagomesrloo, I looked at the specification but it doesn't say anything about it how that's done :-/17:31
lucasagomesI was assuming that there's some type of delegation involved ?17:31
rloolucasagomes: ha ha. we should just put a caveat that it isn't explicitly supported then!17:32
lucasagomesif not, sushy should handle that by looking at the "//" and connecting to that addr, BUT, and the credentials ?17:32
lucasagomesI've no clue to be honest17:32
lucasagomesrloo, ++17:32
* dtantsur is fine with not supporting it for now17:32
lucasagomesyeah let's do that17:32
lucasagomescause look: http://redfish.dmtf.org/schemas/DSP0266_1.1.html17:32
dtantsuralso going a bit earlier today, see you17:32
lucasagomesthere's literaly nothing about it17:32
lucasagomesdtantsur, see ya17:32
*** dtantsur is now known as dtantsur|afk17:32
rloociao dtantsur|afk17:32
*** mtanino has quit IRC17:33
*** causten has joined #openstack-ironic17:34
*** links has quit IRC17:39
*** jaosorior has quit IRC17:40
*** ipukha has quit IRC17:41
mgouldanish: left some comments17:43
anishthanks !17:44
mgouldnp, sorry I took so long to get to it17:44
*** Sukhdev_ has joined #openstack-ironic17:45
Nisha_Agarwallucasagomes, hi we expect system_id to be inputted17:46
Nisha_Agarwallucasagomes, hi17:46
lucasagomesNisha_Agarwal, hi17:46
Nisha_Agarwallucasagomes, we expect the system_id to be imputted y user17:46
Nisha_Agarwalby*17:47
lucasagomesrloo, so you want me to put a note in that spec ?17:47
rlooyeah, give me a few secs, just commenting.17:47
lucasagomesNisha_Agarwal, in the ironic driver ? Yes17:47
lucasagomesrloo, cool, thanks17:47
Nisha_Agarwallucasagomes, but why? We can get the system_id from the System_collection()17:48
lucasagomesNisha_Agarwal, but how would I know that a the node A in ironic is the System B in the Redfish controller ?17:48
Nisha_Agarwallucasagomes, Sorry I didnt get it....17:49
lucasagomesNisha_Agarwal, I mean, when you register the node in ironic17:49
*** pbandark has quit IRC17:49
lucasagomesyou have to tell the Ironic which System (in redfish) that node represents17:49
lucasagomesso I need the ID17:50
Nisha_Agarwalthat can be tell by the Ip address?17:50
Nisha_Agarwali am missing something?17:50
jlvillalI have to admit, this looks like a good idea.  https://github.com/Qix-/better-exceptions   If it was a stable project.17:50
lucasagomesNisha_Agarwal, the address is the same for all nodes in that redfish controller17:50
lucasagomesI think you are missing something17:50
Nisha_Agarwallucasagomes, ok basic ques...what do u mean here by a redfish controller? a chassis?17:51
lucasagomesNisha_Agarwal, I the "redfish system" (in broader terms, I hate the terminology there). When you connect to redfish you then can get a system collection right ? As you said17:52
lucasagomesinside that collection there are 0..N systems17:52
lucasagomeswhere each is represented by a node in Ironic17:53
lucasagomesso the ironic node needs to know which system in the redfish controller it's suppose to manage17:53
Nisha_Agarwallucasagomes, Ok. i got ur point.17:53
mgouldjlvillal: oh, neat17:55
jlvillalmgould: yeah. I like that it prints the values. Which is usually the info I want.17:55
mgouldjlvillal: YES17:56
openstackgerritMario Villaplana proposed openstack/ironic master: Add rescuewait timeout periodic task  https://review.openstack.org/35315617:56
openstackgerritMario Villaplana proposed openstack/ironic master: Agent driver rescue implementation  https://review.openstack.org/40043717:56
openstackgerritMario Villaplana proposed openstack/ironic master: Add API and conductor methods for [un]rescue  https://review.openstack.org/35083117:56
Nisha_Agarwallucasagomes, but then managers, chassis etc are common to all those systems if i say ...because these are outside the systems collection17:56
*** mgoddard has quit IRC17:57
mariojvjoanna: soliosg: thanks for the rescue reviews, addressed your comments on the api/conductor and rescuewait patches17:57
mariojvi'll be updating the agent driver patch tomorrow17:58
joannamariojv: thanks! I will look at it today :)17:58
*** derekh has quit IRC17:59
joannajlvillal: better exceptions are really better! nice, I like it!17:59
jlvillaljoanna: Maybe it can become a stable project at some point and then we can use it :)18:00
joannajlvillal: what do you consider a right moment to ack sth as stable?18:01
lucasagomesNisha_Agarwal, right, but Ironic doesn't have any abstraction for those18:02
jlvillaljoanna: Longer than 10 days for sure :)18:02
lucasagomesI mean, the library will handle it as part of inspector but Ironic won't actually manage any other resources (that said we have an abstraction for Chasiss but we don't use it)18:02
lucasagomesrloo, replied, lemme know what you think about redfish_resource_id vs redfish_system_id18:03
joannajlvillal: I could never find a definitive answer for that question :)18:03
jlvillaljoanna: Probably because there isn't one ;)18:03
Nisha_Agarwallucasagomes, we would need chassis for inspection though18:03
Nisha_Agarwalto get disk size18:03
lucasagomesNisha_Agarwal, sure, but for that we don't need an abstraction in the ironic api that's what I mean18:04
Nisha_Agarwallucasagomes, yes...we need to handle all that in library18:04
lucasagomessure yeah18:04
*** ralonsoh has quit IRC18:04
Nisha_Agarwal:)18:04
*** milan has joined #openstack-ironic18:06
rloolucasagomes: so I don't understand what 'system_id' is. Is that term/nomenclature used in the dmtf?18:06
rloolucasagomes: I mean the term 'system'?18:06
lucasagomesrloo, yeah, it's the name of the resource18:06
lucasagomesrloo, http://redfish.dmtf.org/schemas/v1/ComputerSystem.v1_0_0.json#/definitions/ComputerSystem18:06
rloolucasagomes: 'ComputerSystem' then, not 'system'.18:07
mgouldnaming things is hard :-(18:07
lucasagomesrloo, yeah, it's computer_system but it's kinda long18:07
rloomgould: true, but in this case, dmft has already named them.18:07
lucasagomesalso the collection in the API is called "Systems"18:07
lucasagomesso that's why I think "system" should be OK18:07
lucasagomesrloo, so redfish_computer_system_id ?18:08
rloolucasagomes: yeah, i'm fine with it being long and more explicit.18:08
Nisha_Agarwalrloo, when u do a GET call for /redfish/v1, you get one key as  "Systems": {18:08
Nisha_Agarwal    "@odata.id": "/redfish/v1/Systems/"18:08
rloolucasagomes: esp if 'system' is used in other contexts for redfish18:08
*** athomas has quit IRC18:09
lucasagomesrloo, hmm ok, I mean I don't mind I still prefer system18:09
rlooNisha_Agarwal: is that the ComputerSystem?18:10
Nisha_Agarwalrloo, yes18:10
*** jcoufal_ has quit IRC18:10
rloolucasagomes: so i'm fine with system if that term isn't overloaded in dmtf (ie, if we know we won't have to distinguish this sytem from some other FooSystem)18:10
*** gfhellma has quit IRC18:10
rloolucasagomes: and then we make sure we document that it is the ComputerSystem ID, a resource ID blah blah.18:10
Nisha_AgarwalSo the URI traversal which give u the ComputerSystem data is "/redfish/v1/Systems/<identifier>"18:10
lucasagomesrloo, I think we are fine, cause, if you look at that schema it uses a lot of "system" references, e.g: "The manufacturer or OEM of this system.", "The model number for this system" and so on18:11
lucasagomesrloo, ack18:11
Nisha_Agarwallucasagomes, ++18:11
rloolucasagomes: ok, i trust you and Nisha_Agarwal. I really don't want to become an expert in this :)18:12
*** maratoid_ has quit IRC18:12
Nisha_Agarwalrloo, :)18:12
lucasagomesme neither heh18:12
*** mtanino has joined #openstack-ironic18:12
Nisha_Agarwallucasagomes, one ques18:12
Nisha_Agarwallucasagomes, why username and password is optional?18:13
lucasagomesNisha_Agarwal, cause it's not mandatory to have a username/password when connecting to redfish18:13
lucasagomessame as ipmi password for example18:13
lucasagomescan be empty18:13
lucasagomesbut ofc we document it in a way to indicate that you should set one18:14
Nisha_Agarwalbut that would be security issue to not have credentials18:14
Nisha_Agarwallucasagomes, do we want to support systems in insecure manner?18:15
lucasagomesNisha_Agarwal, I mean, what can we do ? If it was configured in that way I think it's not up to Ironic to refuse to manage it18:16
lucasagomeswe should do, alert operatiors about it18:16
*** gfhellma has joined #openstack-ironic18:17
lucasagomesalso, people can use it for enrollment I believe, like ipmi, having a default or no password to start with18:17
Nisha_Agarwallucasagomes, IMHO atleast username/password should be required options and not optional18:17
lucasagomesthan using something like inspector to change that crendentials and enroll it in ironic18:17
*** mgould is now known as mgould|afk18:18
*** maratoid_ has joined #openstack-ironic18:20
Nisha_Agarwallucasagomes, but the parsing driver info is not done for enrollment alone here in the driver code(thats what i understand). It applies to all the states....18:20
Nisha_Agarwallucasagomes, it does open gates for insecurity ( as it looks to me)18:21
openstackgerritMerged openstack/ironic-lib master: Remove translation of log messages  https://review.openstack.org/44763318:21
lucasagomesNisha_Agarwal, sure it does... but I think that Ironic can't or shouldn't do much about it apart from make sure we document it and maybe even have some warnings in the code about it being insecure18:22
lucasagomeswhat are the options ?18:22
lucasagomesrefuse to manage the node ?18:22
openstackgerritJoanna Taryma proposed openstack/ironic master: Add base cinder common interface  https://review.openstack.org/35562518:22
Nisha_Agarwalhmmm but in other drivers we say username/passwd are mandatory options to be passed by user in driver_info...isnt it?18:23
openstackgerritLucas Alvares Gomes proposed openstack/ironic-specs master: Follow-up patch for the Redfish spec  https://review.openstack.org/44547818:24
lucasagomesNisha_Agarwal, some yes, some not18:24
Nisha_Agarwallucasagomes, :) ok anyway i dont think we can do much about it.18:24
lucasagomesNisha_Agarwal, no worries, but yeah look at ipmitool18:25
Nisha_Agarwalexcept documentation18:25
lucasagomesonly address is mandatory for example18:25
Nisha_Agarwallucasagomes,:) sure..18:25
aNuposicvdrok, are you around?18:25
vdrokaNuposic: hi, yes18:26
lucasagomesfolks, calling it a day!18:26
lucasagomeshave a great evening all18:26
vdrokgood night lucasagomes !18:26
lucasagomeso/18:26
*** lucasagomes is now known as lucas-afk18:26
aNuposichi, so i agree I can use single option object_store_?_endpoint for configdrive+tempurls18:26
aNuposicvdrok,18:26
Nisha_Agarwalgood night lucas-afk18:26
aNuposicvdrok, but i am hesitant in using a new option for auth_type18:27
aNuposicbecause consider object_store=radosgw and auth_type=keystone then it will not work18:27
aNuposici mean auth_type and object_store_endpoint for configdrive has to match18:28
openstackgerritMiles Gould proposed openstack/ironic-ui master: Show boot device in Node Details/Configuration  https://review.openstack.org/44708518:28
vdrokaNuposic: well, I was kind of out of the discussion for a while :) PS8 comments still suggest the separate auth option. I'm fine with using a single one for both, if it works the way you describe18:28
*** Goneri has quit IRC18:28
aNuposicvdrok, I agree to deprecate temp_url_endpoint_type and use single object_store_endpoint_? for tempurls and configdrive18:29
aNuposicjlvillal, soliosg mariojv ^18:29
vdrokyup, I'm good with this18:29
aNuposicvdrok, cool, lets see if folks are around to suggest about deprecating temp_url option18:30
aNuposicjoanna, jroll ^18:31
vdrokI mean, that's not much to do, just set deprecated_group and deprecated_name on object_store_type and remove the temp url endpoint type option :) But yeah, sure, let's wait what others think18:32
thomas_oneillis there a document somewhere that explains exactly how simple-init works and how it calls glean?18:34
*** jkilpatr_ has quit IRC18:34
aNuposicvdrok, yes otherwise will do that by EOD :) thanks18:36
vdroknp, thank you :)18:36
*** yatinkarel has quit IRC18:37
aNuposicthomas_oneill, https://docs.openstack.org/developer/diskimage-builder/elements/simple-init/README.html might be helpful18:38
aNuposicthomas_oneill, has some instructions for running glean ^18:38
*** jkilpatr has joined #openstack-ironic18:41
openstackgerritMilan Kováčik proposed openstack/ironic-inspector-specs master: Multiple PXE filtering backends  https://review.openstack.org/43986218:41
joannavdrok, dtantsur|afk: I updated the BFV patch https://review.openstack.org/#/c/35562518:44
thomas_oneillaNuposic: yeah I've seen that... that doc sucks imo lol it only tells you how to use a different version of glean D:18:44
vdrokjoanna: morning, thanks. tho it's unlikely I'll look at it today :)18:45
joannavdrok: no worries. I just wanted to let you know that I adressed your comment. This patch is pretty short if it changes anything :)18:46
aNuposicthomas_oneill, ah i see, i have not called glean with simple-init anytime. I just used requirements.txt to install glean18:46
aNuposicthomas_oneill, what are you trying to do?18:46
thomas_oneillaNuposic: it also says that the /usr/local/sbin/simple-init.sh is called but this doesn't exist on the deployment I've got18:46
*** dsneddon has quit IRC18:47
thomas_oneillaNuposic: just need to troubleshoot how the static network configuration isn't being applied properly following a deployment. There are lots of things I need to look at that are wrong with the process actually, I should do more on my own, ty though18:47
aNuposicthomas_oneill, hmm, i see. Good luck :)18:48
*** dsneddon has joined #openstack-ironic18:49
*** jcoufal has joined #openstack-ironic18:50
*** yatinkarel has joined #openstack-ironic18:51
*** baha has quit IRC18:52
*** harlowja has quit IRC18:59
*** harlowja has joined #openstack-ironic19:00
*** harlowja has quit IRC19:01
openstackgerritVladyslav Drok proposed openstack/ironic-specs master: Add synchronize-events-with-neutron spec  https://review.openstack.org/34368419:02
mariojvthanks aNuposic i'll take a look later19:05
*** amoralej is now known as amoralej|off19:06
aNuposicmariojv, cool19:06
vdrokgood night everyone!19:08
aNuposicoops sorry19:08
aNuposicvdrok, good night19:08
*** srart has quit IRC19:11
*** srart has joined #openstack-ironic19:16
*** jcoufal has quit IRC19:19
*** Nisha_Agarwal has quit IRC19:23
*** ChubYann has joined #openstack-ironic19:24
*** joanna is now known as joana_lunch19:26
*** xavierr has quit IRC19:31
*** xavierr has joined #openstack-ironic19:33
*** xavierr has quit IRC19:34
*** jcoufal has joined #openstack-ironic19:34
*** krtaylor has quit IRC19:36
*** xavierr has joined #openstack-ironic19:37
*** fxpester has quit IRC19:38
*** xavierr has quit IRC19:38
*** xavierr has joined #openstack-ironic19:41
*** harlowja has joined #openstack-ironic19:45
*** baha has joined #openstack-ironic19:46
*** xavierr has quit IRC19:46
*** xavierr has joined #openstack-ironic19:49
*** xavierr has quit IRC19:50
*** xavierr has joined #openstack-ironic19:52
*** xavierr has quit IRC19:55
*** xavierr has joined #openstack-ironic19:56
openstackgerritMerged openstack/ironic master: Fix local copy of scenario manager  https://review.openstack.org/44793019:57
*** xavierr has quit IRC19:59
*** xavierr has joined #openstack-ironic20:00
*** hamzy has quit IRC20:01
*** zackf has joined #openstack-ironic20:03
*** xavierr has quit IRC20:03
*** fragatina has quit IRC20:03
*** xavierr has joined #openstack-ironic20:04
*** dprince has quit IRC20:04
*** gfhellma has quit IRC20:05
*** xavierr has quit IRC20:07
*** sasha21 has quit IRC20:08
*** xavierr has joined #openstack-ironic20:09
*** rybridges has joined #openstack-ironic20:09
zer0c00l_When someone wants to power 'on' a machine using ipmitool driver, it looks like we set the power state to 'on' first and check power status in subsequent loops (Dynamiclooping call)20:09
zer0c00l_https://github.com/openstack/ironic/blob/master/ironic/drivers/modules/ipmitool.py#L51120:09
zer0c00l_Let's say when the power state on command itself fails on the first try20:09
*** xavierr has quit IRC20:09
zer0c00l_We seem to be keep running power status command needlessly20:09
zer0c00l_ipmitool <args>  power on <- Failed20:10
zer0c00l_is this expected behavior?20:10
*** sasha21 has joined #openstack-ironic20:10
*** xavierr has joined #openstack-ironic20:12
*** xavierr has quit IRC20:14
*** zackf has quit IRC20:14
*** xavierr has joined #openstack-ironic20:17
*** tesseract has quit IRC20:17
*** vishwanathj has quit IRC20:18
*** harlowja has quit IRC20:18
*** vishwanathj has joined #openstack-ironic20:18
*** e0ne has joined #openstack-ironic20:22
*** zackf has joined #openstack-ironic20:23
*** zackf has quit IRC20:27
*** maratoid_ has quit IRC20:31
*** racedo has quit IRC20:33
*** maratoid_ has joined #openstack-ironic20:34
*** snecklifter has joined #openstack-ironic20:39
*** maratoid_ has quit IRC20:39
*** crushil has quit IRC20:40
*** crushil has joined #openstack-ironic20:40
*** jcoufal has quit IRC20:42
*** alex_xu has quit IRC20:43
*** maratoid_ has joined #openstack-ironic20:45
*** alex_xu has joined #openstack-ironic20:46
*** Goneri has joined #openstack-ironic20:46
mrdaMorning Ironic20:50
*** mat128 has quit IRC20:50
*** mat128 has joined #openstack-ironic20:50
*** joana_lunch is now known as joanna20:51
joannamorning mrda :)20:51
mrdahey joanna20:52
*** mat128 has quit IRC20:55
*** Sukhdev has quit IRC20:56
*** xavierr has quit IRC20:57
*** crushil has quit IRC20:58
*** xavierr has joined #openstack-ironic21:00
*** trown is now known as trown|outtypewww21:00
*** xavierr has quit IRC21:01
*** xavierr has joined #openstack-ironic21:04
*** xavierr has quit IRC21:05
*** krtaylor has joined #openstack-ironic21:06
*** chlong has quit IRC21:08
*** xavierr has joined #openstack-ironic21:09
*** fragatina has joined #openstack-ironic21:10
*** e0ne has quit IRC21:19
*** e0ne has joined #openstack-ironic21:20
*** baoli has quit IRC21:22
*** e0ne has quit IRC21:22
*** baoli has joined #openstack-ironic21:22
mariojvmorning mrda21:28
mariojvi proposed some topics for the review party tommorrow: https://etherpad.openstack.org/p/ironic-review-party mainly just the weekly priorities21:28
mariojvfeel free to add anything there if you're going to attend :)21:29
*** baha has quit IRC21:33
*** harlowja has joined #openstack-ironic21:33
*** baoli has quit IRC21:36
*** vgadiraj has quit IRC21:38
*** joanna has quit IRC21:38
*** mpotdar has quit IRC21:38
*** baha has joined #openstack-ironic21:40
*** hamzy has joined #openstack-ironic21:42
*** baha has quit IRC21:46
*** pcaruana has quit IRC21:47
*** baoli has joined #openstack-ironic21:47
*** baoli has quit IRC21:48
*** soliosg has quit IRC21:48
*** sayantani01 has quit IRC21:48
*** sayantani01 has joined #openstack-ironic21:49
*** bfournie1 has quit IRC21:53
*** baoli has joined #openstack-ironic21:58
*** vinsh has joined #openstack-ironic21:58
openstackgerritRamamani Yeleswarapu proposed openstack/ironic master: Follow-up patch to remove translation of log messages  https://review.openstack.org/44880221:58
*** causten has quit IRC22:00
*** causten has joined #openstack-ironic22:01
*** causten has quit IRC22:05
*** systemdave has quit IRC22:08
*** vinsh has quit IRC22:12
*** vinsh has joined #openstack-ironic22:13
*** krtaylor has quit IRC22:16
openstackgerritOpenStack Proposal Bot proposed openstack/ironic master: Updated from global requirements  https://review.openstack.org/44813722:19
*** vinsh has quit IRC22:21
*** systemdave has joined #openstack-ironic22:21
*** jkilpatr has quit IRC22:25
*** cdearborn has quit IRC22:28
*** pmannidi has quit IRC22:29
Sukhdev_TheJulia : ping22:30
TheJuliaSukhdev_: hi22:30
Sukhdev_TheJulia : QQ22:30
TheJuliaSure22:30
Sukhdev_TheJulia : I want to change the PXE booting mode from IPXE to standard PXE - i.e. equivalent of IPXE_ENABLED=False22:31
Sukhdev_TheJulia : but, do not want to go through the unstack/stack - is there any painless way to do it?22:32
Sukhdev_TheJulia : i.e. by hacking something in /etc/ironic/ironic.config ?22:32
Sukhdev_TheJulia : Currently I have IPXE_ENABLED=true22:33
TheJuliaYou can edit ironic.conf, and restart or HUP the conductor22:33
*** snecklifter has quit IRC22:34
TheJuliaIpxe_enabled22:35
Sukhdev_TheJulia : this is how the ironic.conf looks like - http://paste.openstack.org/show/603833/22:36
TheJuliaYup, just change the value of ipxe_enabled to false send the HUP signal using kill22:39
Sukhdev_TheJulia : What do you mean by HUP signal? Just simply restarting the ir-cond should do it, right?22:41
TheJuliaRestarting ir-cond is an option as well.  HUP as in kill -1 <PID>22:41
Sukhdev_TheJulia : got it.... thanks - let me see if this does the trick22:42
*** vills has joined #openstack-ironic22:44
*** pmannidi has joined #openstack-ironic22:51
*** pmannidi has quit IRC22:51
*** mat128 has joined #openstack-ironic22:52
*** pmannidi has joined #openstack-ironic22:52
*** pmannidi has quit IRC22:52
*** pmannidi has joined #openstack-ironic22:52
openstackgerritMerged openstack/ironic master: Add functional API tests for portgroups  https://review.openstack.org/43687822:57
*** baoli has quit IRC22:58
*** baoli has joined #openstack-ironic22:59
*** vills has quit IRC23:02
*** baoli has quit IRC23:03
*** jkilpatr has joined #openstack-ironic23:05
*** joanna has joined #openstack-ironic23:15
*** maratoid_ has quit IRC23:23
*** mat128 has quit IRC23:25
*** baoli has joined #openstack-ironic23:33
*** mtanino has quit IRC23:37
openstackgerritRamamani Yeleswarapu proposed openstack/ironic master: Remove translation of log messages from ironic/conductor  https://review.openstack.org/44772723:39
openstackgerritMilan Kováčik proposed openstack/ironic-inspector-specs master: Multiple PXE filtering backends  https://review.openstack.org/43986223:48
*** milan has quit IRC23:52
*** mnaser has quit IRC23:56
*** mnaser has joined #openstack-ironic23:57

Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!