Friday, 2014-12-12

ChuckCjroll: r u around?00:04
jrollChuckC: barely, what's up?00:04
ChuckClast week we talked about moving a VM between networks00:04
ChuckCjroll: ^^ I'm wondering if the VM has an IP address?00:05
ChuckCjroll: it was 12/400:06
jrollChuckC: you mean an instance between provisioning and tenant networks?00:06
ChuckCjroll: yep, i believe so00:07
*** romcheg has quit IRC00:07
ChuckCjroll: never mind, I think I'm barely here also :P00:10
jrollChuckC: yes, it has an IP on each network00:10
JoshNangNobodyCam: :( that's what i was trying to avoid00:11
JoshNangsince decorators are just wrappers, we could decorate it with zap_step and clean_step, plus zap_step shouldn't need a priority (you only ever run one at a time)00:12
NobodyCamJoshNang: I suppose you could have just one as lone as it allowed for different priorites for the different states00:12
ChuckCjroll: ok, good, thanks00:12
NobodyCamJoshNang: I can see a case here in clean step x has a priority of 3 and in zap it needs to be 500:13
NobodyCams/here/where/00:14
JayFclif_h: did you push that patchset with the fix? I just didn't see openstackgerrit announce it here00:14
NobodyCamor do thid in zap but not in clean type control00:14
clif_hJayF: you mean in response to the pep8 comments?00:15
JoshNangNobodyCam: what would priority be useful for in zap?00:15
JayFclif_h: I meant generally based on what you said in internal IRC :)00:15
NobodyCamordering the steps .. ie break up the raid before clean each disk00:15
clif_hJayF: not yet, I'm not sure if what I did to fix the unit tests is proper or not00:16
clif_hthe failing tests didn't reveal anything wrong with what I wrote afiak00:16
jrollJayF: it's a nova project00:16
jroller s/project/patch00:16
JayFooooooh00:16
NobodyCambut in clean I want clean the disk as step 100:16
JayFof course it is00:16
JayFwhee00:16
JoshNangNobodyCam: hrmmm. i'd lean towards making separate api calls in zap00:16
JoshNangotherwise you'd need to pass in some type of task flow to say "break raid, clean disks"00:17
JoshNangbecause you'll likely have other zap steps you want to run, like upgrade firmware00:17
JoshNangbut you probably don't want to run firmware upgrades while breaking raids.00:17
*** bradjones has quit IRC00:18
NobodyCamJoshNang: how are you going to control the order of those steps?00:18
JoshNanghave break raid be an api call, have clean disks be another api call00:18
JoshNangi see zap as basically an interface for external system to do things to hardware, so forcing the systems to pretty explicit about what they want done shouldn't be too much of a burden00:19
* NobodyCam is missing a pice. (most likly it is a pice of his mind)00:19
NobodyCambut zap is going to run the clean steps too?00:19
NobodyCamor will those have to be called for via api?00:20
JoshNangit could run a single clean step via the api, i'd say00:20
NobodyCamJoshNang: let me re read the zap spec. I may have had a basic misunderstanding00:21
JoshNanglike, a nodes in managed state, you post to the api saying "run zap step X", it runs it, goes back to managed when its done, you call more steps if you want more zapping, or you move the node to available00:21
NobodyCamso nothing in zap will ever happen on its own00:21
JoshNangthat'd be my intention, yes00:22
JayFexactly00:22
JayFthat's why it was important to differentiate CLEAN00:22
NobodyCam:) that was where I missed the boat00:22
JoshNangahhh00:22
JoshNangyeah, i'll rewrite that to be much more explicit.00:22
NobodyCamplease ignore me... :-p00:22
JoshNangha no worries. if you got confused by it, i'm sure many others have/would00:23
*** david-lyle is now known as david-lyle_afk00:23
*** Masahiro has joined #openstack-ironic00:23
* NobodyCam point to the afor mentioned too many reviews comment00:23
NobodyCams/point/points/00:23
JoshNangheh00:23
SpamapSNobodyCam: re "we like UUIDs" .. I do too. However, the benefit of using names is that it allows unrolling the dependency graph.. you can create nodes and then create and replace the images later without bothering the nodes.00:26
*** yuanying has quit IRC00:26
NobodyCamSpamapS: right now we have no way of dooing the lookup00:27
*** smoriya has joined #openstack-ironic00:27
*** Masahiro has quit IRC00:28
*** yuanying has joined #openstack-ironic00:29
SpamapSNobodyCam: sure you do. -> glanceclient. :)00:29
NobodyCam:) might be able to slide a function on to the end of https://github.com/openstack/ironic/blob/master/ironic/common/images.py00:31
*** yuanying has quit IRC00:33
smoriyaHello Ironic,00:33
NobodyCamthen check if the image id looks like a uuid .. if not get the uuid from glanceclient and run like normal00:33
smoriyaI have a question about Ironic milestone for kilo00:33
NobodyCamHi smoriya00:33
*** yuanying has joined #openstack-ironic00:34
SpamapSNobodyCam: I don't like the "if it looks like uuid", but just have a name field instead of id that can cause population of the id at deploy time.00:34
smoriyaDo you have any spec submission deadline or patch submission deadline for kilo?00:34
*** humble__ has quit IRC00:34
*** humble__ has joined #openstack-ironic00:35
NobodyCamsmoriya: https://wiki.openstack.org/wiki/Kilo_Release_Schedule00:35
smoriyaNobodyCam: Thank you!00:35
clif_hhttps://review.openstack.org/#/c/141188/ <- tests pass now00:36
smoriyaNobodyCam: Does "Feature Proposal Freeze" mean spec submission deadline?00:38
NobodyCamyes00:38
smoriyaOK, then we can introduce new feature in kilo-3?00:39
NobodyCamtho if it is a large change please allow for review time.00:39
JayFI'd say it's never a good idea to plan on getting something in on the last cycle00:40
JayFbecause features half-done from k-2 are likely to take precedence over shiny new ones00:40
smoriyaJayF: Yes, I understand00:40
JayFthe best advice is: if you want the feature in, put up a spec now, even if it's just a backlog spec and you arne't starting work right away00:40
clif_hmrda: could you take a look?00:40
JayFthat just requires describing the problem and a proposed fix00:40
*** marcoemorais1 has quit IRC00:41
smoriyaJayF: Thank you for your advice. I'll try :)00:42
JayFfor an example, see: https://review.openstack.org/#/c/131272/00:42
JayFthat's something we all wanted in Ironic, but wouldn't have time for in K, so we filed a backlog spec about it00:42
JayFeven if there's time in K, getting the backlog spec in means people generally agree your feature should exist00:42
mrdaclif_h: at your review?00:43
JayFmrda: yeah, https://review.openstack.org/#/c/141188/00:44
JayFmrda: he just left for the day though :) he's CST00:44
mrdathat's ok, I'll look at it later today00:44
smoriyaJayF: Thanks! That example is really helpful to me.00:45
*** cuihaozhi has quit IRC00:45
*** cuihaozhi has joined #openstack-ironic00:46
JayFNo problem at all! You can see all completed specs at: http://specs.openstack.org/openstack/ironic-specs/ -- the two items in "Idea back-log" are good examples :)00:46
clif_hmrda: yes please00:46
clif_hwhen you get a chance00:46
openstackgerritJay Faulkner proposed openstack/ironic-specs: Remove spec added for an example to render  https://review.openstack.org/14122800:47
JayF^ easy review, just noticed when linking that the example to test rendering was left in even after we landed a box00:47
*** andreykurilin__ has quit IRC00:47
JayFs/box/backlog spec/00:47
smoriyaJayF: great! now I'm starting to learn developing process in Ironic00:47
JayFHave you seen https://wiki.openstack.org/wiki/How_To_Contribute -- that's always a good first step overall00:48
smoriyaJayF: yeah, I understand basic rules00:49
JayFCool, then you're well on your way. Good luck.00:50
*** yuanying has quit IRC00:50
*** Masahiro has joined #openstack-ironic00:54
NobodyCamsmoriya: there are also several unassigned bugs I believe that too is a good place to get to know the code00:55
NobodyCamand ofc my /shameless_plug for doc patches :)00:55
smoriyaNobodyCam: yup, for my first step I'm fixing document bug :)00:56
NobodyCamw00 h00 :) /happy_dance00:56
JayFsmoriya: there's a tag in launchpad for 'low-hanging-fruit' although at this point many of those are /very/ low hanging :)00:57
* NobodyCam thinks of evil weekend project... a PixieBoot irc bot.00:58
*** hemna has quit IRC00:59
NobodyCamgah PixieBoots*00:59
JayFnot a bad idea00:59
NobodyCamhttp://willie.dftba.net01:00
smoriyaJayF: yes, I've been assigned one of 'low-hanging-fruit' to me and my first fix was merged to master today :)01:03
*** yuanying has joined #openstack-ironic01:04
JayFawesome01:05
cuihaozhiHi,all. when i deploy bm-node with pxe_ipmi driver. the screen display "!PXE entry point found(we hope) at 98DA:0106 via plan A...TFTP prefix Unable to locate configuration file", any body resolved the same problem?01:13
*** afaranha_ has joined #openstack-ironic01:18
*** afaranha_ has quit IRC01:20
*** PixieBoots has joined #openstack-ironic01:20
*** yjiang5 is now known as yjiang5_away01:20
NobodyCamPixieBoots: hellp01:21
NobodyCamPixieBoots: hello01:21
*** r-mibu has joined #openstack-ironic01:22
NobodyCam.helloworld01:23
*** ChuckC has quit IRC01:23
NobodyCamoh well.. I'll play some more this weekend01:24
*** chenglch has joined #openstack-ironic01:25
NobodyCamI cloned that bot into my Nobodycam repo. in case any one wants to play with it01:25
*** PixieBoots has quit IRC01:25
*** Marga_ has quit IRC01:26
*** ChuckC has joined #openstack-ironic01:32
*** PixieBoots has joined #openstack-ironic01:32
NobodyCam.roll 1d1001:35
PixieBootsNobodyCam: You roll 1d10: (2) = 201:35
*** Marga_ has joined #openstack-ironic01:35
*** ChuckC has quit IRC01:36
JayF.roll 65535d6553501:37
PixieBootsJayF: I only have 1000 dice. =(01:37
*** ChuckC has joined #openstack-ironic01:37
JayFoverflow you crazy bot! overflow!01:37
NobodyCamhttps://www.youtube.com/watch?v=mYvAYwpUDv801:37
PixieBootsException: CA Certificate bundle /etc/ssl/certs/ca-certificates.crt is not readable (file "/Users/NobodyCam/BareMetal/nobodycam/PixieBootsIrcBot/willie/web.py", line 165, in connect)01:37
JayFhahaha01:37
NobodyCamlol01:37
NobodyCam.xkcd01:37
PixieBootshttp://xkcd.com/931 [Lanes]01:37
*** shakamunyi has joined #openstack-ironic01:38
NobodyCamI never said I actually set it up .. lol like 10 of 35 modules failed to load for one reason or another01:38
*** r-mibu has left #openstack-ironic01:40
NobodyCam.github01:40
NobodyCamoh cool01:41
*** humble__ has quit IRC01:41
NobodyCam.makebug01:41
PixieBootsPlease title the issue01:41
NobodyCamgota set that up01:41
*** humble__ has joined #openstack-ironic01:41
* JayF has learned in experience allowing peopel to file bugs from IRC leads to shittier bugs01:41
NobodyCam.findbug01:43
PixieBootsNobodyCam: What are you searching for?01:43
NobodyCam.findbug ironic bugs01:43
PixieBootsGit module not configured, make sure github.oauth_token and github.repo are defined01:43
NobodyCamlol01:43
NobodyCami'll stop the spam01:43
*** PixieBoots has quit IRC01:43
*** Marga_ has quit IRC01:48
*** sambetts has quit IRC02:04
*** sambetts has joined #openstack-ironic02:05
cuihaozhiHi,all. tftp log said: "RRQ from 172.16.40.112 filename pxelinux.cfg/01-94-de-80-8c-d0-fa, sending NAK (1, File not found) to 172.16.40.112",but /tftpboot/pxelinux.cfg/01-94-de-80-8c-d0-fa exists and file onwer is ironic:ironic, whick point i can check next?02:08
lintanrloo: Hi02:09
NobodyCamcuihaozhi: are all the read bits set?02:09
rloohi lintan02:09
lintanrloo: I am thinking to put the boot_device/boot_persistent into node.instance_info, so admin will not aware this from driver_info anymore.02:10
cuihaozhiNobodyCam: seems like tftp read pxelinux.cfg/AC102870. pxelinux.cfg/AC10287...pxelinux.cfg/A all02:12
rloolintan: i'm not that familiar with what goes in instance_info. Is the boot_device/boot_persistence an instance thing?02:12
cuihaozhiNobodyCam: and find pxelinux.cfg/01-94-de-80-8c-d0-fa is softlink from /tftpboot/270664fa-fd80-4242-b284-1af2fc482f33/config, is that problem?02:12
*** dlaube has quit IRC02:13
NobodyCamcuihaozhi: shouldn't be a issue02:13
rloolintan: I'm glad it isn't in driver_info. so basically, you need to keep that information around, and I believe the other ManagementInterface's that exist, do not store that info with the Node object.02:13
NobodyCamany selinux or apparmor block messages in the logs02:14
lintanrloo: we have ramdisk/kernel/root_gb in instance_info, it make sense to see the boot_device is for the instance02:15
rloolintan: I would suggest that you propose that in the spec, but the spec has already been approved.02:16
cuihaozhiNobodyCam: selinux disabled and no log with them yet.02:16
lintanrloo: We can not expect everything is done with spec.... We find changes during coding... but what do you mean the ManagementInterface?02:18
rloolintan: (assuming I remember what we're discussing), don't you have a ManagementInterface with set/get_boot_device?02:18
rloolintan: I believe other drivers with ManagementInterfaces handle the set/get_boot_device by talking directly to the bm node.02:19
lintanrloo: Yes, you are right about this. But it's complex for AMT/vPro02:20
rloolintan: wrt the spec. agreed, not everything can be done with spec, but that's where we hope to flush things out like what new properties get added to driver_info, what new API, etc. That makes the coding easier :-)02:20
rloolintan: so if AMT/vPro is more complex than the other drivers, then it would help to know why it is more complicated ;) I think you had some stuff commented in the code, but to be honest, I didn't spend enough time trying to figure out what that complexity was.02:23
lintanrloo: OK, first thing is AMT/vPro doesn't support persistent, that's why we need to save boot_device in nodes02:24
rloolintan: sorry, I don't mean to be rude, but it is the end of my day and I need to log off soon. I'm off until next week and realistically, I don't know that I will be able to get back to you on this on Monday. I want to spend some time with the new state machine patches because I think that is a higher priority.02:24
cuihaozhiNobodyCam: try tftp $host_ip->get pxelinux.cfg/01-94-de-80-8c-d0-fa/config faild. but tftp $host_ip->get 270664fa-fd80-4242-b284-1af2fc482f33/config successed02:25
rloolintan: it would probably help whoever reviews your patch, if you add your comments/description with that patch (or maybe submit a patch to your spec with more info if you want). I'm not sure what the most effective way might be, but other folks need to review your stuff too and like I said I'm not always avail.02:25
cuihaozhiNobodyCam: seems like some thing wrong with link?02:25
lintanrloo: OK No problem.02:25
lintanrloo: thanks for your advise.02:26
rloosorry, lintan. I wanted to help get your patch in to juno-1 but it seemed like it needs a bit more work.02:26
rloolintan: I mean kilo not juno. losing track of everything...02:26
rloolintan: and sorry, i haven't had a chance to look at your bm trust spec again.02:27
rloonight everyone!02:28
*** rloo has quit IRC02:28
*** chenglch has quit IRC02:31
*** chenglch|2 has joined #openstack-ironic02:50
*** ramineni has joined #openstack-ironic02:54
*** shakamunyi has quit IRC02:57
*** tylerdurden has joined #openstack-ironic02:57
*** jerryz has joined #openstack-ironic03:16
*** naohirot has quit IRC03:29
*** harlowja is now known as harlowja_away03:36
*** anderbubble has joined #openstack-ironic03:51
*** chenglch|2 has quit IRC03:54
*** naohirot has joined #openstack-ironic03:59
*** Nisha has joined #openstack-ironic04:09
*** Nisha_away has joined #openstack-ironic04:13
*** yuanying_ has joined #openstack-ironic04:15
*** Nisha has quit IRC04:16
*** anderbubble has quit IRC04:17
*** yuanying has quit IRC04:17
*** r-daneel has quit IRC04:19
*** yuanying has joined #openstack-ironic04:28
*** yuanying_ has quit IRC04:32
NobodyCamcuihaozhi: map file04:41
*** pradipta_away is now known as pradipta04:44
NobodyCamcuihaozhi: see step #6 of http://docs.openstack.org/developer/ironic/deploy/install-guide.html#pxe-setup04:44
*** chenglch has joined #openstack-ironic04:46
openstackgerritTan Lin proposed openstack/ironic-specs: update amt-pxe-driver spec  https://review.openstack.org/14126904:49
cuihaozhiNobodyCam: thank u, i find it deploy_kernel in pxe-config files can't use absolute paths.04:54
*** Marga_ has joined #openstack-ironic04:59
*** Marga_ has quit IRC05:12
*** Marga_ has joined #openstack-ironic05:13
*** marcoemorais has joined #openstack-ironic05:15
*** marcoemorais1 has joined #openstack-ironic05:16
*** marcoemorais has quit IRC05:19
*** Nisha_away has quit IRC05:27
*** yuanying has quit IRC05:31
*** killer_prince is now known as lazy_prince05:36
*** pensu has joined #openstack-ironic05:39
*** yuanying has joined #openstack-ironic05:40
*** masco has joined #openstack-ironic05:41
*** Marga_ has quit IRC06:13
*** anderbubble has joined #openstack-ironic06:20
*** Marga_ has joined #openstack-ironic06:22
*** Marga_ has quit IRC06:27
openstackgerritNisha Agarwal proposed openstack/ironic-specs: Discover node properties using new CLI node-inspect  https://review.openstack.org/10095106:27
*** Marga_ has joined #openstack-ironic06:30
*** achanda has joined #openstack-ironic06:31
*** pcrews has quit IRC06:35
*** Marga_ has quit IRC06:43
*** lintan_ has joined #openstack-ironic06:48
*** lintan_ has quit IRC06:49
humble__hi, why we need user kernel and user ramdisk in ironic06:55
*** Marga_ has joined #openstack-ironic06:55
kesbecause we need pxe, so we need those two files.06:57
*** chenglch has quit IRC07:03
*** yuanying has quit IRC07:03
*** yuanying has joined #openstack-ironic07:03
*** achanda has quit IRC07:04
*** achanda has joined #openstack-ironic07:05
*** achanda_ has joined #openstack-ironic07:07
*** Haomeng|2 has joined #openstack-ironic07:08
*** Haomeng has quit IRC07:09
*** achanda has quit IRC07:09
*** mrda is now known as mrda-weekend07:11
*** anderbubble has quit IRC07:12
*** achanda_ has quit IRC07:17
*** Marga_ has quit IRC07:27
openstackgerritTan Lin proposed openstack/ironic: Add AMT-PXE-Driver to deploy cloud on PC  https://review.openstack.org/13518407:31
*** lifeless has quit IRC07:36
*** lifeless has joined #openstack-ironic07:43
*** kes has quit IRC07:43
*** achanda has joined #openstack-ironic07:46
Haomeng|2humble__: hi07:47
Haomeng|2humble__: because our ironic will not install bootloader and kernel into local harddisk07:47
Haomeng|2humble__: will support these in IPA I£ô£è£é£î£ë07:47
Haomeng|2Haomeng|2:  if we can install bootloader and kernel into local harddisk, then set bootdevice to localdisk, then it will  not depend on pxe after the provision07:50
*** yuanying_ has joined #openstack-ironic08:05
*** dtantsur|afk is now known as dtantsur08:05
dtantsurMorning Ironic08:05
*** yjiang5 has joined #openstack-ironic08:08
*** jcoufal has joined #openstack-ironic08:09
*** ndipanov_ has joined #openstack-ironic08:09
dtantsurJayF, re packaging discoverd: I package the service as usual python service (setup.py sdist, pip), I do not package the ramdisk. What exactly was your question about?08:09
*** achanda has quit IRC08:11
*** achanda has joined #openstack-ironic08:12
*** yuanying has quit IRC08:14
*** naohirot has quit IRC08:14
*** ndipanov has quit IRC08:14
*** yjiang5_away has quit IRC08:14
*** naohirot_ has joined #openstack-ironic08:14
*** erwan_taf_ has joined #openstack-ironic08:16
*** erwan_taf has joined #openstack-ironic08:16
*** achanda has quit IRC08:16
*** erwan_taf has quit IRC08:17
*** erwan_taf_ is now known as erwan_taf08:17
*** achanda has joined #openstack-ironic08:18
*** dlpartain has joined #openstack-ironic08:21
openstackgerritMerged openstack/ironic-specs: Remove spec added for an example to render  https://review.openstack.org/14122808:26
*** marcoemorais1 has quit IRC08:28
*** jcoufal has quit IRC08:30
*** teju has joined #openstack-ironic08:42
*** chenglch has joined #openstack-ironic08:47
*** dtantsur is now known as dtantsur|bbl08:59
*** lazy_prince has quit IRC08:59
*** dlpartain has left #openstack-ironic09:00
*** jcoufal has joined #openstack-ironic09:00
*** ifarkas has joined #openstack-ironic09:09
*** yuanying_ has quit IRC09:15
*** yuanying has joined #openstack-ironic09:16
*** jistr has joined #openstack-ironic09:19
*** derekh has joined #openstack-ironic09:25
*** romcheg has joined #openstack-ironic09:28
*** Masahiro has quit IRC09:53
*** killer_prince has joined #openstack-ironic10:00
*** killer_prince is now known as lazy_prince10:00
openstackgerritRamakrishnan G proposed openstack/ironic-specs: New driver interface for RAID configuration  https://review.openstack.org/13589910:01
*** alexpilotti has joined #openstack-ironic10:08
Haomeng|2humble__: hi10:08
humble__why we don't install bootloader and kernel into local hardisk10:09
Haomeng|2humble__: IPA will support these functions10:10
*** athomas has joined #openstack-ironic10:10
*** erwan_taf has quit IRC10:10
Haomeng|2humble__: but our default pxe_ipmitool driver does not support these advance functions10:10
Haomeng|2humble__: ipa will be default driver - https://github.com/openstack/ironic-python-agent/10:11
Haomeng|2humble__: default agent10:11
humble__in pxe_ipmitool when we write bootloader and kernel into local harddisk10:11
Haomeng|2humble__: pxe_ipmitool does not support bootloader creation I think10:11
humble__must pxe when machine restart to get the kernel and initrd?10:12
*** chenglch has quit IRC10:14
*** yuanying_ has joined #openstack-ironic10:15
*** yuanying has quit IRC10:17
Haomeng|2humble__: yes, if pxe boot, the remote kernel and initrd is required, else it can be found in local boot fs10:18
humble__thks10:20
Haomeng|2humble__: welcome10:27
*** achanda has quit IRC10:28
*** rakesh_hs has joined #openstack-ironic10:29
*** naohirot_ has quit IRC10:35
*** andreykurilin_ has joined #openstack-ironic10:36
*** teju has left #openstack-ironic10:48
*** Haomeng|2 has quit IRC10:51
*** athomas has quit IRC10:54
*** Haomeng has joined #openstack-ironic10:54
*** rameshg87 has joined #openstack-ironic10:56
rameshg87ifarkas, hi10:56
rameshg87ifarkas, i have posted full spec for raid configuration https://review.openstack.org/#/c/135899/10:57
rameshg87ifarkas, please have a look at it10:57
ifarkasrameshg87, ok, thanks11:01
ifarkasrameshg87, the changes are only in machine-independent way?11:03
rameshg87ifarkas, yeah11:03
rameshg87ifarkas, based on the comments recieved11:03
rameshg87ifarkas, i will need to leave now. please have a look at it and comment on the spec.11:03
ifarkasrameshg87, sure11:03
rameshg87ifarkas, i will discuss this with you if we get a chance today itself later.11:03
rameshg87ifarkas, thanks11:03
*** rameshg87 has quit IRC11:04
*** ramineni has quit IRC11:05
*** athomas has joined #openstack-ironic11:08
*** lazy_prince has quit IRC11:23
*** smoriya has quit IRC11:39
*** pradipta is now known as pradipta_away11:39
*** Masahiro has joined #openstack-ironic11:42
*** pensu has quit IRC11:45
*** Masahiro has quit IRC11:46
*** naohirot_ has joined #openstack-ironic11:49
*** jiangfei has quit IRC11:53
*** pensu has joined #openstack-ironic12:00
*** naohirot_ is now known as naohirot12:08
*** masco has quit IRC12:09
*** dtantsur|bbl is now known as dtantsur12:10
sambettsMorning dtantsur12:12
dtantsursambetts, morning12:14
*** vdrok has quit IRC12:23
openstackgerritDmitry Tantsur proposed stackforge/ironic-discoverd: Retry on Conflict exceptions from Ironic  https://review.openstack.org/14109612:24
*** vdrok has joined #openstack-ironic12:25
*** rakesh_hs has quit IRC12:25
*** achanda has joined #openstack-ironic12:39
*** humble__ has quit IRC12:39
*** pensu has quit IRC12:43
openstackgerritYuriy Zveryanskyy proposed openstack/ironic-specs: Add a new driver for Fuel Agent  https://review.openstack.org/13811512:44
*** dprince has joined #openstack-ironic12:45
openstackgerritDmitry Tantsur proposed stackforge/ironic-discoverd: Retry on Conflict exceptions from Ironic  https://review.openstack.org/14109612:49
*** pensu has joined #openstack-ironic13:03
*** afaranha has quit IRC13:07
openstackgerritDmitry Tantsur proposed stackforge/ironic-discoverd: Review and fix logging messages  https://review.openstack.org/14136613:13
*** pensu has quit IRC13:15
dtantsurifarkas, hi, could you have a look at these 2 patches? ^^^13:15
ifarkasdtantsur, yeah13:16
*** pensu has joined #openstack-ironic13:17
*** erwan_taf has joined #openstack-ironic13:18
*** Masahiro has joined #openstack-ironic13:30
openstackgerritNaohiro Tamura proposed openstack/ironic-specs: iRMC Virtual Media Deploy Driver for Ironic  https://review.openstack.org/13486513:32
*** Masahiro has quit IRC13:35
openstackgerritDmitry Tantsur proposed stackforge/ironic-discoverd: Wait for power off before finishing discovery  https://review.openstack.org/14137013:45
dtantsurifarkas, also this if you have some time ^^^13:48
ifarkasdtantsur, wow, okay13:48
dtantsurifarkas, heh sorry for the amount, started real testing and hit a lot of issues :) these must be the last for today13:49
*** pensu has quit IRC13:58
*** masco has joined #openstack-ironic14:00
openstackgerritTan Lin proposed openstack/ironic-specs: Update amt-pxe-driver spec  https://review.openstack.org/14126914:08
*** achanda has quit IRC14:11
*** ryanpetrello has joined #openstack-ironic14:11
*** achanda has joined #openstack-ironic14:12
openstackgerritMerged stackforge/ironic-discoverd: Retry on Conflict exceptions from Ironic  https://review.openstack.org/14109614:16
*** k4n0 has quit IRC14:17
*** trown has quit IRC14:26
*** trown has joined #openstack-ironic14:27
*** dlpartain has joined #openstack-ironic14:48
*** Marga_ has joined #openstack-ironic14:59
*** Marga_ has quit IRC14:59
*** Marga_ has joined #openstack-ironic15:00
*** r-daneel has joined #openstack-ironic15:00
openstackgerritMerged stackforge/ironic-discoverd: Review and fix logging messages  https://review.openstack.org/14136615:03
openstackgerritNaohiro Tamura proposed openstack/ironic-specs: iRMC Management Driver for Ironic  https://review.openstack.org/13602015:05
naohirotgood night ironic15:09
*** naohirot has quit IRC15:09
NobodyCamgood morning Ironic15:17
dtantsurNobodyCam, morning, TGIF15:18
NobodyCammorning dtantsur and oh YES TGIF!!!!15:18
NobodyCamcuihaozhi: you get it working?15:19
*** Masahiro has joined #openstack-ironic15:19
*** Masahiro has quit IRC15:24
ShrewsNobodyCam: morning15:24
*** anderbubble has joined #openstack-ironic15:25
ShrewsNobodyCam: also, testing out 139215... i'm not seeing any of the debug state transition messages show up15:25
Shrewsoh. maybe it needs the followup patches for that15:26
*** killer_prince has joined #openstack-ironic15:26
NobodyCammorning Shrews15:26
*** killer_prince is now known as lazy_prince15:27
ShrewsNobodyCam: yup. nm. stupid pre-second-cup-of-coffee comment... feel free to ignore me  :)15:28
NobodyCamhehehe I'm only just starting my first cup15:28
*** dlpartain has left #openstack-ironic15:34
openstackgerritVladyslav Drok proposed openstack/ironic: Get rid of set_failed_state duplication  https://review.openstack.org/14140815:38
openstackgerritMerged stackforge/ironic-discoverd: Wait for power off before finishing discovery  https://review.openstack.org/14137015:46
BadCubGood morning Ironic15:55
NobodyCamgood morning BadCub15:55
NobodyCamand morning to TheJulia :-p15:55
TheJuliaGood morning NobodyCam15:56
NobodyCam:)15:56
BadCubGood Morning Nobodycam and TheJulia16:00
*** yjiang5 is now known as yjiang5_away16:00
*** ifarkas has quit IRC16:01
*** marcoemorais has joined #openstack-ironic16:01
*** marcoemorais1 has joined #openstack-ironic16:02
*** pcrews has joined #openstack-ironic16:05
*** marcoemorais has quit IRC16:05
NobodyCamanyone have a quick second to take a quick look at something.16:05
NobodyCamlines 95 thru 98 of https://review.openstack.org/#/c/136020/3..4/specs/kilo/irmc-management-driver.rst16:06
NobodyCamother end user section.16:06
*** tylerdurden has quit IRC16:06
NobodyCamI don't think thats end user tasks.. but I am low on cafine16:06
NobodyCamcaffeine even16:07
*** achanda has quit IRC16:11
ShrewsNobodyCam: if "end user" is the operator adding nodes, then it makes sense i suppose16:11
NobodyCamI generally think of that as a deployer16:11
NobodyCambimbw16:11
*** achanda has joined #openstack-ironic16:11
ShrewsNobodyCam: yeah, i agree with you. there is a deployer impact section16:12
NobodyCam:) TY Shrews :) and you've had > two cups of coffee?16:12
Shrewsyou appear to have reached proper caffination level16:12
NobodyCamlol16:12
*** achanda has quit IRC16:16
*** jerryz has quit IRC16:18
*** achanda has joined #openstack-ironic16:18
*** yjiang5_away is now known as yjiang516:19
dtantsurNobodyCam, did I answer your questions on https://review.openstack.org/#/c/135605/7/specs/kilo/inband-properties-discovery.rst ?16:21
dtantsuralso, I don't quite understand what you meant by "plans to support DIB"16:21
NobodyCamdtantsur: I have not gotten there yet. will look in a couple of minutes. oh that was be cause I say the devstack work listed and didn't see anything about adding to DIB16:23
NobodyCams/say/saw/16:23
dtantsurNobodyCam, adding to DIB - what? ramdisk is already there, link is in the references :)16:23
NobodyCamlol dtantsur my bad16:25
NobodyCamI see it16:25
NobodyCam:-p16:25
dtantsur:)16:25
NobodyCamI will update after I get thru the current review16:25
NobodyCamdtantsur: Thank you :)16:25
dtantsurok thanks16:25
devanandamorning, all16:29
NobodyCamgood morning devananda16:30
dtantsurdevananda, morning16:31
*** marcoemorais1 has quit IRC16:32
*** jcoufal has quit IRC16:33
*** Marga_ has quit IRC16:34
NobodyCambrb16:40
*** anderbubble has quit IRC16:42
*** kbyrne has quit IRC16:44
*** kbyrne has joined #openstack-ironic16:46
NobodyCamand back16:47
*** david-lyle_afk is now known as david-lyle16:49
*** achanda has quit IRC16:50
*** achanda has joined #openstack-ironic16:51
*** Nisha has joined #openstack-ironic16:53
*** kbyrne has quit IRC16:53
NobodyCamdtantsur: added comments to the spec. are you going to push up a new version?16:56
*** achanda has quit IRC16:56
dtantsurNobodyCam, depending on how much is to be changed16:56
dtantsurlemme see16:56
*** kbyrne has joined #openstack-ironic16:57
dtantsuryeah, I'll update now, I think I mostly answered Wan-yen's questions16:58
*** andreykurilin_ has quit IRC16:58
*** andreykurilin_ has joined #openstack-ironic16:58
NobodyCamya17:00
*** marcoemorais has joined #openstack-ironic17:02
*** masco has quit IRC17:02
*** achanda has joined #openstack-ironic17:03
NobodyCamand here comes the rain!17:03
* NobodyCam runs out side to make sure things are covered out side17:03
NobodyCamlol makes me feel like we're back up in WA17:07
*** Masahiro has joined #openstack-ironic17:08
TheJuliaNobodyCam: That is one thing that I actually kind of like, is a good rain storm.  The weather in North Carolina alternates back and forth too often so some seasons we barely get more than just small rain storms and fast moving fronts.17:09
JayFdtantsur: how does the ramdisk run? do you have it in DIB? or what?17:10
JayFhi TheJulia17:10
dtantsurJayF, yep https://github.com/openstack/diskimage-builder/blob/master/elements/ironic-discoverd-ramdisk17:10
JayFdtantsur: aha, that's what I was asking.17:11
NobodyCamTheJulia: :)17:11
NobodyCamJayF: I had the same question17:11
NobodyCamlol17:11
JayFdtantsur: I'm curious; I'm thinking of putting up a spec to move all ironic ramdisk builders/elements into a unified place17:11
JayFdtantsur: this would include dib elements, the builder for IPA, ISO builder, etc17:11
JayFdtantsur: would you think that'd be useful + would you want your builder there too?17:11
dtantsurJayF, ++many. yes, I do.17:11
NobodyCamJayF: that sounds like good meeting topic17:12
JayFNobodyCam: I think I'll set aside a couple of hours this weekend to put the short spec up17:12
NobodyCamI can see +'s and -'s to it17:12
dtantsurJayF, actually it sucks to have it in d-i-b: I have to ask folks who have no idea about discoverd for approval of my changes  :)17:12
JayFNobodyCam: honestly, I'm not going to lie, part of my goal for this is to get the IPA CoreOS builder separated so I can use it for other things too17:12
JayFNobodyCam: like ... discoverd, or the pxe ramdisk17:12
* JayF giving away his secret plan to have CoreOS take over the world17:12
openstackgerritDmitry Tantsur proposed openstack/ironic-specs: In-band hardware properties inspection via ironic-discoverd  https://review.openstack.org/13560517:12
dtantsurNobodyCam, ^^^17:13
*** Masahiro has quit IRC17:13
NishaNobodyCam, JayF, jroll , devananda can i request reviews for https://review.openstack.org/#/c/100951/17:13
* JayF will likely not have time to give that a good review today17:13
NobodyCamJayF: hehehehe https://www.youtube.com/watch?v=mYvAYwpUDv817:13
JayFNobodyCam: the best episode of that is when Brain uses his brain to make Pinky smarter, and super-smart Pinky develops and equation that proves the Brain is why all their schemes fail17:14
* JayF loved animaniacs as a kid17:14
dtantsurcalling it a day, folks, having a corporate Christmas celebration today :)17:14
dtantsurso unless you have questions about discoverd....17:14
dtantsur... g'night :)17:14
NobodyCamlol ++++17:15
NobodyCamnight dtantsur17:15
Nishadtantsur, :)17:15
NobodyCamhave a good weekend17:15
Nishagood night dtantsur17:15
JayFnight17:15
NobodyCamhi Nisha17:15
*** dtantsur is now known as dtantsur|afk17:15
NishaNobodyCam, hi17:15
NishaNobodyCam, i responded to your comment in the spec17:15
NishaNobodyCam, did you had a chance to see it17:16
Nisha?17:16
NobodyCamnot just yet... going grab a fresh cup of Joe and give it a look over17:17
*** sambetts has quit IRC17:20
*** sambetts has joined #openstack-ironic17:20
TheJuliaJayF: Greetings :)17:21
*** bauzas is now known as bauzer17:27
*** dlaube has joined #openstack-ironic17:29
*** ryanpetrello_ has joined #openstack-ironic17:29
*** tylerdurden has joined #openstack-ironic17:29
*** athomas has quit IRC17:29
NobodyCamNisha: are you planing on adding the periodic task for timeout detection?17:30
*** ryanpetrello has quit IRC17:31
*** ryanpetrello_ is now known as ryanpetrello17:31
*** yjiang5 is now known as gingeryjiang17:32
*** gingeryjiang is now known as gingerjiang17:33
*** openstackgerrit has quit IRC17:34
*** openstackgerrit has joined #openstack-ironic17:34
dlaubeg'morning17:36
NobodyCammorning dlaube :)17:36
*** andreykurilin_ has quit IRC17:38
*** andreykurilin_ has joined #openstack-ironic17:39
*** anderbubble has joined #openstack-ironic17:41
NishaNobodyCam, thanks for review17:44
NobodyCamNisha: thats about it. I +2 after that minor change :)17:45
NishaNobodyCam, i saw the comment. Its actually the periodic task only17:45
NobodyCamNisha: are you going to create that task17:45
NishaNobodyCam, do you want me to clarify that?17:45
NishaThe inspection will poll as we do for power today,,,17:46
Nishas/,,,/...17:46
Nisha:)17:46
NobodyCamyes but we dont have that task now. (that I recall) so you'll be creating it?17:46
Nishayes17:46
NobodyCamthen yep can you just add that to the work items section17:47
NobodyCam:)17:47
Nishaok17:47
Nisha:)17:47
NobodyCamthank you for all the work and effort :)17:47
NishaNobodyCam, thanks for the reviews17:47
Nisha:)17:47
NobodyCam:)17:48
Nishawhen is K1 deadline?17:48
NishaK2 i remember is Jan 5 or so17:48
*** ndipanov_ has quit IRC17:49
*** openstackgerrit has quit IRC17:49
*** openstackgerrit has joined #openstack-ironic17:49
*** marcoemorais has quit IRC17:50
NobodyCamNisha: https://wiki.openstack.org/wiki/Kilo_Release_Schedule17:50
*** marcoemorais has joined #openstack-ironic17:50
NishaNobodyCam, during K2 we wont have much access to the systems during shutdown17:51
JayFshutdown?17:52
NishaNobodyCam, so basically we would be working during k1 and then k317:52
NishaJayF, HP has christmas shutdown17:52
JayFaha17:52
*** derekh has quit IRC17:52
* JayF taking christmas week off17:52
NishaJayF, :)17:52
NobodyCam:)17:54
*** andreykurilin_ has quit IRC17:54
openstackgerritNisha Agarwal proposed openstack/ironic-specs: Discover node properties using new CLI node-inspect  https://review.openstack.org/10095117:56
NishaNobodyCam, ^^^^^17:56
NobodyCamNisha: voted :)18:04
JayFNobodyCam: Why are we OK with making node.create async in some cases?18:04
JayFNobodyCam: reading through nisha's spec now18:04
NobodyCamthats the not chosen way?18:05
* JayF very confused by that paragraph then18:05
* JayF will have a -1 for that spec with a couple of nits18:05
NobodyCamJayF: see line 9718:05
NishaJayF, request review for introspection spec :)18:06
JayFNisha: that's what I'm doing18:06
Nisha#link https://review.openstack.org/#/c/100951/5418:06
NishaJayF, wow18:06
Nishathanks18:06
JayFyeah if NobodyCam is going to +2 it18:07
JayFI wanna have a look ;)18:07
devanandaso I still have concerns about the non-REST-fullness of this18:07
devanandacreating a new resource, nodes/UUID/inspect18:07
devanandawhere a POST to that resource modifies a *different* resource18:08
devanandaeg, POST {} /v1/nodes/UUID/inspect18:08
devanandathen GET /v1/nodes/UUID/state/provision to see the result18:08
devananda... wtf?18:08
devanandaseems a little odd to me18:08
devanandamaybe others have a different opinion18:08
NishaPOST is not there18:08
Nishadevananda, i assume you are saying about introspection spec18:09
*** Marga_ has joined #openstack-ironic18:09
devanandasorry, PUT18:09
Nishadevananda, why provision is required to see the values18:10
Nishajust node-show should be fine to see finally updated values18:10
devanandaNisha: you are confusing the CLI with the REST API18:11
*** romcheg has quit IRC18:11
Nishadevananda, ok in terms of API we will do GET18:11
devanandaNisha: the provision state changes18:12
Nishadevananda, yes18:12
devanandathat is accessible via GET /v1/nodes/UUID/state18:12
Nishaok18:12
Nishadevananda, then what is your suggestion18:12
Nishahow it shall be?18:12
Nishashould it be subelement of state?18:13
Nishaie. PUT /v1/nodes/UUID/state/inspect18:13
Nisha?18:13
Nishasomething like this?18:13
*** jistr has quit IRC18:14
dlaubehey guys, I just want to confim something here… when I use nova boot to schedule and deploy an ironic node, is it nova that is talking to neutron or ironic?18:16
*** harlowja_away is now known as harlowja18:17
NobodyCamdlaube: ironic only updates neutron18:18
NobodyCamwith dhcp stuff18:18
*** Marga_ has quit IRC18:19
dlaubehmm18:19
*** Marga_ has joined #openstack-ironic18:19
dlaubebut I see a port created with public IPs when I do a neutron port-list18:19
*** Marga_ has quit IRC18:20
dlaubegranted, there are certainly some dhcp options when I port-show18:20
NobodyCamdlaube: https://github.com/openstack/ironic/blob/master/ironic/drivers/modules/pxe.py#L33218:21
NobodyCamso nova creates the port we just update the dhcp prams18:22
*** russellb is now known as rustlebee18:29
dlaubecool, ty NobodyCam18:29
*** pensu has joined #openstack-ironic18:30
*** Nisha has quit IRC18:31
*** anderbubble has quit IRC18:35
*** datajerk1 has quit IRC18:45
NobodyCamroflmao: http://wonderfulengineering.com/skyscraper-literally-melts-a-car-in-england/18:46
NobodyCamNg: ^^^^^18:46
*** datajerk has joined #openstack-ironic18:47
NobodyCamcan we say parabolic18:47
JayFadam_g: if I suspect a change is more likely to break under parallel testing, is there any way to get jenkins to run the parallel-nv job against a Nova change?18:49
*** afaranha has joined #openstack-ironic18:50
*** Marga_ has joined #openstack-ironic18:51
adam_gJayF, ATM no. if it were added to the nova's experimental check, you could trigger one that way18:51
JayFaight, just thought I'd ask because the results of that on https://review.openstack.org/#/c/141188/ could be interesting given that could introduce some contention18:51
*** mikedillion has joined #openstack-ironic18:53
adam_gJayF, if you wanted to be sneaky you could backport the change to stable/juno and have it run the grenade job there, which runs tempest in parallel similar to the -nv job18:53
*** anderbubble has joined #openstack-ironic18:54
JayFnah it's fine; was just generally curious. If I really want it run I'll do it in devstack18:54
*** Masahiro has joined #openstack-ironic18:57
*** Marga_ has quit IRC18:57
NobodyCambrb19:00
*** dprince has quit IRC19:00
*** marcoemorais has quit IRC19:01
*** Masahiro has quit IRC19:01
*** marcoemorais has joined #openstack-ironic19:01
*** marcoemorais has quit IRC19:02
*** marcoemorais has joined #openstack-ironic19:02
dlaubeso I think I remember someone here telling me that rackspace does not use the metadata service for config at provision time. If so, are you using configdrive?19:04
*** pensu has quit IRC19:04
JayFOnMetal uses configdrive.19:04
JayFI'm not sure what our virt cloud does.19:05
JayFbut I'm pretty sure it's not metadata service19:05
dlaubeJayF: thanks, I'm reading through http://docs.openstack.org/user-guide/content/enable_config_drive.html  right now19:07
*** Marga_ has joined #openstack-ironic19:08
dlaubeis there anything specific to ironic that I should know if I want to make use of config drive?19:08
JayFit's not supported yet.19:08
JayFUnless you use jroll's patches19:08
JayFJoshNang: ^ did that land?19:09
* JayF can't remember19:09
JoshNangJayF: hmm don't remember19:09
JoshNangit's hard to keep track of what's upstream and downstream :( :( :(19:09
JayFwe're like salmon19:10
JayFalways swimming upstream19:10
dlaubejumping waterfalls19:11
dlaubedodging bears19:11
JayFdlaube: looks like the Ironic side is in, but the Nova configdrive support in the virt driver isn't19:11
dlaubeok ty jayf19:12
*** marcoemorais has quit IRC19:15
*** marcoemorais has joined #openstack-ironic19:15
*** Nisha has joined #openstack-ironic19:18
*** Marga_ has quit IRC19:21
openstackgerritAbhishek Chanda proposed openstack/ironic: Remove unused oslo-incubator modules  https://review.openstack.org/14147219:22
*** Marga_ has joined #openstack-ironic19:26
*** ChuckC has quit IRC19:27
openstackgerritAbhishek Chanda proposed openstack/ironic: Remove unused oslo-incubator modules  https://review.openstack.org/14147219:32
*** bigjools has quit IRC19:34
*** bigjools has joined #openstack-ironic19:34
NgNobodyCam: yeah, that is pretty crazy19:36
*** mikedillion has quit IRC19:38
NobodyCamlol19:40
*** Marga_ has quit IRC19:42
*** Marga_ has joined #openstack-ironic19:42
*** ryanpetrello has quit IRC19:50
*** ryanpetrello has joined #openstack-ironic19:54
NobodyCambrb ... foody things19:58
openstackgerritAbhishek Chanda proposed openstack/ironic: Remove unused oslo-incubator modules  https://review.openstack.org/14147820:01
*** Marga_ has quit IRC20:02
*** Marga_ has joined #openstack-ironic20:03
*** Marga_ has quit IRC20:08
*** marcoemorais has quit IRC20:08
*** ryanpetrello has quit IRC20:08
*** marcoemorais has joined #openstack-ironic20:08
*** alexpilotti has quit IRC20:10
*** dtantsur|afk has quit IRC20:11
*** alexpilotti has joined #openstack-ironic20:12
*** dprince has joined #openstack-ironic20:17
*** alexpilotti has quit IRC20:22
*** alexpilotti has joined #openstack-ironic20:23
*** dtantsur has joined #openstack-ironic20:24
*** alexpilotti has quit IRC20:27
*** mjturek has quit IRC20:28
*** Marga_ has joined #openstack-ironic20:33
*** alexpilotti has joined #openstack-ironic20:37
*** achanda has quit IRC20:37
*** Marga_ has quit IRC20:38
*** mjturek has joined #openstack-ironic20:39
*** Masahiro has joined #openstack-ironic20:46
*** ChuckC has joined #openstack-ironic20:50
*** Masahiro has quit IRC20:50
*** alexpilotti has quit IRC20:57
*** alexpilotti has joined #openstack-ironic20:58
*** alexpilotti has quit IRC20:59
*** alexpilotti has joined #openstack-ironic21:01
*** alexpilotti has quit IRC21:01
*** alexpilotti has joined #openstack-ironic21:02
Shrewsheh. started up my coffee maker about an hour ago but forgot about it b/c i was so deep into reviewing the state machine code21:03
*** alexpilotti has joined #openstack-ironic21:03
*** alexpilotti has quit IRC21:05
*** ryanpetrello has joined #openstack-ironic21:06
*** dprince has quit IRC21:07
*** alexpilotti has joined #openstack-ironic21:07
*** alexpilotti has quit IRC21:09
*** alexpilotti has joined #openstack-ironic21:14
*** alexpilotti has quit IRC21:15
NobodyCamlol forget about COFFEE never21:17
Shrews:)21:18
NishaNobodyCam, when is the node in INSPECTED state and when in MANAGED mode for the node fr which introspection is going on?21:19
NobodyCamNisha: ?? I don't quite understand the question.21:23
NishaNobodyCam, leave it...i think i asked wrong ques21:23
NobodyCamok be the spec a node must be in managed state to be able to go in to inspected21:24
NobodyCams/be/per/21:24
*** romcheg has joined #openstack-ironic21:29
*** Marga_ has joined #openstack-ironic21:34
openstackgerritNisha Agarwal proposed openstack/ironic-specs: Discover node properties using new CLI node-inspect  https://review.openstack.org/10095121:37
NishaNobodyCam, JayF ^^^^^21:37
NishaNobodyCam, require your vote again21:38
*** Marga_ has quit IRC21:38
NobodyCamhehehe gota see what changed :)21:38
NobodyCamlookng now21:39
Nishaaddressed JayF's minor comments21:39
Nishaok21:39
NobodyCamJayF: where you looking for wording changes in the Alternatives section?21:42
*** alexm__ has quit IRC21:43
* JayF replied with -1 and a couple of comments21:49
NishaNobodyCam, not sure if JayF is there....21:49
JayFNobodyCam: yes :)21:49
NishaJayF, :)21:49
NobodyCamNisha:21:49
NobodyCamd'oh21:49
JayFAlso whether or not I respond in IRC doesn't always map to whether or not I'm here :)21:49
JayFalso fwiw, I'll likely vote 0 or +1 on this not +2, as I haven't given it a through enough review21:50
NishaJayF, i have addressed your comments21:50
Nishaand reposted the patch21:50
NobodyCamI think JayF was looking for stronger wording in the Alternatives section21:50
JayFI put two new comments on the review21:50
JayFwhich are continuations of the old one21:50
* JayF talks about reviews in gerrit where they are documented and such21:51
JayFbecause conversations in here == I'm likely to forget21:51
*** Marga_ has joined #openstack-ironic21:52
NobodyCamJayF: theres always http://eavesdrop.openstack.org/irclogs/%23openstack-ironic/21:52
NobodyCamlol21:53
JayFI log all my IRC21:53
JayFand have for years and years21:53
JayFbut reading IRC logs is not very friendly :)21:53
NishaJayF, the NIC names can be a label like NIC1, NIC2 which is a seperate field  apart from mac address21:53
NobodyCam++ JayF21:53
JayFNisha: Given there's no analogue for that for some other mechanisms21:53
JayFI'm not sure it should be part of the general interface21:53
JayFMAC address should always be enough to uniquely identify a NIC21:54
Nishathat is the implementation each driver will do in the inspect_hardware() function. it will not come anywhere else21:54
Nishamoreover it is not saved anywhere21:54
JayFMy thought is more21:54
JayFI implement this in $jays_fancy_bmc_driver21:54
JayFbut my BMC doesn't provide a "name" field21:55
JayFwhat do I put there?21:55
JayFIt's not going to be consistent across drivers21:55
JayFwhich means it's not well abstracted21:55
JayFso I'd prefer that either go away or be more specifically indicated as to what it means21:55
Nishathats fine, because this not an explicit field. This is just anidentifier along with mac address21:55
JayFwhy do you need it then?21:56
JayFIf it doesn't matter; remove it. If it matters, it should be more well defined. Right?21:56
NishaI am not sure how other drivers will have the logic of port create/delete21:56
Nishafor port create logic i actually required it for iLO drivers21:57
JayFThat's important to hash out in the spec then, because this is a general spec all drivers will have to implement, right?21:57
*** Marga_ has quit IRC21:57
Nishaok i will remove if the drivers can implement just with mac addresses21:57
Nishai can have it in iLO drivers21:57
JayFI think that works for me, thanks.21:58
*** Marga_ has joined #openstack-ironic21:59
*** anderbubble has quit IRC21:59
openstackgerritNisha Agarwal proposed openstack/ironic-specs: Discover node properties using new CLI node-inspect  https://review.openstack.org/10095122:02
NishaJayF, NobodyCam ^^^^22:02
*** romcheg1 has joined #openstack-ironic22:03
NishaJayF, could you check again22:03
*** romcheg has quit IRC22:05
NobodyCambrb22:07
*** romcheg has joined #openstack-ironic22:09
*** romcheg1 has quit IRC22:10
openstackgerritOpenStack Proposal Bot proposed openstack/ironic: Updated from global requirements  https://review.openstack.org/14150722:15
NishaJayF, i have reposted the patch with your comments addressed. Could you relook22:18
*** tylerdurden has quit IRC22:20
*** shakamunyi has joined #openstack-ironic22:20
*** Masahiro has joined #openstack-ironic22:34
*** r-daneel has quit IRC22:35
*** ryanpetrello has quit IRC22:37
*** Masahiro has quit IRC22:39
NishaNobodyCam, and others : one quick ques22:44
NobodyCamsure22:44
Nishai upgraded nova code and restarted nova services22:44
Nishabut n-cpu doesnt compe up22:44
Nishai upgraded oslo modules22:44
Nishabut still get errors22:44
Nishais this known issue?22:45
NobodyCamwhat are the erorrs?22:45
Nishaif yes how to overcome that22:45
NobodyCamNisha: what are the errors your getting??22:46
Nishaits like in the db:22:46
Nisha u'OperationalError: (OperationalError) (1054, "Unknown column \'compute_nodes_1.host\' in \'field list\'") \'SELECT anon_1.services_created_at AS anon_1_ser22:46
Nishavices_created_at, anon_1.services_updated_at AS anon_1_services_updated_at,22:46
Nishawhile as of now i havent changed anything apart from pulling the code22:46
NobodyCamI haven't seen that but kinda looks like you need to do a db sync22:47
Nishai did all that22:47
Nisha:(22:47
NobodyCam:( check with the nova folk?22:48
NobodyCammaybe they know of it?22:48
NishaNobodyCam, request your vote for introspection spec again22:48
Nishayes i asked there but none is responding22:48
NobodyCamack will look again in a bit.22:49
Nishaok22:49
Nishathank you22:49
NobodyCam:)22:49
clif_hJayF: try to figure out from adam_g in ironic channel how to get tempest parallel-nv job for Ironic running locally?23:05
clif_hadam_g: do you know how to accomplish this?23:06
*** andreykurilin_ has joined #openstack-ironic23:07
JayFadam_g: ^ clif_h wrote that patch for nova that I was asking about getting the tempest job running on23:07
JayFadam_g: he has it in devstack locally, is there a way he can run it?23:07
clif_hthis is to further test https://review.openstack.org/#/c/141188/23:07
JayFbtw, adam_g meet clif_h, clif_h is a new Racker working with our team23:07
JayFadam_g is king of the Ironic gate (whether he likes it or not :P)23:08
*** Haomeng|2 has joined #openstack-ironic23:09
*** Haomeng has quit IRC23:09
adam_gha23:11
adam_gclif_h, hey!23:11
clif_hadam_g: greetings23:11
adam_gclif_h, are you attempting to run it against a devstack-deployed environment or one of your onw?23:12
adam_g*own23:12
clif_hdevstack deployed23:12
adam_gthe parallel-nv job is basically just a run of the entire tempest suite, and relies on certain config flags being set in tempest to know what tests to skip23:12
adam_gok, those flags are set by tempest so you should be setup to run it already23:12
adam_gclif_h, how many ironic nodes do you have enrolled?23:13
adam_ger, those flags are set by devstack23:13
JayFIs tempest installed by default in devstack?23:13
adam_gJayF, yeah23:13
JayFadam_g: he's installed straight stock from the wiki for Ironic + agent23:13
adam_gand configured23:13
JayFniiiice23:13
clif_hthere are 3 ironic nodes23:14
clif_hone on the other two are off currently23:14
*** romcheg has quit IRC23:14
adam_gclif_h, so if you free all your nodes up (no instances, powered off, etc) you should be able to just run tempest out of /opt/stack/tempest23:15
adam_gtempest tox -esmoke -- --concurrency=223:16
adam_ger23:16
adam_gtox -esmoke -- --concurrency=223:16
adam_gclif_h, note that theres a new test that was merged today that isn't passing against ironic, you can pull this down if you want or just ignore the fails https://review.openstack.org/#/c/141456/23:16
clif_hwhen you say free up, you mean delete the instance from nova right?23:17
adam_gclif_h, yea23:18
clif_hhrm when I did 'nova delete testing'23:23
clif_hIt went to an error state23:23
clif_hfault:  {"message": "HTTPResponse instance has no attribute 'text'", "code": 500, "created": "2014-12-12T23:21:45Z"}23:23
*** andreykurilin_ has quit IRC23:24
*** andreykurilin_ has joined #openstack-ironic23:25
*** romcheg has joined #openstack-ironic23:29
clif_hprobably my change has a bug23:32
*** Masahiro has joined #openstack-ironic23:35
*** Masahiro has quit IRC23:40
adam_gclif_h, hmm. no idea there, whats your change?23:40
clif_hhttps://review.openstack.org/#/c/141188/23:40
clif_htrying to cache the ironic client23:41
clif_hinstead of getting an entirely new one for each call to ironic23:41
JayF^ BTW root cause for caching that client is everytime you get a new client, you get a new token23:44
JayFso things like the power status loop which does for instance in instances == ddos on keystone23:45
JayFs/dd/d/23:45
clif_hhere's the trace in nova log causing the problem *I think*23:45
clif_hhttps://gist.github.com/ClifHouck/dd7ea92bff4862d6567e23:45
clif_hnot sure how the cached client would become unauthorized so quickly23:45
NobodyCamahh starbucks just got here... bbiafm23:46
*** ryanpetrello has joined #openstack-ironic23:57
adam_gclif_h, huh, i wonder if repsonse.body has anything useful23:58

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