| openstackgerrit | A change was merged to openstack/baremetal-specs: Initial commit https://review.openstack.org/94922 | 00:19 |
|---|---|---|
| *** matsuhashi has joined #openstack-ironic | 00:23 | |
| *** killer_prince has quit IRC | 00:25 | |
| *** igor_ has joined #openstack-ironic | 00:30 | |
| *** igor_ has quit IRC | 00:34 | |
| *** Penick has joined #openstack-ironic | 00:38 | |
| *** Penick has quit IRC | 00:43 | |
| *** yongli has joined #openstack-ironic | 00:51 | |
| *** rloo has quit IRC | 00:51 | |
| *** r-daneel has joined #openstack-ironic | 01:17 | |
| *** blamar_ has joined #openstack-ironic | 01:30 | |
| *** igor_ has joined #openstack-ironic | 01:31 | |
| *** blamar has quit IRC | 01:31 | |
| *** blamar_ is now known as blamar | 01:31 | |
| *** killer_prince has joined #openstack-ironic | 01:33 | |
| *** igor_ has quit IRC | 01:35 | |
| *** nosnos has joined #openstack-ironic | 01:35 | |
| openstackgerrit | Adam Gandelman proposed a change to openstack/ironic: Serialize dd writes to iSCSI targets https://review.openstack.org/95051 | 01:35 |
| *** dkehn_ has joined #openstack-ironic | 01:48 | |
| *** lazy_prince has joined #openstack-ironic | 01:57 | |
| *** killer_prince has quit IRC | 01:58 | |
| *** dkehnx has quit IRC | 01:58 | |
| *** lazy_prince is now known as killer_prince | 01:58 | |
| *** r-daneel has quit IRC | 01:59 | |
| *** dkehn_ is now known as dkehnx | 02:02 | |
| openstackgerrit | Adam Gandelman proposed a change to openstack/ironic: Serialize dd writes to iSCSI targets https://review.openstack.org/95051 | 02:08 |
| *** radsy has quit IRC | 02:20 | |
| *** igor_ has joined #openstack-ironic | 02:32 | |
| *** igor_ has quit IRC | 02:36 | |
| *** killer_prince has quit IRC | 02:44 | |
| *** lazy_prince has joined #openstack-ironic | 02:47 | |
| *** lazy_prince is now known as killer_prince | 02:48 | |
| *** harlowja_ is now known as harlowja_away | 02:55 | |
| *** zdin0bot has joined #openstack-ironic | 03:00 | |
| *** killer_prince has quit IRC | 03:02 | |
| *** killer_prince has joined #openstack-ironic | 03:02 | |
| *** nosnos has quit IRC | 03:03 | |
| *** zdin0bot has quit IRC | 03:11 | |
| *** ramineni has joined #openstack-ironic | 03:18 | |
| *** ramineni has left #openstack-ironic | 03:20 | |
| *** zdin0bot has joined #openstack-ironic | 03:21 | |
| *** zdin0bot has quit IRC | 03:22 | |
| *** zdin0bot has joined #openstack-ironic | 03:22 | |
| *** ramineni has joined #openstack-ironic | 03:23 | |
| *** matsuhashi has quit IRC | 03:29 | |
| *** eghobo has joined #openstack-ironic | 03:32 | |
| *** igor_ has joined #openstack-ironic | 03:32 | |
| *** matsuhashi has joined #openstack-ironic | 03:36 | |
| *** igor_ has quit IRC | 03:37 | |
| *** matsuhashi has quit IRC | 03:43 | |
| *** matsuhashi has joined #openstack-ironic | 03:43 | |
| *** matsuhas_ has joined #openstack-ironic | 03:45 | |
| *** matsuhashi has quit IRC | 03:45 | |
| *** nosnos has joined #openstack-ironic | 03:45 | |
| *** k4n0 has joined #openstack-ironic | 03:46 | |
| *** OldCrowEW has joined #openstack-ironic | 03:59 | |
| *** killer_prince has quit IRC | 04:04 | |
| *** killer_prince has joined #openstack-ironic | 04:05 | |
| *** coolsvap|afk is now known as coolsvap | 04:05 | |
| *** killer_prince has quit IRC | 04:14 | |
| *** zdin0bot has quit IRC | 04:24 | |
| *** eghobo has quit IRC | 04:28 | |
| *** matsuhas_ has quit IRC | 04:34 | |
| *** killer_prince has joined #openstack-ironic | 04:41 | |
| *** matsuhashi has joined #openstack-ironic | 04:43 | |
| *** radsy has joined #openstack-ironic | 04:44 | |
| *** radsy has joined #openstack-ironic | 04:44 | |
| *** eghobo has joined #openstack-ironic | 04:44 | |
| *** zdin0bot has joined #openstack-ironic | 04:55 | |
| *** zdin0bot has quit IRC | 04:56 | |
| *** zdin0bot has joined #openstack-ironic | 04:56 | |
| *** zdin0bot has quit IRC | 05:01 | |
| *** lazy_prince3 has joined #openstack-ironic | 05:08 | |
| *** OldCrowEW has quit IRC | 05:10 | |
| *** eghobo has quit IRC | 05:15 | |
| *** eguz has joined #openstack-ironic | 05:15 | |
| *** eghobo has joined #openstack-ironic | 05:15 | |
| *** eguz has quit IRC | 05:20 | |
| *** igor_ has joined #openstack-ironic | 05:34 | |
| *** igor_ has quit IRC | 05:38 | |
| *** eghobo has joined #openstack-ironic | 05:42 | |
| *** lazy_prince3 has quit IRC | 05:51 | |
| *** zdin0bot has joined #openstack-ironic | 05:57 | |
| *** eghobo has quit IRC | 05:58 | |
| *** zdin0bot has quit IRC | 06:02 | |
| openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/ironic: Imported Translations from Transifex https://review.openstack.org/95078 | 06:02 |
| *** radsy has quit IRC | 06:12 | |
| *** geertj has joined #openstack-ironic | 06:26 | |
| *** mikir has joined #openstack-ironic | 06:30 | |
| *** lazy_prince3 has joined #openstack-ironic | 06:48 | |
| *** zdin0bot has joined #openstack-ironic | 06:58 | |
| *** jcoufal has joined #openstack-ironic | 07:00 | |
| *** lazy_prince3 is now known as lazy_prince | 07:02 | |
| *** zdin0bot has quit IRC | 07:03 | |
| openstackgerrit | Sirushti Murugesan proposed a change to openstack/ironic: Adds support for deploying whole disk images https://review.openstack.org/73054 | 07:10 |
| *** killer_prince has quit IRC | 07:12 | |
| *** mikir has quit IRC | 07:20 | |
| *** foexle has joined #openstack-ironic | 07:20 | |
| *** killer_p- has joined #openstack-ironic | 07:20 | |
| *** killer_p- is now known as killer_prince | 07:20 | |
| *** mrda is now known as mrda-weekend | 07:24 | |
| mrda-weekend | seeya later y'all | 07:27 |
| *** ifarkas has joined #openstack-ironic | 07:29 | |
| *** igor has joined #openstack-ironic | 07:42 | |
| *** jistr has joined #openstack-ironic | 07:43 | |
| *** ndipanov has joined #openstack-ironic | 07:47 | |
| *** pelix has joined #openstack-ironic | 07:50 | |
| *** rwsu has quit IRC | 07:58 | |
| *** zdin0bot has joined #openstack-ironic | 07:58 | |
| *** derekh_ has joined #openstack-ironic | 08:00 | |
| *** Mikhail_D_ltp has joined #openstack-ironic | 08:01 | |
| *** zdin0bot has quit IRC | 08:03 | |
| *** ifarkas has quit IRC | 08:27 | |
| *** rameshg87 has joined #openstack-ironic | 08:28 | |
| *** lucasagomes has joined #openstack-ironic | 08:45 | |
| *** matsuhashi has quit IRC | 08:48 | |
| Mikhail_D_ltp | Good morning all! :) | 08:50 |
| agordeev | Mikhail_D_ltp: morning | 08:50 |
| *** shausy has joined #openstack-ironic | 09:00 | |
| dtantsur | morning, Ironic! | 09:02 |
| *** athomas has joined #openstack-ironic | 09:13 | |
| *** matsuhashi has joined #openstack-ironic | 09:16 | |
| agordeev | dtantsur: morning! | 09:17 |
| *** jcoufal has quit IRC | 09:23 | |
| *** jcoufal has joined #openstack-ironic | 09:23 | |
| openstackgerrit | A change was merged to openstack/ironic: Cleanup images after deployment https://review.openstack.org/92340 | 09:39 |
| *** datajerk has joined #openstack-ironic | 09:43 | |
| *** nosnos has quit IRC | 09:45 | |
| *** athomas has quit IRC | 09:57 | |
| *** ifarkas has joined #openstack-ironic | 10:01 | |
| *** athomas has joined #openstack-ironic | 10:04 | |
| *** datajerk has quit IRC | 10:11 | |
| *** matsuhashi has quit IRC | 10:12 | |
| *** matsuhas_ has joined #openstack-ironic | 10:15 | |
| *** openstackstatus has quit IRC | 10:20 | |
| *** openstack has joined #openstack-ironic | 10:21 | |
| *** openstackstatus has joined #openstack-ironic | 10:21 | |
| *** ChanServ sets mode: +v openstackstatus | 10:21 | |
| openstackgerrit | Lucas Alvares Gomes proposed a change to openstack/ironic: Remove 'node' parameter from the Console and Rescue interfaces https://review.openstack.org/91097 | 10:23 |
| openstackgerrit | Lucas Alvares Gomes proposed a change to openstack/ironic: Remove 'node' parameter from the Power interface https://review.openstack.org/91093 | 10:23 |
| openstackgerrit | Lucas Alvares Gomes proposed a change to openstack/ironic: Remove 'node' parameter from the Deploy interface https://review.openstack.org/91062 | 10:23 |
| openstackgerrit | Lucas Alvares Gomes proposed a change to openstack/ironic: Remove 'node' parameter from the validate() methods https://review.openstack.org/91104 | 10:23 |
| *** Mikhail_D_ltp has quit IRC | 10:23 | |
| *** datajerk has joined #openstack-ironic | 10:25 | |
| *** datajerk has quit IRC | 10:29 | |
| *** coolsvap is now known as coolsvap|afk | 10:38 | |
| openstackgerrit | Lucas Alvares Gomes proposed a change to openstack/ironic: Move PXE instance level parameters to instance_info https://review.openstack.org/94855 | 10:43 |
| openstackgerrit | Lucas Alvares Gomes proposed a change to openstack/ironic: Move PXE instance level parameters to instance_info https://review.openstack.org/94855 | 10:46 |
| openstackgerrit | Sandhya Balakrishnan proposed a change to openstack/ironic: Update Ironic Guide post landing for 569c0f395a9 https://review.openstack.org/94604 | 10:50 |
| *** sabah has joined #openstack-ironic | 10:50 | |
| *** geertj has quit IRC | 10:52 | |
| *** subah has joined #openstack-ironic | 10:55 | |
| *** sabah has quit IRC | 10:55 | |
| *** matsuhas_ has quit IRC | 10:55 | |
| *** matsuhashi has joined #openstack-ironic | 10:57 | |
| *** matsuhashi has quit IRC | 10:58 | |
| *** matsuhashi has joined #openstack-ironic | 10:58 | |
| *** matsuhashi has quit IRC | 10:58 | |
| *** matsuhashi has joined #openstack-ironic | 10:59 | |
| *** rameshg87 has left #openstack-ironic | 11:02 | |
| openstackgerrit | Sandhya Balakrishnan proposed a change to openstack/ironic: Update Ironic Guide post landing for 569c0f395a9 https://review.openstack.org/94604 | 11:04 |
| *** igor_ has joined #openstack-ironic | 11:05 | |
| dtantsur | NobodyCam, once you're here and have your morning coffee :) could you review/approve https://review.openstack.org/#/c/92625 please? | 11:05 |
| *** ramineni has left #openstack-ironic | 11:07 | |
| *** igor has quit IRC | 11:08 | |
| openstackgerrit | Lucas Alvares Gomes proposed a change to openstack/ironic: Move PXE instance level parameters to instance_info https://review.openstack.org/94855 | 11:13 |
| *** subah has quit IRC | 11:13 | |
| *** subah has joined #openstack-ironic | 11:13 | |
| *** subah has quit IRC | 11:17 | |
| *** sabah has joined #openstack-ironic | 11:17 | |
| *** shausy has quit IRC | 11:26 | |
| *** sabah has quit IRC | 11:31 | |
| *** lucasagomes is now known as lucas-hungry | 11:39 | |
| *** k4n0 has quit IRC | 11:42 | |
| dtantsur | sirushti, yeah, I would vote for moving get_dev there, but changing the signature is not the best thing to do | 12:02 |
| dtantsur | (for those out of context, it's about https://review.openstack.org/#/c/73054/15/ironic/drivers/modules/deploy_utils.py ) | 12:02 |
| sirushti | dtantsur, right, is there a better way? | 12:03 |
| dtantsur | I'm thinking | 12:03 |
| sirushti | I didn't want to change the signature, so left get_dev there | 12:03 |
| sirushti | but thats sort of like duplicated code | 12:03 |
| sirushti | s/sort of like/is/ | 12:03 |
| dtantsur | what about replacing @decorator with context manager like: with iscsi_deploy(address, port, iqn, lun) as dev: ... | 12:04 |
| sirushti | also, we're passing **params in pxe.py and making required parameters iqn,lun, *args, **kwargs would probably change the way we send **params too from pxe.py | 12:05 |
| dtantsur | not, Python should be smart enough :) | 12:05 |
| sirushti | Oh :-) my bad, thanks! | 12:06 |
| dtantsur | gimme some time, I'll show you an example of what I mean | 12:06 |
| sirushti | could you elaborate on 'what about replacing @decorator with context manager like...'? | 12:07 |
| sirushti | Okay, cool :-) | 12:07 |
| dtantsur | sirushti, http://paste.openstack.org/show/81214/ | 12:12 |
| dtantsur | how do you like this approach? | 12:12 |
| sirushti | dtantsur, checking | 12:12 |
| dtantsur | https://docs.python.org/2.6/library/contextlib.html is what I rely on here | 12:13 |
| sirushti | dtantsur, Ooh, this eliminates kwargs and changing method signatures :-) Looks fantastic | 12:17 |
| dtantsur | Python power to the rescue :) | 12:17 |
| *** matsuhashi has quit IRC | 12:20 | |
| *** OldCrowEW has joined #openstack-ironic | 12:21 | |
| *** matsuhashi has joined #openstack-ironic | 12:21 | |
| sirushti | dtantsur, Alright, will make those changes, thanks! | 12:21 |
| dtantsur | np | 12:21 |
| *** matsuhashi has quit IRC | 12:26 | |
| *** foexle has quit IRC | 12:28 | |
| *** geertj has joined #openstack-ironic | 12:32 | |
| *** lazy_prince has quit IRC | 12:32 | |
| *** jdob has joined #openstack-ironic | 12:37 | |
| *** matsuhashi has joined #openstack-ironic | 12:37 | |
| *** matsuhashi has quit IRC | 12:38 | |
| *** matsuhashi has joined #openstack-ironic | 12:38 | |
| *** matsuhashi has quit IRC | 12:38 | |
| *** matsuhashi has joined #openstack-ironic | 12:39 | |
| *** matsuhashi has quit IRC | 12:43 | |
| *** lucas-hungry is now known as lucasagomes | 12:43 | |
| *** jcoufal has quit IRC | 12:53 | |
| *** jcoufal has joined #openstack-ironic | 12:54 | |
| *** jbjohnso has joined #openstack-ironic | 12:54 | |
| *** jgrimm has joined #openstack-ironic | 12:57 | |
| *** athomas has quit IRC | 12:57 | |
| *** jcoufal has quit IRC | 13:00 | |
| *** jcoufal has joined #openstack-ironic | 13:01 | |
| *** athomas has joined #openstack-ironic | 13:05 | |
| NobodyCam | good morning Ironic | 13:06 |
| dtantsur | morning NobodyCam :) | 13:07 |
| NobodyCam | morning dtantsur | 13:08 |
| NobodyCam | :) | 13:08 |
| NobodyCam | TGIF! | 13:08 |
| dtantsur | :) | 13:08 |
| dtantsur | NobodyCam, may I ask you to review https://review.openstack.org/#/c/92625 ? It's Yet Another (tm) patch on caching in PXE, already got one +2 | 13:10 |
| NobodyCam | dtantsur: sure... | 13:11 |
| dtantsur | thnx! | 13:11 |
| dtantsur | oh, btw! lucasagomes, do you have any updates on https://bugs.launchpad.net/ironic/+bug/1322179 ? I'd like to start doing something about it | 13:11 |
| lucasagomes | dtantsur, hey no I haven't :( I'm currently working on the instance_info thingy so I didn't have much time to look into it | 13:12 |
| lucasagomes | NobodyCam, morning TGIF! | 13:12 |
| lucasagomes | dtantsur, g'afternoon :) | 13:12 |
| dtantsur | lucasagomes, g'afternoon :) I'll start investigating myself than - knowing how Nova works with us should not harm ;) | 13:13 |
| NobodyCam | lucasagomes: good morning... (afternoon for you) | 13:14 |
| lucasagomes | dtantsur, :) yeah | 13:14 |
| lucasagomes | dtantsur, basically we expose the Node.properties.local_gb to nova as the size of the disk | 13:15 |
| lucasagomes | you can look at the driver how it works | 13:15 |
| dtantsur | ok, thanks, will look | 13:15 |
| dtantsur | lucasagomes, so may I suppose that we might do some checks on the validate() stage of deploy? | 13:17 |
| lucasagomes | dtantsur, we should | 13:17 |
| dtantsur | (I'm a bit against putting into Nova driver details about how our partitioning work) | 13:17 |
| lucasagomes | dtantsur, we should calculate if the partition layout fits the disk | 13:17 |
| dtantsur | yeah, that's how I understand it | 13:18 |
| lucasagomes | dtantsur, yeah but afair, nova gets this disk size and use it as it was the root partition size | 13:18 |
| lucasagomes | which is wrong | 13:18 |
| lucasagomes | better u look at it, I don't remember exactly and might end up confusing u | 13:18 |
| dtantsur | ack, I'll research (anyway it should be up to me to clarify the bug I created :) | 13:19 |
| NobodyCam | dtantsur: any reason we didn't _() the log at line 138 of image_cache.py? | 13:21 |
| NobodyCam | lucasagomes: up for a review of 93133? | 13:21 |
| NobodyCam | :-p | 13:21 |
| dtantsur | NobodyCam, we no longer translate debug calls - it's an OpenStack policy | 13:22 |
| dtantsur | I can try find a link for it... | 13:22 |
| lucasagomes | NobodyCam, will do, 1min | 13:22 |
| NobodyCam | ah yes... /me starts sipping his coffee faster | 13:23 |
| NobodyCam | lucasagomes: TY :) | 13:23 |
| dtantsur | NobodyCam, https://review.openstack.org/#/c/93291/ based on https://wiki.openstack.org/wiki/LoggingStandards#Log_Translation | 13:23 |
| lucasagomes | NobodyCam, why we are ereasing bs=512 count=2 on the MBR? | 13:25 |
| lucasagomes | count=1 should be enough no? | 13:25 |
| dtantsur | GPT? | 13:25 |
| lucasagomes | ah yeah | 13:26 |
| NobodyCam | :) yep for gpt | 13:26 |
| lucasagomes | just wondering if it would be better to actually check the disk label before wiping that data | 13:26 |
| lucasagomes | see if it's gpt or mbr and calculate from that | 13:27 |
| dtantsur | lucasagomes, our current partitioner never places anything before 1 MiB offset | 13:27 |
| dtantsur | (parted can't do it at all) | 13:27 |
| *** jrist has quit IRC | 13:28 | |
| *** jcoufal has quit IRC | 13:32 | |
| NobodyCam | dtantsur: the break your adding in pxe.py at line 293, is that for performance, ie why nnot let it check all files? | 13:34 |
| dtantsur | NobodyCam, yes for performance, clean up can be time-consuming | 13:36 |
| dtantsur | and I'd like to proceed with deploy asap | 13:36 |
| dtantsur | we'll anyway have clean up after downloading images probably | 13:36 |
| dtantsur | NobodyCam, also in the next patch we'll set goals on how much to clean up | 13:38 |
| *** jrist has joined #openstack-ironic | 13:39 | |
| dtantsur | for the same reason: don't do too much before proceeding to deploy | 13:39 |
| dtantsur | (imaging we're doing the first deploy after a long pause: there'll be a looooong cleanup :) | 13:40 |
| NobodyCam | dtantsur: do you ever see this becomming a periodic task, running every so often? | 13:41 |
| dtantsur | NobodyCam, clean up you mean? Probably | 13:42 |
| dtantsur | but that won't help much: again, the final goal is to make cache clean up is much as it is required (even if it would stop earlier depending on configuration) | 13:43 |
| dtantsur | NobodyCam, this patch's gonna explain more, I guess: https://review.openstack.org/#/c/94371/ | 13:43 |
| *** r-daneel has joined #openstack-ironic | 13:44 | |
| dtantsur | (sorry for not being clear enough - I'm in the middle of a hangout) | 13:45 |
| NobodyCam | dtantsur: not at all :) | 13:47 |
| *** igordcard has joined #openstack-ironic | 13:53 | |
| NobodyCam | kinda makes me wish oslistdir would sort on size :-p | 13:56 |
| NobodyCam | or better datetime, then size | 13:57 |
| lucasagomes | NobodyCam, reviewed, I +1 becuase I will spawn a test here to see if it's working fine in my env | 14:05 |
| lucasagomes | I will revote in a moment | 14:05 |
| lucasagomes | NobodyCam, also I just rebased https://review.openstack.org/#/c/91062/ again after one merge | 14:06 |
| lucasagomes | if u can take a peek once u free | 14:06 |
| dtantsur | NobodyCam, oh yeah, directory listing is not perfect now :) I think they're making it better in Py3k | 14:06 |
| *** rwsu has joined #openstack-ironic | 14:08 | |
| NobodyCam | dtantsur: the code lgtm, but like lucasagomes I will need to test it here. | 14:12 |
| dtantsur | sure :) | 14:12 |
| *** athomas has quit IRC | 14:15 | |
| NobodyCam | oh what will _find_candidates_for_deletion do if cache dir is say a nfs mount with noatime set | 14:15 |
| *** athomas has joined #openstack-ironic | 14:16 | |
| NobodyCam | anyone have a env where that can be tested? | 14:18 |
| NobodyCam | ahh had to expand the code. your already checking ctome too :) | 14:22 |
| NobodyCam | never mind :-p | 14:22 |
| lucasagomes | NobodyCam, should we have a review jam for the refactoring stuff? | 14:28 |
| NobodyCam | lucasagomes: I was kinda thinking about that too but more for all the older reviews that didnot go in to the specs repo | 14:34 |
| lucasagomes | right | 14:35 |
| lucasagomes | yeah there's a couple of them up right now | 14:35 |
| NobodyCam | ugg and I need to file a exp report too :-p | 14:36 |
| lucasagomes | and like the #91062 sereies they r hard to maintain cause they change small bits of many files | 14:36 |
| *** shausy has joined #openstack-ironic | 14:40 | |
| *** shausy has quit IRC | 14:42 | |
| *** shausy has joined #openstack-ironic | 14:43 | |
| *** coolsvap|afk is now known as coolsvap | 14:48 | |
| NobodyCam | lucasagomes: 91062 +2'd | 14:52 |
| lucasagomes | NobodyCam, awesome :D ta much! | 14:52 |
| NobodyCam | if rloo is around??? | 14:53 |
| NobodyCam | dtantsur: building test env now | 14:54 |
| dtantsur | ack | 14:54 |
| jbjohnso | anyone interested in trying out confluent on their nodes to see how it works (cli wise at least) | 14:54 |
| *** geertj has quit IRC | 14:55 | |
| *** sysexit has joined #openstack-ironic | 14:57 | |
| openstackgerrit | Lucas Alvares Gomes proposed a change to openstack/baremetal-specs: Add spec for the new instance_info field of Nodes https://review.openstack.org/95182 | 14:57 |
| openstackgerrit | Lucas Alvares Gomes proposed a change to openstack/baremetal-specs: Add spec for the new instance_info field of Nodes https://review.openstack.org/95182 | 14:58 |
| Ng | just doing a fresh deploy of tripleo on a rack of actual hardware and going seed->undercloud we saw "Node uuid-blah is locked by host". Seems weird that there could be any hosts that are locked and not reserved, at that point. Are we doing something wrong? | 15:03 |
| Ng | (wild hypothesis, we attempted to deploy too quickly after registering the nodes with ironic, so they were still in some initially-locked state before being properly ingested?) | 15:04 |
| NobodyCam | Ng: i would suspect this the perodic task checking power | 15:04 |
| NobodyCam | s/this the/this is the/ | 15:04 |
| *** yongli has quit IRC | 15:05 | |
| NobodyCam | lucasagomes: didn't we land the retry logic for that? | 15:05 |
| * lucasagomes reads | 15:06 | |
| NobodyCam | and morning Ng :) | 15:06 |
| lucasagomes | yes we do have a retry logic in the driver | 15:06 |
| Ng | hey NobodyCam :) | 15:06 |
| NobodyCam | Ng: can you paste a log that we can take a peek at | 15:08 |
| *** igor_ has quit IRC | 15:09 | |
| Ng | NobodyCam: there's a traceback and some client tool output at the bottom of https://etherpad.openstack.org/p/tripleo-end-to-end-automatic-bm | 15:09 |
| Ng | it seems to be intermittent | 15:10 |
| Ng | after we register the nodes, we wait until nova hypervisor-stats shows the number of nodes that we're expecting, so it shouldn't be that we're deploying before ironic is ready | 15:11 |
| NobodyCam | Ng: can you grab the conductor log? | 15:14 |
| NobodyCam | also could try bumping up https://github.com/openstack/ironic/blob/master/ironic/nova/virt/ironic/driver.py#L70-L76 | 15:14 |
| Ng | NobodyCam: I think another deploy has already started, so the log probably doesn't exist anymore :/ | 15:15 |
| Ng | but I'll ask that we grab it if this happens again | 15:15 |
| NobodyCam | ya... check the retry values I link above too.. how may nodes are we working with here? | 15:16 |
| NobodyCam | s/link/linked/ | 15:16 |
| NobodyCam | # of node can affect the length of time the power check task runs and if real hw ipmi can also be very very slow | 15:17 |
| *** ifarkas has quit IRC | 15:17 | |
| Ng | NobodyCam: so this is seed->undercloud, so probably just 1 atm, but I'll confirm that | 15:21 |
| *** yongli has joined #openstack-ironic | 15:21 | |
| jbjohnso | NobodyCam, # of nodes shouldn't affect things too badly, 4,000 nodes can be done under 10 seconds if you do it right.. | 15:22 |
| jbjohnso | of course, forking and execing would be not doing it right I suppose | 15:22 |
| NobodyCam | jbjohnso: :) | 15:23 |
| *** datajerk has joined #openstack-ironic | 15:23 | |
| jbjohnso | btw, in theory, confluent should be able to do that now | 15:25 |
| jbjohnso | # confetty show /nodes/n1/power/state | 15:25 |
| jbjohnso | state="on" | 15:25 |
| jbjohnso | # confetty show /nodes/n1/boot/nextdevice | 15:25 |
| jbjohnso | nextdevice="default" | 15:25 |
| jbjohnso | one node took real 0m0.056s including time to start the python interpretore | 15:26 |
| jbjohnso | 44 milliseconds of that is starting the interpretor... | 15:26 |
| NobodyCam | jbjohnso: my guess is Ng is using ipmitool? | 15:26 |
| Ng | yep, this will be ipmi | 15:26 |
| jbjohnso | I'm just trying to nag about people trying my ipmi stuff ;) | 15:27 |
| NobodyCam | hehehe | 15:28 |
| jbjohnso | pip install confluent-server | 15:29 |
| jbjohnso | then run confluent | 15:29 |
| jbjohnso | and bob's your uncle | 15:29 |
| NobodyCam | ? bob my dad | 15:29 |
| *** zackf has joined #openstack-ironic | 15:29 | |
| jbjohnso | it is a peculiar expression | 15:30 |
| devananda | morning, all | 15:33 |
| NobodyCam | good morning devananda | 15:34 |
| *** zdin0bot has joined #openstack-ironic | 15:35 | |
| lucasagomes | devananda, morning | 15:36 |
| devananda | Ng: so as NobodyCam already pointed out, changing the setting for api_retry (in nova) may help. also, you may want to try changing sync_power_state_interval (in ironic) here https://github.com/openstack/ironic/blob/master/ironic/conductor/manager.py#L84 | 15:37 |
| devananda | the current setting may be too optimisitc | 15:37 |
| Ng | ok, thanks | 15:39 |
| *** Mikhail_D_ltp has joined #openstack-ironic | 15:43 | |
| jbjohnso | devananda, fyi, as I pointed out, you could try out confluent too now | 15:45 |
| *** datajerk has quit IRC | 15:47 | |
| NobodyCam | gah poor internet just sucks | 15:50 |
| *** eghobo has joined #openstack-ironic | 15:51 | |
| *** zdin0bot has quit IRC | 15:53 | |
| *** jistr has quit IRC | 16:05 | |
| NobodyCam | brb | 16:11 |
| NobodyCam | Ng: while you in channel have you seen this b4? http://paste.openstack.org/show/tT4NAIK7GA5B0sl2Hzww/ | 16:22 |
| Ng | NobodyCam: hmm, not that I recall | 16:23 |
| NobodyCam | :) | 16:23 |
| Shrews | NobodyCam: that happens when gerrit has gone bye-bye. they were doing some maintenance today (for project renames), iirc | 16:24 |
| NobodyCam | ahh so just waiting may fix my woes | 16:25 |
| Shrews | http://lists.openstack.org/pipermail/openstack-dev/2014-May/035525.html | 16:25 |
| NobodyCam | Ty Shrews :) | 16:27 |
| * NobodyCam makes a begal :) | 16:28 | |
| Shrews | though, that start time is 4.5 hours from now. *shrug* | 16:30 |
| NobodyCam | *bagel even... | 16:32 |
| NobodyCam | humm | 16:32 |
| dtantsur | NobodyCam, any updates on https://review.openstack.org/#/c/92625 ? :) | 16:35 |
| NobodyCam | dtantsur: thats what I'm trying to build atm... getting a odd error (see paste above) | 16:36 |
| NobodyCam | error is unrelated to the patch | 16:36 |
| *** dkehn_ has joined #openstack-ironic | 16:39 | |
| dtantsur | Oo | 16:39 |
| dtantsur | what command do you use to fetch? | 16:39 |
| NobodyCam | i'm setting DIB_REPOLOCATION_ironic & DIB_REPOREF_ironic and running devtest | 16:39 |
| dtantsur | mmmmm.... | 16:39 |
| dtantsur | git fetch https://review.openstack.org/openstack/ironic refs/changes/25/92625/7 seems to work... | 16:39 |
| openstackgerrit | Dan Prince proposed a change to openstack/ironic: Add parallel_image_downloads option. https://review.openstack.org/95213 | 16:39 |
| *** dkehnx has quit IRC | 16:40 | |
| *** derekh_ has quit IRC | 16:42 | |
| lucasagomes | alright I will call it a day :) | 16:44 |
| lucasagomes | have a good night and great weekend everybody | 16:44 |
| dtantsur | me too :) | 16:44 |
| dtantsur | g'night, lucasagomes | 16:44 |
| *** lucasagomes is now known as lucas-afk | 16:44 | |
| lucas-afk | dtantsur, u too g'night! | 16:44 |
| *** Mikhail_D_ltp has quit IRC | 16:45 | |
| *** dkehn_ is now known as dkehnx | 16:45 | |
| NobodyCam | night lucas-afk | 16:45 |
| NobodyCam | and dtantsur :) | 16:45 |
| devananda | g'night folks! | 16:47 |
| NobodyCam | nnight devananda | 16:48 |
| NobodyCam | lol | 16:48 |
| *** zdin0bot has joined #openstack-ironic | 16:52 | |
| *** sysexit has quit IRC | 16:53 | |
| *** zdin0bot has quit IRC | 16:53 | |
| *** zdin0bot has joined #openstack-ironic | 16:53 | |
| *** harlowja_away is now known as harlowja_ | 16:55 | |
| openstackgerrit | Josh Gachnang proposed a change to openstack/ironic: Factoring out PXE and TFTP functions https://review.openstack.org/90233 | 16:59 |
| openstackgerrit | Josh Gachnang proposed a change to openstack/ironic: Adding a reference driver for the agent https://review.openstack.org/84795 | 16:59 |
| *** sysexit has joined #openstack-ironic | 17:03 | |
| *** jbjohnso has quit IRC | 17:16 | |
| *** ndipanov has quit IRC | 17:17 | |
| devananda | make the virtual-ironic job voting -- https://review.openstack.org/95219 | 17:18 |
| devananda | and make it vote on DIB too -- https://review.openstack.org/95220 | 17:18 |
| devananda | adam_g: ^ | 17:18 |
| adam_g | devananda, cool! i added a elasticsearch query for that nodelocked exception | 17:22 |
| devananda | adam_g: awesome, thanks | 17:22 |
| *** zdin0bot1 has joined #openstack-ironic | 17:27 | |
| *** zdin0bot has quit IRC | 17:27 | |
| openstackgerrit | Dan Prince proposed a change to openstack/ironic: Add parallel_image_downloads option https://review.openstack.org/95213 | 17:30 |
| *** datajerk has joined #openstack-ironic | 17:34 | |
| Shrews | adam_g: got a minute? | 17:36 |
| adam_g | Shrews, just a few. i need to run soon | 17:38 |
| Shrews | adam_g: i'll hit you up later then. it's about issues i'm having with jenkins and tempest changes | 17:39 |
| adam_g | Shrews, i can chat now. im actually on my way out for the weekend, but will be back in monday | 17:39 |
| Shrews | adam_g: i'm having a devil of a time with https://review.openstack.org/94436. jenkins hates me for different reasons at different times | 17:40 |
| Shrews | adam_g: just wondering if this is common with tempest, or if they're having significant issues, or what. i'm fairly lost atm | 17:40 |
| *** athomas has quit IRC | 17:40 | |
| adam_g | Shrews, ive found you end up rechecking more in tempests' gate because they test so much | 17:41 |
| adam_g | and more of it is voting | 17:41 |
| *** zdin0bot1 has quit IRC | 17:41 | |
| Shrews | adam_g: ok. i'll try recheck a couple more time then | 17:41 |
| Shrews | times* | 17:42 |
| Shrews | so frustrating | 17:42 |
| NobodyCam | brb... looking for some food | 17:43 |
| adam_g | Shrews, http://logs.openstack.org/36/94436/1/check/gate-tempest-dsvm-large-ops/41d7139/console.html#_2014-05-22_21_22_43_660 looks like a pip dependency issue that may be transient? | 17:43 |
| *** coolsvap is now known as coolsvap|afk | 17:46 | |
| *** sysexit has quit IRC | 17:55 | |
| *** OldCrowEW has quit IRC | 18:14 | |
| *** OldCrowEW has joined #openstack-ironic | 18:16 | |
| *** comstud is now known as bearhands | 18:23 | |
| *** OldCrowEW has quit IRC | 18:30 | |
| *** OldCrowEW has joined #openstack-ironic | 18:37 | |
| NobodyCam | brb | 18:53 |
| *** shausy has quit IRC | 18:56 | |
| *** pelix has quit IRC | 18:59 | |
| *** jdob has quit IRC | 19:08 | |
| *** user____ has joined #openstack-ironic | 19:09 | |
| *** user____ has quit IRC | 19:10 | |
| *** user____ has joined #openstack-ironic | 19:11 | |
| *** user____ is now known as igordcard_afk | 19:11 | |
| openstackgerrit | Devananda van der Veen proposed a change to openstack/baremetal-specs: Make the REST API fully asynchronous https://review.openstack.org/94923 | 19:12 |
| *** igordcard_afk has joined #openstack-ironic | 19:12 | |
| *** igordcard has quit IRC | 19:13 | |
| *** sysexit has joined #openstack-ironic | 19:17 | |
| *** lsmola has quit IRC | 19:20 | |
| openstackgerrit | Chris Krelle proposed a change to openstack/ironic: Workaround for conflicting volume labels https://review.openstack.org/93133 | 19:21 |
| NobodyCam | lucas-afk: rllo: ^^^^ now with improved doc string.. | 19:22 |
| NobodyCam | humm s/rllo/rloo/ lol | 19:26 |
| devananda | NobodyCam: log suggestion. otherwise lgtm | 19:45 |
| devananda | NobodyCam: also, did you open a bug for the #TODO in there? | 19:45 |
| NobodyCam | :) will take a look in a minute.. the move to sgdisk one... not yet.. wanted to make sure thats the direction we want to go | 19:46 |
| NobodyCam | but I will.. | 19:46 |
| NobodyCam | trying to bang out some reviews | 19:46 |
| NobodyCam | :-p | 19:46 |
| *** Kaizer has joined #openstack-ironic | 19:49 | |
| NobodyCam | devananda: +2's a could of reviews if you have a free eye they may be able to land... | 20:00 |
| NobodyCam | s/2's/2'd/ | 20:00 |
| NobodyCam | reworking the logging on 93133 now shoud have in a couple of minutes | 20:01 |
| devananda | ack | 20:03 |
| NobodyCam | devananda: quick question that I knida dred the answer to... in your example log lines did you want node's uuid, I ask because It may take some work to get that?? | 20:07 |
| devananda | NobodyCam: what use is the error to an operator without knowing which node's deploy broke? | 20:07 |
| NobodyCam | :-p | 20:08 |
| openstackgerrit | A change was merged to openstack/ironic: Imported Translations from Transifex https://review.openstack.org/95078 | 20:11 |
| * NobodyCam cries | 20:13 | |
| devananda | ? | 20:16 |
| NobodyCam | node info is not at all close to these finctions. :( | 20:19 |
| NobodyCam | will need to modify the deploy interface, which makes this change MUCH LARGER then I had hopped for | 20:21 |
| NobodyCam | :-p | 20:21 |
| devananda | NobodyCam: how so? it changes the deploy_utils nodule. not the driver API | 20:22 |
| *** OldCrowEW has quit IRC | 20:26 | |
| *** OldCrowEW has joined #openstack-ironic | 20:26 | |
| -openstackstatus- NOTICE: Gerrit will be offline for about 20 minutes in order to rename some projects starting at 21:00 UTC. | 20:33 | |
| *** annegentle_ has quit IRC | 20:34 | |
| NobodyCam | devananda: may not be as bad I make it out to be.. let me try something | 20:35 |
| devananda | bearhands: have you checked if your patch 91419 overlaps with lucas' patch series (head: https://review.openstack.org/#/c/91062) | 20:40 |
| devananda | yea ,they're overlapping in several places | 20:42 |
| devananda | and both reference the same bug # | 20:42 |
| devananda | bearhands: hm, looks like you WIP'd your patch may 1st and haven't updated it, but lucas' have been pretty active. | 20:45 |
| *** tatyana has joined #openstack-ironic | 20:52 | |
| *** sysexit has quit IRC | 20:55 | |
| JayF | devananda: he's out until Wednesday I believe | 20:57 |
| devananda | JayF: ah, thanks | 20:59 |
| *** sysexit has joined #openstack-ironic | 21:07 | |
| *** dshulyak has quit IRC | 21:14 | |
| *** dshulyak has joined #openstack-ironic | 21:16 | |
| *** annegentle has joined #openstack-ironic | 21:18 | |
| *** zackf has quit IRC | 21:19 | |
| openstackgerrit | Chris Krelle proposed a change to openstack/ironic: Workaround for conflicting volume labels https://review.openstack.org/93133 | 21:23 |
| NobodyCam | devananda: ^^^ but wait for test to pass... they pass for me locally but I never trust my local tests :-p | 21:24 |
| *** annegentle has quit IRC | 21:28 | |
| NobodyCam | ahh devananda you see this one: http://lists.openstack.org/pipermail/openstack-dev/2014-May/035737.html | 21:31 |
| -openstackstatus- NOTICE: Gerrit is offline in order to rename some projects. ETA: 22:00. | 21:34 | |
| *** ChanServ changes topic to "Gerrit is offline in order to rename some projects. ETA: 22:00." | 21:34 | |
| *** tatyana has quit IRC | 21:35 | |
| NobodyCam | lol of course | 21:35 |
| NobodyCam | devananda: 93133 has pep failure i have a fix will push up when gerrit comes back to life | 21:36 |
| *** Penick has joined #openstack-ironic | 21:45 | |
| *** harlowja_ is now known as harlowja_away | 21:59 | |
| *** lucas-afk has quit IRC | 22:12 | |
| *** lucas-afk has joined #openstack-ironic | 22:13 | |
| *** openstackgerrit has quit IRC | 22:14 | |
| bearhands | devananda: Yeah, I'm out until Wednesday or so... and plan to get back onto things then | 22:14 |
| bearhands | devananda: If lucas is ahead, I don't mind abandoning mine | 22:14 |
| bearhands | or | 22:15 |
| bearhands | one should feel free to take mine and rebase it and fix it up | 22:15 |
| *** openstackgerrit has joined #openstack-ironic | 22:15 | |
| bearhands | i won't be offended | 22:15 |
| bearhands | anyway, I gotta drop.. was just checking in for a second | 22:15 |
| *** OldCrowEW has quit IRC | 22:16 | |
| *** openstackstatus has quit IRC | 22:18 | |
| *** openstack has joined #openstack-ironic | 22:19 | |
| *** openstackstatus has joined #openstack-ironic | 22:19 | |
| *** ChanServ sets mode: +v openstackstatus | 22:19 | |
| *** john3213 has joined #openstack-ironic | 22:21 | |
| openstackgerrit | Chris Krelle proposed a change to openstack/ironic: Workaround for conflicting volume labels https://review.openstack.org/93133 | 22:22 |
| *** john3213 has left #openstack-ironic | 22:26 | |
| openstackgerrit | Monty Taylor proposed a change to openstack/ironic-specs: Updated gitreview file for repo rename https://review.openstack.org/95290 | 22:28 |
| *** ChanServ changes topic to "OpenStack Bare Metal Provisioning | Docs: http://docs.openstack.org/developer/ironic/ | Bugs: https://bugs.launchpad.net/ironic | Status: https://etherpad.openstack.org/p/IronicWhiteBoard" | 22:28 | |
| NobodyCam | mordred: looking at https://review.openstack.org/#/c/95290 you are dropping .git from the project too? | 22:32 |
| mordred | NobodyCam: oh - it really doesn't actually matter | 22:33 |
| mordred | NobodyCam: I made the commit with a script - I could totally fix it real quick | 22:33 |
| NobodyCam | :) please :) hehehehe lol | 22:34 |
| openstackgerrit | Monty Taylor proposed a change to openstack/ironic-specs: Updated gitreview file for repo rename https://review.openstack.org/95295 | 22:34 |
| mordred | oh. well, that's not what I wanted to do | 22:34 |
| mordred | oh. I know why that happened. bother | 22:35 |
| NobodyCam | lol | 22:36 |
| * NobodyCam feel bad now ... | 22:36 | |
| mordred | nah. fixed it | 22:36 |
| NobodyCam | s/feel/feels/ | 22:36 |
| NobodyCam | lol I can only +1 | 22:37 |
| *** eguz has joined #openstack-ironic | 23:02 | |
| *** eghobo has quit IRC | 23:06 | |
| *** ewindisch has quit IRC | 23:24 | |
| *** r-daneel has quit IRC | 23:34 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!