Wednesday, 2019-03-20

*** trungnv has quit IRC00:03
*** hamzy has joined #openstack-ironic00:19
*** gyee has quit IRC00:57
*** whoami-rajat has joined #openstack-ironic01:29
*** yajushiSri has joined #openstack-ironic01:34
*** w14161_1_ has joined #openstack-ironic01:56
*** w14161_1 has quit IRC01:59
*** trown|outtypewww has quit IRC02:31
*** trown has joined #openstack-ironic02:33
*** tiendc has joined #openstack-ironic02:34
*** tiendc has quit IRC02:35
*** tiendc has joined #openstack-ironic02:35
*** hwoarang has quit IRC03:03
*** hwoarang has joined #openstack-ironic03:04
*** MattMan_ has quit IRC03:26
*** MattMan_ has joined #openstack-ironic03:26
*** sthussey has quit IRC03:35
*** hwoarang has quit IRC03:44
*** hwoarang has joined #openstack-ironic03:45
*** yajushiSri has quit IRC03:48
*** stendulker has joined #openstack-ironic03:56
*** rachit7 has joined #openstack-ironic04:20
*** tiendc has quit IRC05:26
*** yajushiSri has joined #openstack-ironic05:42
*** rachit7 has quit IRC05:48
*** yajushiSri has quit IRC05:58
*** yajushiSri has joined #openstack-ironic06:01
*** dsneddon has quit IRC06:24
*** dsneddon has joined #openstack-ironic06:40
*** hjensas has quit IRC06:43
*** dsneddon has quit IRC06:45
*** Chaserjim has quit IRC07:07
*** Pragya2804_ has joined #openstack-ironic07:11
*** e0ne has joined #openstack-ironic07:17
*** pcaruana has joined #openstack-ironic07:20
*** e0ne has quit IRC07:27
arne_wiebalckgood morning, ironic!07:34
*** hjensas has joined #openstack-ironic07:37
Pragya2804_Hello! My name is Pragya Sethi. I am an Outreachy Summer 2019 aspirant. I have worked on 3 python projects last semester but I am new to open source. I found the OpenStack projects interesting and would like to contribute in them. Please guide me where to start from. Thanks in advance.07:38
*** e0ne has joined #openstack-ironic07:42
*** dsneddon has joined #openstack-ironic07:51
openstackgerritArne Wiebalck proposed openstack/ironic-specs master: Add details on target_raid_config for Software RAID support  https://review.openstack.org/64310007:53
hjensasPragya2804_: Welcome! :)07:55
*** dsneddon has quit IRC07:55
hjensasPragya2804_: For Ironic (and other Openstack projects) you will find a Developer/Contributor section in docs, the Ironic one is here: https://docs.openstack.org/ironic/latest/contributor/index.html07:56
hjensasPragya2804_: There is also this doc which is generic for all Openstack - https://docs.openstack.org/contributors/07:58
*** Pragya2804_ has quit IRC08:05
arne_wiebalckstendulker: ping08:07
stendulkerGood morning arne_wiebalck08:08
arne_wiebalckhey stendulker, you have 2 mins?08:09
stendulkertell me08:09
arne_wiebalckstendulker: it's about the s/w RAID spec08:09
arne_wiebalckstendulker: thanks for your comments08:09
stendulkerok08:09
arne_wiebalckstendulker: I think we have two different use cases here08:10
arne_wiebalckstendulker: one is to signal whether you want a s/w RAID08:10
arne_wiebalckstendulker: the other whether a s/w RAID has been created08:10
stendulkeryes08:10
arne_wiebalckstendulker: I like the idea of setting sth on the node08:11
arne_wiebalckstendulker: to see if there is indeed a RAID08:11
arne_wiebalckstendulker: cleaning would however always to delete/create08:11
stendulkeryes, that was my point. We alredy have architected method to  update node when RAID is successfully created. We can make use of that08:12
arne_wiebalckstendulker: sounds good08:12
*** andrein has joined #openstack-ironic08:12
arne_wiebalckstendulker: I'm just wondering when we would be in a situation where there is a target_raid_config, but no RAID08:12
arne_wiebalckstendulker: certainly right after the config has been created for the first time08:13
arne_wiebalckstendulker: but after that?08:13
stendulkerone could be - if user sets target_raid_config but does not invoke cleaning and invokes deploy instead08:13
* arne_wiebalck wasn't aware this is a possibility08:14
stendulkerbut why you want to think that way. It would be easier if we say have a field that would be present on node if raid is successfully created and would be wiped of upon raid deletion. It would simplify it08:15
arne_wiebalckstendulker: since we use nova on top, the node goes always through cleaning, instances are not created by the ones  having access to ironic08:15
*** e0ne has quit IRC08:15
arne_wiebalckstendulker: sure, that'd certainly be good, I was just wondering when it would change value08:16
* arne_wiebalck has to run off to a meeting08:16
stendulkerok. that would be one way of using it. but once feature is available, ppl could use it in various combinations08:16
arne_wiebalckstendulker: let me think about it :) thanks!08:16
stendulkeryou are welcome :)08:17
*** rpittau|afk is now known as rpittau08:27
rpittaugood morning ironic! o/08:27
arne_wiebalckhey rpittau o/08:32
rpittauhi arne_wiebalck :)08:32
*** dsneddon has joined #openstack-ironic08:33
*** Pragya2804 has joined #openstack-ironic08:33
*** rcernin has quit IRC08:38
*** dsneddon has quit IRC08:38
*** Lucas_Gray has joined #openstack-ironic08:40
iurygregorygood morning o/08:44
rpittauhi iurygregory :)08:48
iurygregoryo/08:48
*** jtomasek has quit IRC08:49
*** dsneddon has joined #openstack-ironic09:04
*** dougsz has joined #openstack-ironic09:09
*** S4ren has joined #openstack-ironic09:09
*** jtomasek has joined #openstack-ironic09:10
*** etingof has joined #openstack-ironic09:17
*** dsneddon has quit IRC09:19
*** dtantsur|afk is now known as dtantsur09:21
arne_wiebalckhey iurygregory o/09:22
dtantsurdustinc: hi! you can make SDK retry nearly everything with some configuration (we default to retrying 409 and 503)09:22
dtantsurmorning ironic09:22
arne_wiebalckdtantsur: o/09:22
iurygregorymorning dtantsur and arne_wiebalck o/09:22
dtantsurdoes anyone know what happened to the standalone job?09:23
iurygregorythe error is strange lol09:24
*** derekh has joined #openstack-ironic09:25
iurygregorydtantsur, they have merged some changes for gabi tempest in devstack and tempest maybe the fix they provided didnt help us (this is just my look at the first glance)09:30
dtantsurI see.. okay, let's wait for the current wave of rechecks to finish09:31
iurygregorystrange for me is that standalone does not use gabbi_tempest_path O.o09:33
dtantsurCI magic09:33
iurygregoryexactlly \o/09:34
iurygregorythat the fun part about CI09:34
openstackgerritRiccardo Pittau proposed openstack/ironic-specs master: Fixing linters test  https://review.openstack.org/64480009:35
rpittauhey arne_wiebalck, when you have a minute can you please try adding a Depends-on this ^^^ to your https://review.openstack.org/643100 should fix the linters test09:37
patchbotpatch 643100 - ironic-specs - Add details on target_raid_config for Software RAI... - 4 patch sets09:37
rpittauhi dtantsur :)09:37
arne_wiebalckrpittau: cool, will do!09:37
rpittaudtantsur, the CI exploded during the night, should be fixed now :) https://review.openstack.org/64469909:38
patchbotpatch 644699 - tempest - Default gabbi_tempest_path to empty string (MERGED) - 1 patch set09:38
iurygregoryrecheck all the failed patches! <insert meme> XD09:39
rpittauxD09:39
iurygregoryrpittau, but your recheck in https://review.openstack.org/#/c/643716/ standalone still failed =(09:40
patchbotpatch 643716 - ironic - Advance python-dracclient version requirement - 2 patch sets09:40
rpittauiurygregory, still before fix09:40
rpittauand thanks, rechecked now :D09:40
iurygregoryyeah09:40
iurygregoryjsut saw the time was merged XD09:41
* iurygregory looked at the time the patch was submitted =X09:41
arne_wiebalckrpittau: how is that file to be read, i.e. what's the  '# Apache 2.0' comment for?09:42
rpittauarne_wiebalck, I'm excluding version 2.3.1 of stestr that breaks os-testr09:43
rpittauarne_wiebalck, that comment is the license09:43
arne_wiebalckrpittau: TIL :) thanks!09:43
rpittauarne_wiebalck, np :)09:43
arne_wiebalckstendulker: Regarding our discussion earlier today: I added a corresponding task to the s/w RAID story: https://storyboard.openstack.org/#!/story/200458109:45
stendulkerarne_wiebalck: ok. Thanks09:45
openstackgerritArne Wiebalck proposed openstack/ironic-specs master: Add details on target_raid_config for Software RAID support  https://review.openstack.org/64310009:50
*** dsneddon has joined #openstack-ironic09:53
etingofiurygregory, interestingly, my all-py3 devstack @bionic running redfish succeeds09:57
iurygregoryetingof, do you have a patch up?09:58
etingofiurygregory, any idea how is it different from the job that fails?09:58
iurygregoryor the configuration you used?09:58
etingofiurygregory, I have a minor fix to ensure sushy emulator is running under py3, but I am not sure why would it fail under py2... -- https://review.openstack.org/#/c/644622/09:59
patchbotpatch 644622 - ironic - Respect $USE_PYTHON3 settings for gunicorn - 2 patch sets09:59
iurygregoryetingof, let me check with Depends-On with your patch10:00
iurygregorybut many tests have failed with your patch =(10:01
etingofiurygregory, my devstack conf -- http://paste.openstack.org/show/748104/10:02
*** dsneddon has quit IRC10:02
iurygregoryetingof, ty10:02
etingofiurygregory, they fail because of the underlying patch in the chain, though I do not yet understand why...10:02
iurygregoryetingof, gotcha10:04
etingofneedless to say that my devstack never fails with my patches10:04
etingofI think we should add systemd units for both sushy emulator and vbmc, otherwise we do not have logs10:05
arne_wiebalckrpittau: https://review.openstack.org/643100 looks ok now :)10:06
patchbotpatch 643100 - ironic-specs - Add details on target_raid_config for Software RAI... - 5 patch sets10:06
iurygregoryetingof, i think is a good idea10:06
etingofperhaps, that's how it works for other services in devstack10:06
rpittauarne_wiebalck: cool!10:08
arne_wiebalckrpittau: thanks again!10:08
*** Lucas_Gray has quit IRC10:13
openstackgerritMerged openstack/bifrost master: Remove UCS configuration  https://review.openstack.org/64457710:13
*** Lucas_Gray has joined #openstack-ironic10:13
openstackgerritMerged openstack/networking-baremetal master: Update master for stable/stein  https://review.openstack.org/64466110:19
openstackgerritMerged openstack/networking-baremetal stable/stein: Update .gitreview for stable/stein  https://review.openstack.org/64465910:19
*** dsneddon has joined #openstack-ironic10:37
*** dsneddon has quit IRC10:41
*** e0ne has joined #openstack-ironic10:45
*** iurygregory is now known as iurygregory|lunc10:58
*** iurygregory|lunc is now known as iury|lunch10:58
*** yolanda has quit IRC10:59
*** yolanda has joined #openstack-ironic10:59
openstackgerritMerged openstack/python-ironicclient master: Use endpoint_override in version negotiation  https://review.openstack.org/64362411:02
*** Pragya2804 has quit IRC11:02
*** stendulker has quit IRC11:04
openstackgerritIlya Etingof proposed openstack/ironic master: Add systemd unit for sushy emulator in devstack  https://review.openstack.org/64482211:04
openstackgerritMerged openstack/networking-generic-switch master: Update master for stable/stein  https://review.openstack.org/64465811:10
openstackgerritMerged openstack/networking-generic-switch stable/stein: Update .gitreview for stable/stein  https://review.openstack.org/64465611:10
*** jtomasek has quit IRC11:14
*** dsneddon has joined #openstack-ironic11:14
openstackgerritIlya Etingof proposed openstack/ironic master: Add systemd unit for sushy emulator in devstack  https://review.openstack.org/64482211:17
*** dsneddon has quit IRC11:20
*** bfournie has quit IRC11:22
*** jtomasek has joined #openstack-ironic11:25
openstackgerritMerged openstack/python-ironic-inspector-client master: Deprecates "uuid" parameters to Python calls when node is expected  https://review.openstack.org/64256411:27
*** rcernin has joined #openstack-ironic11:27
openstackgerritMerged openstack/ironic master: Advance python-dracclient version requirement  https://review.openstack.org/64371611:29
*** pcaruana has quit IRC11:38
*** tssurya has joined #openstack-ironic11:47
*** S4ren has quit IRC11:49
*** dsneddon has joined #openstack-ironic11:51
*** rh-jelabarre has joined #openstack-ironic11:52
*** bfournie has joined #openstack-ironic11:57
*** dsneddon has quit IRC11:57
*** andrein has quit IRC12:05
*** e0ne has quit IRC12:07
*** andrein has joined #openstack-ironic12:07
*** iury|lunch is now known as iurygregory12:14
*** pcaruana has joined #openstack-ironic12:16
*** dsneddon has joined #openstack-ironic12:17
*** dsneddon has quit IRC12:21
*** S4ren has joined #openstack-ironic12:21
*** Pragya2804 has joined #openstack-ironic12:25
*** S4ren has quit IRC12:26
*** S4ren has joined #openstack-ironic12:28
openstackgerritMerged openstack/ironic master: Update release-mappings and api version data for Stein release  https://review.openstack.org/64423312:32
openstackgerritMerged openstack/ironic master: Add release note on conntrack issue on bionic  https://review.openstack.org/64452912:45
*** dsneddon has joined #openstack-ironic12:48
*** diga has joined #openstack-ironic12:51
*** dsneddon has quit IRC12:54
*** mjturek has joined #openstack-ironic13:00
*** hwoarang has quit IRC13:00
*** hwoarang has joined #openstack-ironic13:01
*** yaawang has quit IRC13:02
*** yaawang has joined #openstack-ironic13:07
*** rcernin has quit IRC13:16
*** e0ne has joined #openstack-ironic13:16
openstackgerritIlya Etingof proposed openstack/ironic master: Add systemd unit for vbmcd in devstack  https://review.openstack.org/63797113:19
*** dsneddon has joined #openstack-ironic13:25
mjturekfyi the ironic pkvmci job is under investigation again.. current error is http://paste.openstack.org/show/748121/ if anyone has any idea what may be causing it please let me know13:28
openstackgerritIlya Etingof proposed openstack/ironic master: Add systemd unit for vbmcd in devstack  https://review.openstack.org/63797113:28
*** dsneddon has quit IRC13:30
rpittaumjturek, that looks like a package missing maybe ? I guess libvirt-bin on bionic, this patch should fix that https://review.openstack.org/64250713:31
patchbotpatch 642507 - ironic - Update dist filter for devstack ubuntu - 1 patch set13:31
rpittaumjturek, it's actually libvirt-daemon-system, which is a dependency for libvirt-bin13:32
TheJuliagood morning fellow humans13:33
dtantsurmorning TheJulia13:33
*** dsneddon has joined #openstack-ironic13:33
rpittauhi TheJulia :)13:33
*** sthussey has joined #openstack-ironic13:34
mjturekgm TheJulia :)13:34
mjturekrpittau oh interestting...13:34
rpittaumjturek, yep, hopefully it will be merged soon13:35
mjturekthanks for the headsup rpittau! If I don't see it merge today I'll give it a shot locally13:35
rpittaumjturek, yup np13:35
*** dsneddon has quit IRC13:38
TheJuliadtantsur: any ideas on my attempt to fix our random unit test failures?13:38
openstackgerritOpenStack Release Bot proposed openstack/ironic-python-agent stable/stein: Update .gitreview for stable/stein  https://review.openstack.org/64485613:41
openstackgerritOpenStack Release Bot proposed openstack/ironic-python-agent stable/stein: Update UPPER_CONSTRAINTS_FILE for stable/stein  https://review.openstack.org/64485713:41
openstackgerritOpenStack Release Bot proposed openstack/ironic-python-agent master: Update master for stable/stein  https://review.openstack.org/64485813:41
dtantsurTheJulia: which one?13:41
openstackgerritRiccardo Pittau proposed openstack/ironic master: Workaround for postgres job with ubuntu bionic  https://review.openstack.org/64454013:42
dtantsurif it's about exceptions, the patch is unfortunately incorrect. there has to be a different root cause.13:42
dtantsurTheJulia: wild guess: we may need to apply my __str__ fix to ironic-lib13:43
iurygregorygood morning TheJulia13:43
TheJuliadtantsur: these have nothing to do with ironic-lib13:43
dtantsurhmm, no, ironic-lib is fine. then it's something else.13:43
* TheJulia starts making coffee and distributing coffee mugs13:44
*** baha has joined #openstack-ironic13:48
openstackgerritMerged openstack/networking-generic-switch stable/stein: Update UPPER_CONSTRAINTS_FILE for stable/stein  https://review.openstack.org/64465713:49
TheJuliadtantsur: the only other thing that I thought was maybe that the base exception handling code in __init__ was somehow going sideways, but looking at it and the errors, it seemed to me that we we're expecting behavior that python doesn't want to do. I guess __init__ could be re-written if we could figure out why...13:51
openstackgerritIlya Etingof proposed openstack/ironic master: Add systemd unit for vbmcd in devstack  https://review.openstack.org/63797113:52
openstackgerritRiccardo Pittau proposed openstack/ironic master: Implement basic interfaces for GraphicalConsole Interface  https://review.openstack.org/54735613:55
openstackgerritMerged openstack/bifrost master: Add log file argument for ironic services  https://review.openstack.org/63959014:09
dtantsurTheJulia: are we ready for the bifrost release ^^?14:10
*** dsneddon has joined #openstack-ironic14:12
*** mjturek has quit IRC14:17
*** dsneddon has quit IRC14:17
*** derekh has quit IRC14:31
*** Pragya2804 has quit IRC14:34
*** dsneddon has joined #openstack-ironic14:35
TheJuliadtantsur: I think so yes14:37
dtantsurTheJulia: should I submit one?14:38
*** dsneddon has quit IRC14:40
*** dsneddon has joined #openstack-ironic14:44
* dtantsur did14:48
*** dsneddon has quit IRC14:49
openstackgerritOpenStack Release Bot proposed openstack/ironic-inspector stable/stein: Update .gitreview for stable/stein  https://review.openstack.org/64488214:49
openstackgerritOpenStack Release Bot proposed openstack/ironic-inspector stable/stein: Update UPPER_CONSTRAINTS_FILE for stable/stein  https://review.openstack.org/64488314:49
openstackgerritOpenStack Release Bot proposed openstack/ironic-inspector master: Update master for stable/stein  https://review.openstack.org/64488414:49
*** dtantsur is now known as dtantsur|bbl14:51
openstackgerritOpenStack Release Bot proposed openstack/ironic-ui stable/stein: Update .gitreview for stable/stein  https://review.openstack.org/64488714:52
openstackgerritOpenStack Release Bot proposed openstack/ironic-ui stable/stein: Update UPPER_CONSTRAINTS_FILE for stable/stein  https://review.openstack.org/64488814:52
openstackgerritOpenStack Release Bot proposed openstack/ironic-ui master: Update master for stable/stein  https://review.openstack.org/64488914:52
TheJuliadtantsur|bbl: thanks14:57
*** mjturek has joined #openstack-ironic14:58
*** dsneddon has joined #openstack-ironic15:00
*** manjeets_ is now known as manjeets15:02
*** hjensas has quit IRC15:07
*** hwoarang has quit IRC15:09
*** hwoarang has joined #openstack-ironic15:10
*** S4ren has quit IRC15:20
openstackgerritMerged openstack/ironic-python-agent master: Update master for stable/stein  https://review.openstack.org/64485815:22
openstackgerritMerged openstack/ironic-python-agent stable/stein: Update .gitreview for stable/stein  https://review.openstack.org/64485615:22
openstackgerritMerged openstack/ironic-ui stable/stein: Update .gitreview for stable/stein  https://review.openstack.org/64488715:22
*** altlogbot_1 has joined #openstack-ironic15:29
*** altlogbot_1 has quit IRC15:30
openstackgerritRiccardo Pittau proposed openstack/ironic master: Workaround for postgres job with ubuntu bionic  https://review.openstack.org/64454015:38
*** w14161_1_ has quit IRC15:45
*** gyee has joined #openstack-ironic15:51
*** diga has quit IRC15:57
*** diga has joined #openstack-ironic16:00
*** Lucas_Gray has quit IRC16:03
*** efried is now known as efried_rolling16:03
*** dtantsur|bbl is now known as dtantsur16:22
*** trown is now known as trown|lunch16:25
*** mjturek has quit IRC16:30
*** priteau has joined #openstack-ironic16:33
openstackgerritMerged openstack/ironic-inspector master: Update master for stable/stein  https://review.openstack.org/64488416:34
openstackgerritMerged openstack/ironic-ui master: Update master for stable/stein  https://review.openstack.org/64488916:35
*** altlogbot_3 has joined #openstack-ironic16:39
*** baha has quit IRC16:40
*** altlogbot_3 has quit IRC16:41
openstackgerritMerged openstack/python-ironic-inspector-client stable/stein: Update UPPER_CONSTRAINTS_FILE for stable/stein  https://review.openstack.org/64414916:42
*** andrein has quit IRC16:45
openstackgerritMerged openstack/ironic-python-agent stable/stein: Update UPPER_CONSTRAINTS_FILE for stable/stein  https://review.openstack.org/64485716:45
*** mjturek has joined #openstack-ironic16:46
*** hjensas has joined #openstack-ironic16:47
*** yolanda has quit IRC16:49
*** Chaserjim has joined #openstack-ironic16:58
ChaserjimQQ - what is the most proper way to kill an on-going introspection?16:58
*** baha has joined #openstack-ironic17:03
dtantsurChaserjim: `openstack baremetal node abort <node>`?17:07
*** e0ne has quit IRC17:07
*** S4ren has joined #openstack-ironic17:14
*** diga has quit IRC17:17
*** mjturek has quit IRC17:19
*** trown|lunch is now known as trown17:20
*** mbeierl has quit IRC17:20
rpittaugood night! o/17:22
*** altlogbot_3 has joined #openstack-ironic17:24
*** altlogbot_3 has quit IRC17:24
*** altlogbot_3 has joined #openstack-ironic17:26
*** altlogbot_3 has quit IRC17:28
*** altlogbot_1 has joined #openstack-ironic17:29
openstackgerritMerged openstack/ironic-ui stable/stein: Update UPPER_CONSTRAINTS_FILE for stable/stein  https://review.openstack.org/64488817:42
arne_wiebalckbye, everyone o/17:42
*** S4ren has quit IRC17:43
openstackgerritMerged openstack/networking-baremetal stable/stein: Update UPPER_CONSTRAINTS_FILE for stable/stein  https://review.openstack.org/64466017:43
openstackgerritMerged openstack/ironic-specs master: Fixing linters test  https://review.openstack.org/64480017:49
openstackgerritIlya Etingof proposed openstack/ironic master: Add systemd unit for vbmcd in devstack  https://review.openstack.org/63797117:50
*** mbeierl has joined #openstack-ironic17:53
*** dtantsur is now known as dtantsur|afk17:53
dtantsur|afk\o17:54
openstackgerritIlya Etingof proposed openstack/ironic master: Respect $USE_PYTHON3 settings for gunicorn  https://review.openstack.org/64462217:54
*** dougsz has quit IRC17:58
*** mbeierl has quit IRC17:58
*** priteau has quit IRC18:04
openstackgerritIlya Etingof proposed openstack/ironic master: Add systemd unit for sushy emulator in devstack  https://review.openstack.org/64482218:06
*** mjturek has joined #openstack-ironic18:07
*** rpioso|afk is now known as rpioso18:11
rpiosoGood morning, ironicers18:12
*** mbeierl has joined #openstack-ironic18:14
*** e0ne has joined #openstack-ironic18:17
tssuryajroll: Hi! I had a couple of questions regarding your comment here: https://review.openstack.org/#/c/636132/1/specs/train/approved/nova-support-instance-power-update.rst@17 are you around now?18:18
patchbotpatch 636132 - nova-specs - Support server power state update through external... - 1 patch set18:18
jrolltssurya: I was just stepping away for a few minutes, but go ahead and ask (here or in the review) and I'll respond when I'm back18:19
tssuryajroll: so regarding ironic having the knowledge regarding at which instance is the machine source of truth versus db.. how does ironic know that ?18:20
*** irclogbot_1 has quit IRC18:21
tssurya(FYI: I am newbie to ironic)18:21
jrolltssurya: we have the "force_power_state_during_sync" config option in ironic-conductor - "During sync_power_state, should the hardware power state be set to the state recorded in the database (True) or should the database be updated based on the hardware state (False)."18:22
*** irclogbot_2 has joined #openstack-ironic18:24
tssuryaah ok18:24
tssuryahowever this is for all the nodes right ? either we regard the db as truth or hardware ?18:25
*** hjensas has quit IRC18:26
*** irclogbot_2 has quit IRC18:27
*** irclogbot_0 has joined #openstack-ironic18:28
tssuryaok I think now I get what you meant: something like we should just send the events to nova when we find that the hardware and db state don't match if force_power_state_during_sync is False and not for every ironic polling right ?18:29
tssuryaI can leave a comment on the review regarding this18:29
tssuryasorry for the bad timing18:29
jrolltssurya: no worries18:29
jrollyes, for all the nodes. and yes, do it based on that config option being False - if we update our database to match, we should also update nova18:30
tssuryajroll: makes sense!18:30
*** PrinzElvis has quit IRC18:30
tssuryaalso just to make sure we are on the same page, we are talking about introducing "events" to nova right ? meaning you are ok with me introducing connection to novaclient in ironic ?18:31
tssuryadoes this need a spec on the ironic side ?18:31
*** NobodyCam has quit IRC18:31
*** pas-ha has quit IRC18:31
*** dirk has quit IRC18:31
*** PrinzElvis_ has joined #openstack-ironic18:33
*** pas-ha has joined #openstack-ironic18:34
*** NobodyCam has joined #openstack-ironic18:34
*** dirk has joined #openstack-ironic18:34
*** S4ren has joined #openstack-ironic18:37
larsksjroll: curious if you have any time to look at https://review.openstack.org/#/c/642734/?18:37
patchbotpatch 642734 - ironic - prevent serial console lockout - 5 patch sets18:37
jrolltssurya: that's fine with me - probably just an RFE bug would be enough. see ... well why can't I find these docs now18:40
tssuryajroll: awesome yea I we have an RFE here https://storyboard.openstack.org/#!/story/200496918:41
jrolltssurya: aha, excellent. thanks! sounds like we're just waiting to sort out the nova spec :)18:42
jrolllarsks: I'll put it on the list, hopefully tomorrow or friday18:42
larsksAwesome, thank you!18:42
tssuryajroll: yep I will put out a few WIPs first and then ping the nova cores18:43
tssuryawill keep you posted18:43
jroll++ thanks18:43
tssuryajroll: also last question for the evening:18:43
tssuryawe would also need to send the event for an explicit "baremetal node set power on" API call right ?18:43
tssuryaother than just during the ironic polling18:44
jrolltssurya: yes18:44
tssuryacool ! I'll start on the WIP code on both sides then18:44
jrolltssurya: which makes me wonder if we should also send one if force_power_state_during_sync is True as well18:44
tssuryaum why >18:45
jrollheh18:45
jrollmaybe not? I haven't fully thought it through18:45
jrollI guess nova would already have the db state, which is what we're forcing it to18:45
jrollthe code would be easier though, just send for every power state change18:46
tssuryayea true18:46
tssuryawe could keep it consistent on the ironic side18:46
tssuryaand just add a workaround config on nova side for the users to choose if they want to allow ironic to be the source of truth18:47
jrollyeah, for backwards compatibility?18:47
tssuryayea18:47
jrollmakes sense18:47
tssuryathanks a lot jroll! got to head home now, have a nice day! I'll update the spec and the code for this soon18:49
jrolltssurya: thanks, have a good evening :)18:49
*** e0ne has quit IRC18:53
openstackgerritMerged openstack/ironic-tempest-plugin master: Test BM with VM on the same network  https://review.openstack.org/63659818:54
*** altlogbot_1 has quit IRC19:00
*** altlogbot_1 has joined #openstack-ironic19:01
*** PrinzElvis_ is now known as PrinzElvis19:05
*** altlogbot_1 has quit IRC19:06
*** tssurya has quit IRC19:07
*** altlogbot_1 has joined #openstack-ironic19:07
*** efried_rolling is now known as efried19:11
*** irclogbot_0 has quit IRC19:16
*** tssurya has joined #openstack-ironic19:16
*** irclogbot_0 has joined #openstack-ironic19:17
*** hjensas has joined #openstack-ironic19:18
*** irclogbot_0 has quit IRC19:30
*** irclogbot_0 has joined #openstack-ironic19:31
*** altlogbot_1 has quit IRC19:34
*** altlogbot_1 has joined #openstack-ironic19:35
*** hjensas has quit IRC19:45
*** dhellmann has quit IRC19:57
openstackgerritMerged openstack/ironic master: Update dist filter for devstack ubuntu  https://review.openstack.org/64250719:58
*** hjensas has joined #openstack-ironic19:58
*** dhellmann has joined #openstack-ironic19:59
*** andrein has joined #openstack-ironic20:09
*** problem_v has joined #openstack-ironic20:23
*** dtruong has joined #openstack-ironic20:23
problem_vHey! I am working on setting up a standalone ironic and I had a few questions about getting the CLI working.20:23
problem_vAccording to the docs the only env variables I need to set are OS_TOKEN, and OS_URL but when I do that the openstack command still errors on me saying "Missing value auth-url required for auth plugin password"20:24
problem_vIs there a step I am mis-understanding/missing?20:25
TheJuliaproblem_v: what commands are you trying to use?20:26
problem_vopenstack baremetal chassis list20:28
problem_vI was able to register a chassis with the ironic command but I didn't want to use it because of the warning of depreciation in stein.20:29
problem_vand using the command "ironic chassis-list" totally works as expected.20:29
problem_vBecause I intend to roll out ironic for HV provisioning I was hoping to be able to avoid setting up a separate keystone for this thing :).20:32
*** andrein has quit IRC20:38
*** yajushiSri has quit IRC20:44
*** andrein has joined #openstack-ironic20:44
*** irclogbot_0 has quit IRC20:45
*** S4ren has quit IRC20:46
*** irclogbot_3 has joined #openstack-ironic20:46
openstackgerritOpenStack Release Bot proposed openstack/bifrost stable/stein: Update .gitreview for stable/stein  https://review.openstack.org/64500520:49
openstackgerritOpenStack Release Bot proposed openstack/bifrost stable/stein: Update UPPER_CONSTRAINTS_FILE for stable/stein  https://review.openstack.org/64500620:49
openstackgerritOpenStack Release Bot proposed openstack/bifrost master: Update master for stable/stein  https://review.openstack.org/64500720:49
*** S4ren has joined #openstack-ironic20:49
*** e0ne has joined #openstack-ironic20:52
*** altlogbot_1 has quit IRC20:53
*** trown is now known as trown|outtypewww20:53
*** altlogbot_3 has joined #openstack-ironic20:54
*** S4ren has quit IRC20:58
*** Lucas_Gray has joined #openstack-ironic21:07
*** mjturek has quit IRC21:07
*** baha has quit IRC21:08
*** bfournie has quit IRC21:21
*** whoami-rajat has quit IRC21:29
*** e0ne has quit IRC22:00
*** pcaruana has quit IRC22:11
problem_vTheJulia I got it figured out btw I guess if you have an IRONIC_URL variable set it still trys to auth through keystone,22:12
problem_vDidn't realize that. :)22:12
TheJuliathe "openstack  baremetal" commands right?22:13
problem_vyeah22:13
problem_vI ended up unsetting all of my env variables and then just setting the ones described in the doc and it worked the way i expected it to :)22:14
TheJuliaYeah, its... a little... tricky22:16
TheJulialargely because it does a lot of pre-flight actions22:16
problem_vYeah it seems like not a lot of people run this thing in stand alone mode.22:16
TheJuliadedicated OS_CLOUD configurations22:17
TheJuliaor use the env-vars we have in bifrost22:17
*** Lucas_Gray has quit IRC22:51
*** w14161_1 has joined #openstack-ironic22:56
*** rcernin has joined #openstack-ironic22:57
*** rcernin has quit IRC22:58
*** rcernin has joined #openstack-ironic22:58
*** bfournie has joined #openstack-ironic23:03
*** andrein has quit IRC23:29
*** sthussey has quit IRC23:53

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