| *** marcoemorais has quit IRC | 00:00 | |
| *** eghobo has quit IRC | 00:14 | |
| *** sheile has joined #openstack-ironic | 00:24 | |
| *** ChuckC_ has quit IRC | 00:26 | |
| *** Tsukasa_ has joined #openstack-ironic | 00:26 | |
| *** ramineni has joined #openstack-ironic | 00:32 | |
| *** ChuckC_ has joined #openstack-ironic | 00:33 | |
| *** ChuckC_ is now known as ChuckC | 00:37 | |
| *** rloo has quit IRC | 00:53 | |
| *** Haomeng|2 has quit IRC | 01:24 | |
| *** dlaube has quit IRC | 01:30 | |
| *** ramineni has quit IRC | 01:37 | |
| *** smoriya has joined #openstack-ironic | 01:45 | |
| jroll | 22:21:31 NobodyCam | zomg I just read the best amazon review I have seen... I truly roflmao <- you can't say that without a link! | 01:55 |
|---|---|---|
| *** openstackgerrit has quit IRC | 02:01 | |
| *** Poornima_ has joined #openstack-ironic | 02:12 | |
| *** openstackgerrit has joined #openstack-ironic | 02:21 | |
| *** pcrews has quit IRC | 02:30 | |
| NobodyCam | lol | 02:37 |
| NobodyCam | jroll: see pm | 02:38 |
| *** ramineni has joined #openstack-ironic | 02:46 | |
| *** Haomeng has joined #openstack-ironic | 02:55 | |
| *** marcoemorais has joined #openstack-ironic | 03:09 | |
| *** marcoemorais1 has joined #openstack-ironic | 03:11 | |
| *** marcoemorais has quit IRC | 03:14 | |
| *** Haomeng has quit IRC | 03:18 | |
| *** killer_prince is now known as lazy_prince | 03:32 | |
| *** penick has joined #openstack-ironic | 03:46 | |
| *** penick_ has joined #openstack-ironic | 03:47 | |
| *** penick has quit IRC | 03:51 | |
| *** penick_ is now known as penick | 03:51 | |
| *** Poornima_ has quit IRC | 03:52 | |
| *** lazy_prince is now known as killer_prince | 04:02 | |
| *** killer_prince has quit IRC | 04:09 | |
| *** pcrews has joined #openstack-ironic | 04:11 | |
| *** Tsukasa_ has quit IRC | 04:21 | |
| *** killer_prince has joined #openstack-ironic | 04:21 | |
| *** killer_prince is now known as lazy_prince | 04:22 | |
| *** teju has joined #openstack-ironic | 04:22 | |
| *** Poornima_ has joined #openstack-ironic | 04:22 | |
| *** nikunj2512 has joined #openstack-ironic | 04:30 | |
| *** k4n0 has joined #openstack-ironic | 04:46 | |
| *** chenglch has joined #openstack-ironic | 04:47 | |
| openstackgerrit | xu-haiwei proposed a change to openstack/ironic: Fix typo to show correct log message https://review.openstack.org/122316 | 04:56 |
| *** rameshg87 has joined #openstack-ironic | 04:56 | |
| *** pensu has joined #openstack-ironic | 05:02 | |
| *** pcrews has quit IRC | 05:08 | |
| *** HenryG has quit IRC | 05:09 | |
| teju | hi all....i would like to implement a simple driver with IPMI features & additional features....any idea where to start? | 05:14 |
| *** Haomeng has joined #openstack-ironic | 05:14 | |
| *** rushiagr_away is now known as rushiagr | 05:16 | |
| teju | Haomeng : morning :) | 05:18 |
| Haomeng | teju: morning:) | 05:18 |
| *** pensu has quit IRC | 05:18 | |
| *** pensu has joined #openstack-ironic | 05:19 | |
| *** harlowja is now known as harlowja_away | 05:20 | |
| *** HenryG has joined #openstack-ironic | 05:22 | |
| teju | Haomeng : can u give me a high level idea on how to implement an ironic driver with IPMI features + few more features? | 05:22 |
| Haomeng | teju: you want to create new ironic driver? | 05:23 |
| Haomeng | teju: we have the base interface for driver, let me check the code | 05:23 |
| Haomeng | teju: also, you can take the existing driver as reference | 05:23 |
| teju | Haomeng : i wanna create a driver and register the same with ironic so that the driver is listed in ironic driver-list | 05:24 |
| Haomeng | ok | 05:24 |
| Haomeng | this is base driver - https://github.com/openstack/ironic/blob/master/ironic/drivers/base.py | 05:24 |
| Haomeng | and the driver interfaces definations | 05:24 |
| Haomeng | let me check the commit which add new driver | 05:25 |
| Haomeng | should be good reference for you for all changed files | 05:25 |
| rameshg87 | Haomeng, teju, may be one of mine: https://review.openstack.org/#/c/115885/ :) | 05:26 |
| Haomeng | rameshg87: thank you | 05:27 |
| rameshg87 | Haomeng, teju, the driver is defined in https://review.openstack.org/#/c/115885/14/ironic/drivers/ilo.py and you inform about the driver in https://review.openstack.org/#/c/115885/14/setup.cfg | 05:27 |
| Haomeng | teju: and I found out this one - https://github.com/openstack/ironic/commit/2089d3c01b4168e07d303e14da9a05608e921f28#diff-5f0956ffc87c3889beab022191583a11 | 05:27 |
| Haomeng | rameshg87: yes | 05:27 |
| teju | Haomeng,rameshg87 : ok....i will extend IPMI driver and register the same with conductor so that i can use in ironic node-create -d <mydriver> ....this should be possible...right? | 05:28 |
| Haomeng | teju: the -d arguments is defined in this code | 05:29 |
| Haomeng | setup.cfg | 05:29 |
| Haomeng | https://github.com/openstack/ironic/blob/master/setup.cfg#L37 | 05:29 |
| Haomeng | once you have done your new driver, just register your driver into setup.cfg, then the new driver will be populated into database when the ironic is installed by python setup.py install | 05:30 |
| Haomeng | and remember to enable them in our ironic.conf | 05:31 |
| teju | Haomeng : in my ironic,conf, i dont see any option like enabled_drivers | 05:31 |
| Haomeng | on default section - enabled_drivers = fake,agent_ssh,agent_ipmitool | 05:32 |
| Haomeng | then it can be seen in the 'ironic driver-list' comamnd call | 05:33 |
| Haomeng | http://paste.openstack.org/show/112710/ | 05:33 |
| Haomeng | teju: this is our sample configuration file - https://github.com/openstack/ironic/blob/master/etc/ironic/ironic.conf.sample#L220 | 05:34 |
| teju | Haomeng : i have icehouse setup and i dont see any setup.cfg file ..... | 05:36 |
| Haomeng | this one - https://github.com/openstack/ironic/blob/master/setup.cfg#L37 | 05:37 |
| Haomeng | it is used in installation time | 05:37 |
| Haomeng | not runtime | 05:37 |
| Haomeng | it is the feed file for python setup.py | 05:37 |
| *** lazy_prince has quit IRC | 05:37 | |
| *** lazy_prince has joined #openstack-ironic | 05:38 | |
| Haomeng | by default devstack installation, it is located in this path - /opt/stack/ironic/setup.cfg | 05:38 |
| teju | Haomeng : i thought i can just write a driver and restart ironic services to register the driver....guess I am wrong... | 05:39 |
| Haomeng | teju: let me check the logic how our driver is registered into our ironic database | 05:40 |
| Haomeng | should be handled by conductor on starting time | 05:41 |
| Haomeng | ironic.common.driver_factory | 05:41 |
| Haomeng | https://github.com/openstack/ironic/blob/07122257699706ff42e9c2a958b42eaa3c30e0a0/ironic/common/driver_factory.py#L115 | 05:42 |
| teju | Haomeng : i wrote a pxe_dup.py in ironic/drivers and i restarted ironic services...the driver is not listed in ironic driver-list....anything else to be done? | 05:45 |
| Haomeng | teju: check conductor logs to see the err/exceptions | 05:49 |
| teju | Haomeng : i see errors, but those are bcoz of seamicro drivers | 05:51 |
| Haomeng | teju: did you add your driver into entrypoints of setup.cfg, and run python setup.py again? | 05:51 |
| Haomeng | https://github.com/openstack/ironic/blob/8fdd7d0d8d4005fa913d453e0b086f9a7ae924d7/ironic/conductor/manager.py#L26 | 05:51 |
| Haomeng | then driver factory can get the new driver, and our conductor manager will load it on init - https://github.com/openstack/ironic/blob/8fdd7d0d8d4005fa913d453e0b086f9a7ae924d7/ironic/conductor/manager.py#L190 | 05:53 |
| Haomeng | so how about the conductor log for such info - INFO ironic.common.driver_factory [-] Loaded the following drivers: | 05:55 |
| Haomeng | teju: make sure it is in ironic.conf enabled_drivers and in our setup.cfg also, then run 'python setup.py install' again to register the new python *entry_points* | 05:57 |
| teju | Haomeng : i dont have setup.py or setup.cfg file anywhere | 06:01 |
| Haomeng | teju: how did you install the ironic? | 06:01 |
| Haomeng | it is one of our ironic source code | 06:01 |
| teju | Haomeng : http://docs.openstack.org/developer/ironic/deploy/install-guide.html | 06:02 |
| Haomeng | I understand you use the rpm which is pre-installed already, so setup.cfg is not required | 06:03 |
| Haomeng | can you find ironic.egg-info folder in your sys? | 06:05 |
| Haomeng | we can modify entry_points.txt by manually to add your new drivers if you install the ironic with rpm | 06:06 |
| teju | Haomeng : ironic-2014.1.rc1.egg-info | 06:06 |
| Haomeng | yes | 06:06 |
| Haomeng | cd it | 06:06 |
| Haomeng | and find entry_points.txt file | 06:06 |
| Haomeng | it is python entry points definations | 06:06 |
| rameshg87 | Haomeng, teju, it's easier to do "python setup.py egg_info" | 06:06 |
| rameshg87 | Haomeng, teju, on the ironic directory, it will regenerate entry_points.txt | 06:06 |
| Haomeng | rameshg87: ok, good idea | 06:07 |
| teju | Haomeng : got it...so, if I add an entry in entry_points.txt and restart conductor service, will it show up in ironic driver-list? | 06:08 |
| Haomeng | rameshg87: but we have no setup.cfg, what is the input for these new entry points | 06:08 |
| Haomeng | teju: first, we make sure it is added into python entry_points | 06:08 |
| Haomeng | teju: then enable it in our ironic.conf enabled_drivers, and restart conductor process | 06:08 |
| teju | Haomeng : k | 06:09 |
| Haomeng | teju: good luck:) | 06:09 |
| rameshg87 | Haomeng, oh sorry, i missed that part | 06:09 |
| rameshg87 | Haomeng, i thought it was devstack :) | 06:09 |
| Haomeng | rameshg87: np | 06:09 |
| Haomeng | rameshg87: yes, teju is using the rpm which is pre-installed, so no such setup.cfg files | 06:10 |
| openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/ironic: Imported Translations from Transifex https://review.openstack.org/120701 | 06:15 |
| teju | Haomeng : i just modified entry_points.txt and restarted ironic services and the driver showed up in ironic driver-list.....:) | 06:15 |
| Haomeng | teju: cool:) | 06:15 |
| Haomeng | teju: great work:) | 06:15 |
| *** rom_little has joined #openstack-ironic | 06:16 | |
| teju | Haomeng : now, i must be able to use the same driver in ironic node-create -d <mydriver>....right? | 06:16 |
| Haomeng | teju: this is used to register a pyhsical node and which driver is used for this baremetal | 06:17 |
| Haomeng | teju: you can have try | 06:17 |
| *** rom_little has quit IRC | 06:17 | |
| Haomeng | teju: if your new driver implemented the ironic driver interfaces, should work | 06:18 |
| Haomeng | teju: especial for the power driver and deploy two main interface | 06:18 |
| Haomeng | teju: :) | 06:19 |
| teju | Haomeng : i will extend IPMI driver for power control & add few features.... | 06:19 |
| Haomeng | teju: so you define new power driver, right? | 06:23 |
| Haomeng | teju: that is cool, one question, can I ask - do you want to contribute this new driver back to our ironic community? | 06:24 |
| Haomeng | teju: if yes, the spec is required also | 06:24 |
| Haomeng | teju: if no, just ignore my question:) | 06:25 |
| teju | Haomeng : the features I am gng to add will work only for specific products...so, the new driver wont help any one else | 06:26 |
| Haomeng | teju: ok, np:) | 06:26 |
| Haomeng | teju: should be vendor suppor | 06:27 |
| Haomeng | support | 06:27 |
| teju | Haomeng : yes...it is vendor specific :0 | 06:27 |
| Haomeng | teju: :) | 06:27 |
| *** sheile has quit IRC | 06:28 | |
| teju | Haomeng : any new loadable driver should implement both power and deploy?...since they are core interfaces? | 06:39 |
| Haomeng | teju: yes | 06:40 |
| Haomeng | teju: you can take the existing driver code as sample | 06:40 |
| teju | Haomeng : ok :) | 06:42 |
| Haomeng | teju: and your new driver can reuse the existing modules also, for example - https://github.com/openstack/ironic/blob/master/ironic/drivers/pxe.py#L47 | 06:42 |
| Haomeng | teju: depends on your requirements | 06:42 |
| Haomeng | teju: if the existing modules function is not enough for you, we have to create new driver modules, else, reuse them to compile the existing driver modules into your new driver | 06:43 |
| teju | Haomeng : i will use IPMI for power interface and implement my own features for deploy interface | 06:44 |
| Haomeng | teju: great | 06:44 |
| *** ifarkas has joined #openstack-ironic | 06:50 | |
| teju | Haomeng : should my deploy class extend base.DeployInterface ? or can I just extend class object? | 06:51 |
| Haomeng | teju: should extend from DeployInterface I think, we have same interface to call same behavior for deploying | 06:54 |
| *** penick has quit IRC | 06:55 | |
| *** pensu has quit IRC | 06:57 | |
| *** pensu has joined #openstack-ironic | 06:57 | |
| *** marcoemorais1 has quit IRC | 07:11 | |
| *** bluex-pl has joined #openstack-ironic | 07:21 | |
| *** ndipanov_gone is now known as ndipanov | 07:40 | |
| *** lazy_prince has quit IRC | 07:42 | |
| *** killer_prince has joined #openstack-ironic | 07:52 | |
| *** killer_prince is now known as lazy_prince | 07:52 | |
| *** ndipanov has quit IRC | 07:53 | |
| *** ndipanov has joined #openstack-ironic | 07:56 | |
| *** jistr has joined #openstack-ironic | 08:03 | |
| *** athomas has joined #openstack-ironic | 08:05 | |
| *** lazy_prince has quit IRC | 08:12 | |
| *** lazy_prince has joined #openstack-ironic | 08:12 | |
| *** Poornima_ has quit IRC | 08:14 | |
| *** lucasagomes has joined #openstack-ironic | 08:17 | |
| *** Poornima_ has joined #openstack-ironic | 08:18 | |
| *** derekh has joined #openstack-ironic | 08:25 | |
| *** bluex-pl has quit IRC | 08:28 | |
| *** MattMan has quit IRC | 08:31 | |
| *** MattMan has joined #openstack-ironic | 08:31 | |
| *** dtantsur|afk is now known as dtantsur | 08:34 | |
| dtantsur | Morning Ironic | 08:34 |
| *** k4n0 has quit IRC | 08:37 | |
| lucasagomes | dtantsur, morning | 08:40 |
| *** igordcard has joined #openstack-ironic | 08:42 | |
| *** pelix has joined #openstack-ironic | 08:48 | |
| openstackgerrit | Lucas Alvares Gomes proposed a change to openstack/ironic: Revert "Revert "Search line with awk itself and avoid grep"" https://review.openstack.org/122360 | 08:53 |
| *** k4n0 has joined #openstack-ironic | 08:53 | |
| lucasagomes | derekh, ^ -2'ed until the test env is rebuilt | 08:54 |
| derekh | lucasagomes: ok, thanks, will get the update done next time I notice the test queue is short | 08:55 |
| lucasagomes | derekh, no problem :) thank u | 08:55 |
| *** romcheg has quit IRC | 09:03 | |
| *** romcheg has joined #openstack-ironic | 09:03 | |
| *** Poornima|mtg has joined #openstack-ironic | 09:18 | |
| openstackgerrit | A change was merged to openstack/ironic: Sync the doc with latest code https://review.openstack.org/121742 | 09:19 |
| *** Poornima_ has quit IRC | 09:21 | |
| *** k4n0 has quit IRC | 09:22 | |
| *** lazy_prince is now known as killer_prince | 09:35 | |
| *** k4n0 has joined #openstack-ironic | 09:35 | |
| *** ifarkas has quit IRC | 09:41 | |
| *** ifarkas has joined #openstack-ironic | 09:41 | |
| openstackgerrit | A change was merged to openstack/ironic: Add unit tests for SNMPClient https://review.openstack.org/122175 | 09:57 |
| *** chenglch has quit IRC | 10:02 | |
| *** killer_prince is now known as lazy_prince | 10:02 | |
| openstackgerrit | Vladyslav Drok proposed a change to openstack/ironic: node_uuid should not be used to create test port https://review.openstack.org/122380 | 10:11 |
| *** nikunj2512 has quit IRC | 10:28 | |
| *** athomas has quit IRC | 10:31 | |
| openstackgerrit | A change was merged to openstack/ironic: Check whether specified FS is supported https://review.openstack.org/98102 | 10:35 |
| *** krtaylor has quit IRC | 10:53 | |
| openstackgerrit | A change was merged to openstack/python-ironicclient: Replace calls to Mock.assert_not_called https://review.openstack.org/122236 | 10:55 |
| openstackgerrit | Vladyslav Drok proposed a change to openstack/ironic: node_uuid should not be used to create test port https://review.openstack.org/122380 | 11:00 |
| * dtantsur brb | 11:01 | |
| *** ramineni has quit IRC | 11:01 | |
| *** romcheg has quit IRC | 11:06 | |
| teju | Haomeng,lucasagomes : is it possible to add a column in the ironic.nodes table just by adding an attribute in the Node class in /usr/lib/python2.7/dist-packages/ironic/db/sqlalchemy/models.py ? | 11:07 |
| lucasagomes | teju, you need a migration script too | 11:07 |
| * lucasagomes tries to find an example | 11:08 | |
| teju | lucasagomes : there is a file migration.py in the same dir | 11:08 |
| lucasagomes | teju, here https://review.openstack.org/#/q/I4688676767a80ac6f1f3d571f0df4ea78bc756e1,n,z | 11:08 |
| lucasagomes | ops | 11:09 |
| lucasagomes | teju, https://review.openstack.org/#/c/79466/ | 11:09 |
| lucasagomes | no the migration.py is fine, you don't have to touch that | 11:09 |
| lucasagomes | also, if u want to expose that new field in the api, you have to change it there too | 11:10 |
| lucasagomes | that example ^ of adding the instance_info may be complete so you can take a look at how to do it | 11:10 |
| *** romcheg has joined #openstack-ironic | 11:10 | |
| teju | lucasagomes : is this DB migration a safe operation? | 11:11 |
| lucasagomes | teju, what you mean by "safe"... I suspect it is | 11:12 |
| Haomeng | teju: we can reuse the node's driver_info/properties/instance_info fields which are dict if your new attribute is grouped by these existing dict | 11:12 |
| lucasagomes | for all db changes we make in the project we need a migration script | 11:12 |
| Haomeng | yes, we need migration script for new db object changes | 11:13 |
| *** romcheg has quit IRC | 11:15 | |
| teju | lucasagomes, Haomeng : so, instead of adding new columns, i can use extra/properties/metadata to store additional data? | 11:16 |
| lucasagomes | teju, yup, well if the type of data you want to store makes sense in one of those fields | 11:17 |
| lucasagomes | currently we have "extra" for any extra data, "driver_info" for parameters for the drivers, "instance_info" same parameters required to deploy an instance and "properties" which are the physical characteristcs of the node | 11:18 |
| lucasagomes | I would try to reuse it if possible | 11:19 |
| Haomeng | teju: yes, depends on your new field concept | 11:19 |
| teju | Haomeng : lets say if i wanna store cinder volume ID for booting a bare metal......where can that value possibly go? | 11:21 |
| *** lucasagomes is now known as lucas-hungry | 11:25 | |
| Haomeng | teju: if it is cinder volume id, how do you think about instance_info field? | 11:28 |
| Haomeng | teju: if it is instance level and the same lifecycle with instance or not? | 11:29 |
| teju | Haomeng : yes.. | 11:30 |
| Haomeng | teju: but not sure if the instance_info is populated by nova | 11:30 |
| teju | Haomeng : in my ironic.nodes table, i dont have instance_info column....i have to upgrade....and i will check | 11:31 |
| Haomeng | teju: once nova boot the instance for ironic, the instance_info will be populated with root_gb/image_url/image_source/image_checksum/swap_mb these fields | 11:32 |
| Haomeng | teju: run ironic-dbsync please | 11:32 |
| *** romcheg has joined #openstack-ironic | 11:34 | |
| teju | Haomeng : is it ok to run ironic-dbsync --config-file /etc/ironic/ironic.conf more than once? | 11:34 |
| Haomeng | teju: depends on your code version and db version | 11:35 |
| Haomeng | teju: if they are same version, will run nothing | 11:35 |
| Haomeng | teju: and ironic-dbsync will update the db objects to latest version with current code | 11:35 |
| teju | Haomeng : i haven't updated my code base | 11:36 |
| Haomeng | teju: ok, once is fine | 11:38 |
| *** krtaylor has joined #openstack-ironic | 11:39 | |
| *** dprince has joined #openstack-ironic | 11:41 | |
| teju | Haomeng : last question of the day.....i added a sub command like....ironic boot-from-volume <NODEUUID> <VolumID>.....how can I map this command to a function/method in ironic API? | 11:45 |
| lucas-hungry | Haomeng, teju so yeah instance_info seems to be the right place | 11:46 |
| lucas-hungry | nova boot has a --boot-volume <volume_id> options | 11:46 |
| lucas-hungry | that I suspect is used in this case | 11:46 |
| lucas-hungry | so we just proxy it to ironic | 11:47 |
| lucas-hungry | http://docs.openstack.org/user-guide/content/novaclient_commands.html | 11:47 |
| * lucas-hungry brb lunching | 11:47 | |
| Haomeng | teju: yes, agree with lucas, we should have same entry from nova boot with --boot-volume options | 11:51 |
| Haomeng | teju: it will be passed to ironic | 11:51 |
| *** dhellmann is now known as dhellmann_ | 11:52 | |
| teju | Haomeng : ok....anyways i wanted to understand how ironic client CLI commands are mapped to methods in ironic API? | 11:53 |
| Haomeng | teju: let me find the commit which add new sub command | 11:53 |
| *** Poornima|mtg has quit IRC | 11:53 | |
| Haomeng | teju: first we have new api for the subcommand, then we have the python ironic client new command introduced which call the new api | 11:54 |
| teju | Haomeng : k....i have API and also a new sub command, but dont know how to bind them | 11:56 |
| Haomeng | teju: we have python-ironic-client project | 11:56 |
| Haomeng | which is command code | 11:57 |
| Haomeng | teju: https://github.com/openstack/python-ironicclient | 11:57 |
| Haomeng | teju: this is a reference - https://review.openstack.org/#/c/119825/ | 11:58 |
| *** romcheg has quit IRC | 11:58 | |
| *** k4n0 has quit IRC | 12:01 | |
| Haomeng | teju: this method * do_chassis_list* is to handle *ironic chassis-list* command call - https://review.openstack.org/#/c/119825/2/ironicclient/v1/chassis_shell.py | 12:01 |
| *** romcheg has joined #openstack-ironic | 12:01 | |
| *** lucas-hungry is now known as lucasagomes | 12:04 | |
| teju | Haomeng : ok..got it....i am looking for a reference in which an API is added and also it's corresponding command in ironic client...as of now, i have dummy implementation for a sub command(just prints something on the console) | 12:07 |
| Haomeng | teju: cool | 12:07 |
| teju | Haomeng, lucasagomes : thanks :) ....bye :) | 12:10 |
| lucasagomes | teju, yvw see ya | 12:10 |
| Haomeng | teju: welcome | 12:12 |
| Haomeng | teju: see you:) | 12:12 |
| *** teju has left #openstack-ironic | 12:13 | |
| *** rushiagr is now known as rushiagr_away | 12:14 | |
| *** pensu has quit IRC | 12:20 | |
| *** rushiagr_away is now known as rushiagr | 12:24 | |
| *** romcheg has quit IRC | 12:25 | |
| *** romcheg has joined #openstack-ironic | 12:27 | |
| *** athomas has joined #openstack-ironic | 12:32 | |
| *** Poornima_ has joined #openstack-ironic | 12:34 | |
| *** linggao has joined #openstack-ironic | 12:35 | |
| openstackgerrit | Lucas Alvares Gomes proposed a change to openstack/ironic: Make context mandatory when instantiating a RPC object https://review.openstack.org/121923 | 12:35 |
| openstackgerrit | Lucas Alvares Gomes proposed a change to openstack/ironic: Do not set the context twice when forming RPC objects https://review.openstack.org/122401 | 12:35 |
| *** ramineni has joined #openstack-ironic | 12:39 | |
| *** ramineni1 has joined #openstack-ironic | 12:44 | |
| *** ramineni has quit IRC | 12:46 | |
| *** rameshg87 has quit IRC | 12:50 | |
| *** romcheg has quit IRC | 12:50 | |
| *** pensu has joined #openstack-ironic | 12:51 | |
| *** bluex-pl has joined #openstack-ironic | 12:53 | |
| *** Poornima_ has quit IRC | 12:55 | |
| *** romcheg has joined #openstack-ironic | 12:57 | |
| *** jasondotstar has joined #openstack-ironic | 12:57 | |
| lazy_prince | is there a way to find the port_id when an instance is being provisioned..? | 13:01 |
| *** ramineni1 has quit IRC | 13:04 | |
| *** rloo has joined #openstack-ironic | 13:06 | |
| *** pensu has quit IRC | 13:09 | |
| *** derekh has quit IRC | 13:09 | |
| *** lucasagomes has quit IRC | 13:09 | |
| *** derekh has joined #openstack-ironic | 13:13 | |
| *** lucasagomes has joined #openstack-ironic | 13:13 | |
| *** dhellmann_ is now known as dhellmann | 13:19 | |
| *** pcrews has joined #openstack-ironic | 13:22 | |
| Haomeng | lazy_prince: neutron port id? or ironic port id ? | 13:31 |
| lazy_prince | Haomeng: neutron port id.. | 13:32 |
| *** lucasagomes has quit IRC | 13:33 | |
| *** derekh has quit IRC | 13:33 | |
| Haomeng | lazy_prince: can we check with vm's ip? | 13:33 |
| *** lucasagomes has joined #openstack-ironic | 13:34 | |
| lazy_prince | Haomeng: not sure.. let me explain what i am trying to do. I am trying to implement network isolation using vlan on physical switch. for this, i need to have a dedicated provisioning network. | 13:35 |
| NobodyCam | good morning Ironic | 13:35 |
| Haomeng | NobodyCam: morning:) | 13:36 |
| NobodyCam | morning Haomeng :) | 13:36 |
| lazy_prince | Haomeng: so when deploy() is called, i am changing the node network to point to provisioning network. however, dhcp options are not getting associated with the provisioning network.. | 13:36 |
| lazy_prince | Haomeng: so wanted ot see if i can do it via code.. | 13:37 |
| Haomeng | NobodyCam: :) | 13:37 |
| Haomeng | lazy_prince: let me check with neutron guys tomorrow, is ok? | 13:38 |
| Haomeng | lazy_prince: need neutron expert supporting£º£© | 13:39 |
| lazy_prince | Haomeng: i do not thisnk its a bug in neutron.. but its something i am doing programetically.. and i want to see if we can get neutron port id for the node in ironic.. | 13:40 |
| Haomeng | lazy_prince: yes, it is not bug, but we need to know from neutron to find the way to get port_id | 13:41 |
| Haomeng | lazy_prince: what is the context? what is the input? | 13:41 |
| Haomeng | lazy_prince: dhcp port ? | 13:41 |
| lazy_prince | Haomeng: since the code will execute from ironic, i think node or task could be the input.. | 13:42 |
| Haomeng | lazy_prince: ok, the nova instance id is the input I think | 13:42 |
| lazy_prince | Haomeng: yup.. you could say that.. | 13:42 |
| Haomeng | lazy_prince: I know one neutron expert from China, will check with him tomorrow morning, maybe I can help you | 13:43 |
| lazy_prince | Haomeng: please check and thanks.. | 13:44 |
| *** derekh has joined #openstack-ironic | 13:46 | |
| *** romcheg has quit IRC | 13:49 | |
| openstackgerrit | Vladyslav Drok proposed a change to openstack/ironic: Use standard locale in list_partitions https://review.openstack.org/122414 | 13:50 |
| Haomeng | lazy_prince: ok, good night:) | 13:50 |
| lazy_prince | Haomeng: gn. c ya tomorrow.. | 13:51 |
| openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/ironic: Updated from global requirements https://review.openstack.org/122423 | 13:59 |
| openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/ironic-python-agent: Updated from global requirements https://review.openstack.org/114058 | 13:59 |
| devananda | morning, all | 14:07 |
| *** datajerk has quit IRC | 14:09 | |
| NobodyCam | good morning devananda | 14:11 |
| NobodyCam | your up eqrly | 14:11 |
| NobodyCam | early even | 14:12 |
| *** jistr has quit IRC | 14:15 | |
| *** ifarkas has quit IRC | 14:15 | |
| *** jistr has joined #openstack-ironic | 14:16 | |
| *** smoriya has quit IRC | 14:22 | |
| *** ramineni has joined #openstack-ironic | 14:22 | |
| lucasagomes | NobodyCam, devananda morning :) | 14:25 |
| NobodyCam | good mornig lucasagomes | 14:25 |
| linggao | Hi NobodyCam, will Ironic graduate from incubator in Juno? | 14:25 |
| *** romcheg has joined #openstack-ironic | 14:25 | |
| NobodyCam | good morning linggao :) | 14:26 |
| NobodyCam | we did | 14:26 |
| NobodyCam | !!!! | 14:26 |
| openstack | NobodyCam: Error: "!!!" is not a valid command. | 14:26 |
| NobodyCam | well for K | 14:26 |
| linggao | goog morning NobodyCam | 14:26 |
| linggao | cool! | 14:27 |
| linggao | NobodyCam, what do you mean by "well for k"? | 14:27 |
| NobodyCam | we did graduate! | 14:28 |
| NobodyCam | but in the kilo cycle | 14:28 |
| NobodyCam | linggao: https://review.openstack.org/#/c/120225/1 | 14:29 |
| devananda | we graduated | 14:29 |
| devananda | that means Ironic is integrated in the Kilo release cycle | 14:29 |
| linggao | devananda, so in Juno, Ironic will be official or in Kilo? | 14:29 |
| devananda | we were not part of the integrated gate tests or release process during the Juno cycle | 14:30 |
| * NobodyCam does another little happy dance | 14:30 | |
| romcheg | w00t | 14:30 |
| NobodyCam | mornign romcheg :) | 14:30 |
| romcheg | Hi everyone btw :) | 14:30 |
| *** gilliard has quit IRC | 14:30 | |
| devananda | linggao: ironic will be part of the _integrated_release_ during the Kilo cycle. it's been an "official" project for a while | 14:30 |
| NobodyCam | oh romcheg if you have a free minute can you take a look over https://wiki.openstack.org/wiki/Ironic/NovaBaremetalIronicMigration | 14:31 |
| lucasagomes | romcheg, hey ya, how it's going man? | 14:31 |
| linggao | devananda, thanks. | 14:31 |
| devananda | I hate that integration is seen as somehow making something official, btw | 14:32 |
| devananda | but that's another conversation :) | 14:33 |
| devananda | anteaya: g'morning! question on the extra-atcs things if you're around? | 14:34 |
| *** datajerk has joined #openstack-ironic | 14:34 | |
| anteaya | morning | 14:36 |
| anteaya | here I am | 14:36 |
| anteaya | ask away | 14:36 |
| devananda | anteaya: hi there! | 14:36 |
| anteaya | o/ | 14:36 |
| *** jistr has quit IRC | 14:37 | |
| romcheg | lucasagomes: Moved to Poland and finally managed to go to ikea and buy the necessary stuff. Now I can sleep and eat just normally :) | 14:37 |
| NobodyCam | morning anteaya | 14:37 |
| anteaya | morning NobodyCam | 14:37 |
| devananda | anteaya: since Ironic is not considered part of the integrated release in Kilo, it is, as I undersatnd it, therefor not part of the "core openstack projects" | 14:37 |
| NobodyCam | romcheg: awesome! | 14:37 |
| lucasagomes | romcheg, hah, nice good to hear that | 14:37 |
| lucasagomes | romcheg, I bet you may be eating a lot of pierogi these days :) | 14:38 |
| linggao | devananda,NobodyCam, sorry, I was in a meeting and someone asked me if Ironic will graduate in Juno cycle. | 14:38 |
| anteaya | devananda: ah for that answer we have a lovely canonnical reference: http://git.openstack.org/cgit/openstack/governance/tree/reference/programs.yaml | 14:38 |
| devananda | anteaya: that being said, is there any effect of providing extra-atc's list for Ironic? | 14:38 |
| NobodyCam | linggao: its all good. I luv to tell anyone we did graduate | 14:38 |
| anteaya | devananda: only if those people would not otherwise be able to vote in the ironic ptl election should two or more candidates put their name forward | 14:38 |
| linggao | devananda,NobodyCam, thanks for your responses. | 14:39 |
| *** jistr has joined #openstack-ironic | 14:39 | |
| anteaya | devananda: and I do believe next Tuesday will be the last assurance of getting names in, so if there are names to get in err on the side of including them | 14:39 |
| devananda | linggao: no worries - and I apologize if that seemed snarky -- it was not directed at you, but at my general frustration with the perception that integration == official/good/stable/etc | 14:39 |
| devananda | anteaya: ok. there are 3 names in co-authored-by only, but as far as I can tell, none of them have any commits to any other openstack/ projects nor have they joined the foundation | 14:40 |
| linggao | devananda, yes, that's exactly what my colleagues think. | 14:40 |
| romcheg | lucasagomes, NobodyCam: check out this nice view from my new office :) http://cl.ly/image/381k460B1A0F | 14:40 |
| anteaya | devananda: ah see the joining the foundation bit, it is important | 14:41 |
| anteaya | devananda: you have their emails can you email them and get them to join? | 14:41 |
| anteaya | linggao: please educate your collegues, or invite them to join the channel | 14:41 |
| devananda | linggao: for what it's worth, I do not believe that incubation and integration were ever intended to bestow the blessing of production-worthiness upon any project, and am working with others to help correct this. More to come in the next few months. | 14:42 |
| anteaya | linggao: there is a movement afoot to move to a model of layers so the integrated/incubated distinction may be a poor way to evaluate tool readiness in future | 14:42 |
| lucasagomes | romcheg, o wow great | 14:42 |
| anteaya | if it ever was a good way to evaluate | 14:43 |
| lucasagomes | romcheg, poznan looks quite small | 14:43 |
| romcheg | lucasagomes: Up to yesterday day I ate only sandwiches with cheese and ham because the only cutlery I had was my Swiss Army knife :) | 14:43 |
| anteaya | mostly it was just and easy way | 14:43 |
| lucasagomes | romcheg, lol | 14:43 |
| devananda | linggao: if you (or your colleague) are interested in that discussion, there was a long and insightful thread on the ML | 14:43 |
| anteaya | romcheg: I love my swiss army knife | 14:43 |
| devananda | anteaya: yes, the co-authors' emails are included inthe comit messages. I can email them, but I do not know if they will actually join | 14:44 |
| devananda | anteaya: as they are not people I have met or interacted with before | 14:44 |
| romcheg | NobodyCam: I will review that document later this evening. We have some kind of meetup here very soon | 14:45 |
| linggao | devananda, anteaya, when making product plans, they have to decide if they'll support Ironic or not in Juno time frame. So 'official' is the word they are looking for. I can see the bottle netck if how we define "official" :-) | 14:46 |
| anteaya | devananda: well that is kind of the point of the extra-atcs thing, you are vouching for someone else | 14:46 |
| anteaya | devananda: hard to do if you don't know them | 14:46 |
| NobodyCam | romcheg: Awesome thank you. its a wiki so please feel free to just correct anything you see wrong | 14:47 |
| romcheg | NobodyCam: will do :) | 14:47 |
| anteaya | devananda: you can start by emailing and letting them know that you would like to include them in the list of eligible voters and they can indicate their decisison to accept your offer by joining the foundation and emailing you the url to their profile | 14:47 |
| anteaya | devananda: if they don't do that by Tuesday morning, looks like they don't want to be included as an eligible voter | 14:47 |
| devananda | anteaya: ok, thanks! | 14:47 |
| anteaya | devananda: and you have done your duty by them | 14:48 |
| anteaya | linggao: we don't use official at all | 14:48 |
| anteaya | linggao: official is used by marketers and managers | 14:48 |
| anteaya | linggao: useful, working, funtional | 14:48 |
| anteaya | those are words we like to use | 14:48 |
| devananda | linggao: I do not believe that the OpenStack Foundation has any intention to suggest what any companies' product plans are or are not | 14:48 |
| anteaya | linggao: tested, there is a good word | 14:49 |
| devananda | linggao: when it comes to OpenStack projects, the Foundation really only gets involved if there is a trademark issue, and so being "part of the OpenStack Integrated Release" means that the trademark _may_ be applied | 14:49 |
| devananda | linggao: if a product is using the trademark, you can always add more things to your product, like incubated projects or your own projects | 14:50 |
| linggao | anteaya, guess who I was talking with :-) | 14:50 |
| anteaya | linggao: no idea | 14:50 |
| anteaya | linggao: marketing? | 14:51 |
| anteaya | a manager? | 14:51 |
| devananda | linggao: so whether or not a project is integrated actually should have no bearing on whether or not your product uses it -- it does effect whether or not your product may choose _not_ to include it, though | 14:51 |
| linggao | anteaya, hehe. | 14:51 |
| anteaya | :D | 14:52 |
| linggao | devananda, I know what you mean. We are on the same boat. :-) | 14:54 |
| openstackgerrit | Lucas Alvares Gomes proposed a change to openstack/ironic: Do not use the context parameter on refresh() https://review.openstack.org/122441 | 14:54 |
| lucasagomes | ^ with that we close the RPC objects bug :) (I think it should have been a bp really, it was a lot of work) | 14:55 |
| devananda | lucasagomes: \o/ | 14:55 |
| lucasagomes | devananda, I was looking at the objects/base.py code from nova, they introduced a lot of new stuff like each field in the object has a class representing it's type | 14:56 |
| lucasagomes | StringType() etc... | 14:56 |
| lucasagomes | we may need a bp to sync that in K, and maybe another bp in oslo to actually oslofy it | 14:56 |
| *** rushiagr is now known as rushiagr_away | 14:56 | |
| devananda | lucasagomes: yup. we probably want to propose that to the oslo or cross project tracks | 15:00 |
| devananda | lucasagomes: either you or comstud want to run with that? | 15:00 |
| lucasagomes | devananda, yeah, we can talk about it in paris (I saw you put it in the spreadsheet already, thanks) | 15:00 |
| lucasagomes | devananda, sure | 15:00 |
| devananda | lucasagomes: i mean, it shouldn't be in our track actually | 15:01 |
| lucasagomes | devananda, yeah it's cross-project | 15:01 |
| lucasagomes | but if we can't find a place maybe leaving in our track is fine, then we can try to drag people from nova like | 15:01 |
| *** pensu has joined #openstack-ironic | 15:01 | |
| lucasagomes | dan smith etc | 15:01 |
| *** igordcard has quit IRC | 15:02 | |
| *** igordcard has joined #openstack-ironic | 15:05 | |
| rloo | hi Ironickers. | 15:10 |
| rloo | NobodyCam: did you get a chance to look into support (or not) of nova baremetal-interface-list? [Or do we care?] Wondering if a bug should be opened for it or not. | 15:11 |
| NobodyCam | otp | 15:11 |
| rloo | NobodyCam: no worries. Stay OnTopofP...izzas if you want ;) | 15:13 |
| lucasagomes | rloo, morning :) | 15:15 |
| rloo | hi lucasagomes! | 15:15 |
| dtantsur | morning to everyone whose appearance I missed :) | 15:18 |
| rloo | hi dtantsur ;) | 15:18 |
| dtantsur | romcheg, congratulations with relocation! visit Brno one day ;) | 15:18 |
| romcheg | dtantsur: I will! | 15:20 |
| *** openstackgerrit has quit IRC | 15:33 | |
| *** athomas has quit IRC | 15:35 | |
| *** bluex-pl has quit IRC | 15:35 | |
| *** dlaube has joined #openstack-ironic | 15:45 | |
| *** eghobo has joined #openstack-ironic | 15:52 | |
| *** eghobo has quit IRC | 16:03 | |
| rloo | hey lucasagomes: did you find out if a bug is needed or not? https://review.openstack.org/#/c/122380/ | 16:03 |
| *** openstackgerrit has joined #openstack-ironic | 16:03 | |
| lucasagomes | rloo, oh no... devananda ^ | 16:03 |
| lucasagomes | sorry forgot to ask deva about it | 16:03 |
| lucasagomes | I think it's grand to merge, but I don't wanna break the rules | 16:03 |
| rloo | lucasagomes: I'm not sure I was aware of the rule so it would be good to know ;) | 16:04 |
| *** bluex-pl has joined #openstack-ironic | 16:05 | |
| lucasagomes | rloo, well, I remember we said that only code fixing a bug which is target to RC should be merged while we are in the RC period | 16:05 |
| * lucasagomes maybe is confusing things here | 16:06 | |
| rloo | lucasagomes: that rings a bell. I might have even been the one that asked... | 16:06 |
| lucasagomes | yeah idk, I'm not sure and a bit tired to go over the logs to try to find out :/ | 16:07 |
| *** ndipanov is now known as ndipanov_gone | 16:08 | |
| NobodyCam | brb | 16:09 |
| rloo | lucasagomes: well, it would be good to clarify that. in the meantime, this is only a cleanup in unit tests ;) | 16:10 |
| lucasagomes | yeah | 16:10 |
| devananda | hm? | 16:10 |
| lucasagomes | devananda, can we merge code which is not fixing a bug ? | 16:11 |
| lucasagomes | I think I may have heard it somewhere that we should not merge anything that is not fixing a bug targeting RC in this RC period | 16:11 |
| devananda | lucasagomes: that's a general rule for anything after feature freeze | 16:11 |
| lucasagomes | right, even if it's just a tiny little thing? | 16:12 |
| *** marcoemorais has joined #openstack-ironic | 16:12 | |
| devananda | lucasagomes: we don't necessarily need to adhere to process just for the sake of process, though | 16:12 |
| lucasagomes | https://review.openstack.org/#/c/122380/ | 16:12 |
| devananda | a simple patch which is obviosuly fixing a bug, even if that bug isn't on LP, is probably ok to merge | 16:12 |
| devananda | that said | 16:12 |
| lucasagomes | right, well... if there's a process I would say we should adhere to that | 16:12 |
| devananda | i am not sure that's really a bug | 16:12 |
| rloo | lucasagomes, devananda: then we get into the question of what's a 'bug' :-( | 16:13 |
| rloo | ie, is a bug anything that is reported in the Bug thing. Or is a bug anything that breaks our code. Or breaks a unit test. | 16:14 |
| devananda | lucasagomes: ok, i'm sure - that's a bug :) | 16:14 |
| devananda | ironic/tests/db/utils:get_test_port does indeed just ignore node_uuid arg, but doesn't do any enforcement of unexpected kwargs | 16:14 |
| lucasagomes | yeah, well, it doesn't break anything but it's inputting invalid parameters | 16:14 |
| devananda | because really it shouldn't have to | 16:14 |
| lucasagomes | so I think it's a bug as well... but yeah | 16:15 |
| devananda | yep. I'd classify this as code cleanup | 16:15 |
| lucasagomes | so +1 to merge it now | 16:15 |
| devananda | yea | 16:15 |
| * lucasagomes is going to push the merge button in 5... | 16:16 | |
| * devananda approves | 16:16 | |
| lucasagomes | oh | 16:16 |
| lucasagomes | lol | 16:16 |
| lucasagomes | ok | 16:16 |
| rloo | :D | 16:16 |
| devananda | I see that we don't have any Critical bugs open :) | 16:17 |
| devananda | there are a few High and lots of medium open & targeted to RC1 | 16:17 |
| lucasagomes | I think we have one that Jim is working on no? | 16:18 |
| lucasagomes | was that fixed? /me checks | 16:18 |
| rloo | lucasagomes: yeah, that's fixed | 16:18 |
| lucasagomes | nice | 16:19 |
| devananda | hmm | 16:19 |
| * devananda notices https://review.openstack.org/107629 | 16:19 | |
| lucasagomes | oh yeah fix commited https://bugs.launchpad.net/ironic/+bug/1368984 | 16:19 |
| devananda | lucasagomes: dtantsur: so _not_ running migrations within dbsync is actually really bad | 16:19 |
| devananda | there is a reason why other projects do that, and why we did it too | 16:19 |
| devananda | Nova learned from experience that, sometimes, the model and the migrations don't result in exactly the same thing | 16:20 |
| devananda | they certainly didn't in the past | 16:20 |
| dtantsur | don't we have tests exactly to ensure it? | 16:20 |
| devananda | so unless we actually have those tests... | 16:20 |
| dtantsur | viktors|afk added them IIRC | 16:20 |
| devananda | ok, lemme check. last time i looked they werent running in the gate though | 16:20 |
| *** jistr has quit IRC | 16:21 | |
| lucasagomes | well we had some conversation with viktors about that patch | 16:21 |
| lucasagomes | and as dtantsur the tests to capture problems etc | 16:21 |
| * lucasagomes is not a db expert but that sounded fine | 16:21 | |
| lucasagomes | afair | 16:21 |
| devananda | cool | 16:21 |
| dtantsur | it's ModelsMigrationsSyncMysql in ironic/tests/db/sqlalchemy/test_migrations.py IIRC | 16:21 |
| devananda | it's still a bit alarming to me when I see significant chagnes in the commit log that I didn't even know someone was working on :) | 16:22 |
| lucasagomes | I figured :) | 16:23 |
| lucasagomes | alright folks, I'll call it a day | 16:44 |
| lucasagomes | have a good night :) | 16:44 |
| rloo | bye lucasagomes | 16:44 |
| lucasagomes | bye bye :) | 16:45 |
| *** lucasagomes has quit IRC | 16:45 | |
| *** derekh has quit IRC | 16:47 | |
| *** bluex-pl has quit IRC | 16:49 | |
| *** bluex-pl has joined #openstack-ironic | 16:50 | |
| *** bluex-pl has quit IRC | 16:50 | |
| *** bluex-pl has joined #openstack-ironic | 16:51 | |
| *** ramineni has quit IRC | 16:51 | |
| *** romcheg has quit IRC | 16:53 | |
| *** bluex-pl has quit IRC | 16:56 | |
| *** bluex-pl has joined #openstack-ironic | 16:56 | |
| *** bluex-pl has quit IRC | 16:57 | |
| *** bluex-pl has joined #openstack-ironic | 16:58 | |
| *** bluex-pl has quit IRC | 16:59 | |
| *** bluex-pl has joined #openstack-ironic | 16:59 | |
| *** vdrok_ has joined #openstack-ironic | 17:00 | |
| *** igordcard has quit IRC | 17:01 | |
| *** harlowja_away is now known as harlowja | 17:02 | |
| *** Nisha has joined #openstack-ironic | 17:04 | |
| *** bluex-pl has quit IRC | 17:05 | |
| NobodyCam | night lucas | 17:12 |
| comstud | devananda: yeah, so i wanted to sync what lucas was talking about... by getting it into oslo | 17:15 |
| devananda | comstud: ++ | 17:15 |
| vdrok_ | morning/afternoon/evening Ironic! | 17:17 |
| openstackgerrit | A change was merged to openstack/ironic: node_uuid should not be used to create test port https://review.openstack.org/122380 | 17:20 |
| rloo | hi vdrok_ :) | 17:21 |
| vdrok_ | hi rloo! | 17:21 |
| *** dhellmann is now known as dhellmann_ | 17:22 | |
| vdrok_ | I'm fixing this bug - https://bugs.launchpad.net/ironic/+bug/1337551 . But I can't get the last point - Ensure lint runs against the script when tests are run (even thought there are no unit tests for this script) . How should this be done? | 17:25 |
| vdrok_ | should something be added to tox.ini? | 17:25 |
| jroll | good morning all :) | 17:27 |
| vdrok_ | morning jroll | 17:28 |
| rloo | morning jroll | 17:28 |
| *** lazy_prince is now known as killer_prince | 17:28 | |
| *** openstackgerrit has quit IRC | 17:31 | |
| *** openstackgerrit has joined #openstack-ironic | 17:33 | |
| rloo | jroll: if you feel like it, I touched your agent, and wasn't able to put you down as a reviewer: https://review.openstack.org/#/c/122224/ | 17:34 |
| jroll | yeah, my gerrit account is weird | 17:34 |
| * jroll will look | 17:34 | |
| rloo | jroll: I was given two choices (looked like the same name to me) and I got an error selecting either of them. You just want to avoid being a reviewer ;) | 17:35 |
| jroll | NobodyCam: what ever happened to soft power | 17:35 |
| *** igordcard has joined #openstack-ironic | 17:40 | |
| *** pelix has quit IRC | 17:41 | |
| *** dtantsur is now known as dtantsur|afk | 17:45 | |
| *** rushiagr_away is now known as rushiagr | 17:46 | |
| openstackgerrit | Ruby Loo proposed a change to openstack/ironic: Handle SNMP exception error.PySnmpError https://review.openstack.org/122182 | 17:48 |
| openstackgerrit | Ruby Loo proposed a change to openstack/ironic: Catch IronicExceptions in node_power_action() https://review.openstack.org/122476 | 17:48 |
| JoshNang | vdrok_: yup, you'd add a command under testenv:pep8 to flake that file/directory | 17:49 |
| vdrok_ | JoshNang: oh, thanks, thought pylint should be added there somehow | 17:52 |
| JoshNang | vdrok_: i think by lint he meant flake8 (which is a pep8 compliance checker) | 17:54 |
| vdrok_ | JoshNang: yup, now I see, thanks for the clarification! | 17:54 |
| JoshNang | vdrok_: np! | 17:54 |
| openstackgerrit | David J Hu proposed a change to openstack/python-ironicclient: Add keystone v3 CLI support https://review.openstack.org/111175 | 17:55 |
| *** rushiagr is now known as rushiagr_away | 18:03 | |
| *** vdrok_ has quit IRC | 18:11 | |
| *** rushiagr_away is now known as rushiagr | 18:15 | |
| *** romcheg has joined #openstack-ironic | 18:15 | |
| *** rushiagr is now known as rushiagr_away | 18:20 | |
| NobodyCam | hey jroll it fell off my plate | 18:20 |
| *** rushiagr_away is now known as rushiagr | 18:20 | |
| *** marcoemorais has quit IRC | 18:24 | |
| *** marcoemorais has joined #openstack-ironic | 18:24 | |
| openstackgerrit | David J Hu proposed a change to openstack/python-ironicclient: Add keystone v3 CLI support https://review.openstack.org/111175 | 18:26 |
| *** romcheg has quit IRC | 18:29 | |
| *** rushiagr is now known as rushiagr_away | 18:40 | |
| *** rushiagr_away is now known as rushiagr | 18:49 | |
| * devananda pushes a new client release | 18:51 | |
| rloo | thx devananda. Do you think it would be useful to have release notes posted somewhere? eg glanceclient: http://docs.openstack.org/developer/python-glanceclient/#release-notes | 18:54 |
| devananda | rloo: sure. also they are in the release itself | 18:55 |
| rloo | devananda: OTOH, I have no idea how/if novaclient does it. | 18:56 |
| devananda | rloo: http://git.openstack.org/cgit/openstack/python-ironicclient/tag/?id=0.3.0 | 18:56 |
| rloo | devananda: yeah, but that isn't obvious to folks is it? | 18:56 |
| * NobodyCam has to run out and pick up dog food for lunch :O | 18:56 | |
| *** annegentle_ has joined #openstack-ironic | 18:56 | |
| NobodyCam | I will bbaif | 18:57 |
| devananda | rloo: feel free to put that somewhere you feel is better :) | 18:57 |
| *** annegentle_ is now known as Guest62135 | 18:57 | |
| devananda | rloo: also, keep in mind the cost of maintaining it the next time we tag a cleint release, if we're just copying text around | 18:57 |
| devananda | I don't see any benefit to having an in-tree list of the commits which are in the tree, which is basically what glanceclient seems to have done | 18:58 |
| rloo | devananda: I think a pointer to the link should suffice. I'm not big on copying things. | 18:58 |
| devananda | i maen, i can run "git tag" and "git tag -v {TAG}" | 18:58 |
| *** rushiagr is now known as rushiagr_away | 18:58 | |
| *** yjiang5 has joined #openstack-ironic | 18:59 | |
| devananda | to see what the package maintainer thought was worth mentioning i nthe release notes | 18:59 |
| rloo | devananda: I think http://docs.openstack.org/developer/python-glanceclient/#release-notes is good. I wonder if there is some way to get that to show up at https://pypi.python.org/pypi/python-ironicclient | 19:00 |
| devananda | rloo: I do not think http://docs.openstack.org/developer/python-glanceclient/#release-notes is a good thing | 19:01 |
| *** dhellmann_ is now known as dhellmann | 19:01 | |
| *** dhellmann is now known as dhellmann_ | 19:01 | |
| rloo | devananda: why? (out of curiosity). | 19:01 |
| devananda | 18:58:15 < devananda> I don't see any benefit to having an in-tree list of the commits which are in the tree, which is basically what glanceclient seems to have done | 19:02 |
| devananda | that document lives inside the code base which it is documenting | 19:02 |
| rloo | gotcha. | 19:02 |
| *** dhellmann_ has quit IRC | 19:02 | |
| yjiang5 | hi, does anyone know why ironic requires docker.io in devstack? My 12.04 ubuntu seems has no docker.io package included. | 19:02 |
| *** dhellmann has joined #openstack-ironic | 19:02 | |
| devananda | yjiang5: i do not believe ironic requires that | 19:03 |
| yjiang5 | devananda: great, so I will workaround it. Thanks. | 19:03 |
| devananda | yjiang5: our requirements are http://git.openstack.org/cgit/openstack/ironic/tree/requirements.txt | 19:03 |
| devananda | yjiang5: also, as was said on the mailing list, it would be much better if you use 14.04 instead | 19:04 |
| *** marcoemorais has quit IRC | 19:04 | |
| devananda | there are several issues with ovs packages in 12.04 which we had to work around, which are much easier solved by simply using 14.04 | 19:04 |
| rloo | so i think the easiest thing to do (for next release) is to update the README to have something like 'Release notes: http://git.openstack.org/cgit/openstack/python-ironicclient/tag/?id=<we'd have to update this part each time> | 19:04 |
| *** marcoemorais has joined #openstack-ironic | 19:05 | |
| yjiang5 | devananda: sorry didn't notice that mail. Will upgrade to 14.04 then. For the docker.io, it's at https://github.com/openstack-dev/devstack/blob/master/files/apts/ironic | 19:06 |
| devananda | oh, i see | 19:06 |
| devananda | JayF: ^ that's you guys ... | 19:06 |
| devananda | JayF: AIUI you only need docker.io IF building the CoreOS image for IPA. So it's really not an requirement for ironic in the general sense. | 19:07 |
| devananda | yjiang5: you are correct. I didn't realize taht was there | 19:08 |
| yjiang5 | devananda: JayF: what's IPA? Sorry new to ironic. | 19:08 |
| devananda | yjiang5: ironic-python-agent. it's a separate deployment driver | 19:08 |
| yjiang5 | devananda: thanks. | 19:08 |
| rloo | devananda: we have some ManagementInterface methods that may not necessarily be supported (get_sensors_data, set_boot_device, set_boot_device). | 19:10 |
| *** yjiang5 is now known as yjiang5_away | 19:10 | |
| rloo | devananda: some driver implementations are raising NotImplementedError, and others aren't. Thoughts? | 19:10 |
| rloo | devananda: the NotImplementedError isn't an IronicException so that would mean drivers won't always be returning IronicE, but I'm fine with that. I'd just like it to be consistent. | 19:12 |
| devananda | rloo: when a client validates a node, the drivers may raise an error to indicate that interface is not supported/implemented (as opposed to simply being not configured) | 19:16 |
| devananda | rloo: I agree those should be consistent | 19:16 |
| devananda | rloo: and the exception is the way the driver communicates taht back to the conductor and API | 19:16 |
| devananda | also, I need to step away for a while | 19:16 |
| rloo | devananda: ok thx. | 19:16 |
| linggao | Hi all, what is the current scale we have tested for Ironic? | 19:17 |
| linggao | rloo, I heard someone mentioned that yahoo has a 1000-node cloud that uses Ironic, is it true? | 19:19 |
| rloo | linggao: yahoo isn't using Ironic yet; we're using nova baremetal. | 19:20 |
| devananda | linggao: rackspace is running ironic in production as part of the OnMetal product | 19:20 |
| devananda | linggao: but they have not publicly stated how many nodes are in that cluster | 19:20 |
| devananda | (not afaik, anyway) | 19:20 |
| * devananda really goes afk now | 19:21 | |
| rloo | linggao: maybe over a few beers one of the J's from rackspace will let you know ;) | 19:21 |
| linggao | lol | 19:21 |
| linggao | devananda, rloo, so maybe it was rackspace that scaled to 1000 nodes. | 19:22 |
| linggao | :) | 19:22 |
| linggao | jroll, can you comment? | 19:23 |
| JoshNang | has anyone looked at implementing suspend for nodes? | 19:27 |
| *** igordcard has quit IRC | 19:38 | |
| *** openstackgerrit has quit IRC | 19:45 | |
| *** killer_prince has quit IRC | 19:59 | |
| *** killer_prince has joined #openstack-ironic | 20:12 | |
| *** killer_prince is now known as lazy_prince | 20:12 | |
| JayF | linggao: I don't know what I can say or not say | 20:12 |
| JayF | linggao: but suffice it to say that I personally have reason to believe the agent driver can scale to almost a thousand nodes with 4 conductors, at least if you've applied a similar set of downstream patches as us (most of the perf stuff I think already made it back in) | 20:13 |
| *** harlowja has quit IRC | 20:13 | |
| *** harlowja has joined #openstack-ironic | 20:13 | |
| JayF | devananda: I think jroll was under the impression devstack is supposed to be run under trusty nowadays anyway, but imbw | 20:14 |
| linggao | JayF, thanks. beer? | 20:14 |
| JayF | linggao: I don't drink good beer anymore :) | 20:15 |
| linggao | :) | 20:15 |
| adam_g | devstack + ironic depends on stuff only in trusty, not only docker.io | 20:17 |
| rloo | adam_g: so we should update our docn: http://docs.openstack.org/developer/ironic/dev/dev-quickstart.html#deploying-ironic-with-devstack | 20:19 |
| *** yjiang5_away is now known as yjiang5 | 20:20 | |
| adam_g | rloo, yeah, i think that probably makes sense now. the precise runs dont happen in the gate anywhere anymore, so its just bitrotting | 20:22 |
| *** victor_lowther__ has quit IRC | 20:23 | |
| *** victor_lowther__ has joined #openstack-ironic | 20:24 | |
| rloo | adam_g: i'll open a bug about it. thx. | 20:27 |
| adam_g | rloo, https://review.openstack.org/#/c/122519/ :) | 20:31 |
| adam_g | is the bot asleep? | 20:32 |
| rloo | maybe taking a break? | 20:32 |
| rloo | adam_g: here's the bug: https://bugs.launchpad.net/ironic/+bug/1371300 | 20:33 |
| *** igordcard has joined #openstack-ironic | 20:35 | |
| adam_g | rloo, thanks | 20:36 |
| rloo | adam_g: thank You! Oh, I guess I should look at your patch ;) | 20:36 |
| *** igordcard has quit IRC | 20:40 | |
| *** marcoemorais has quit IRC | 20:41 | |
| * NobodyCam dislikes road work very much :-p | 20:41 | |
| *** marcoemorais has joined #openstack-ironic | 20:43 | |
| linggao | JayF, does IPA do RAID configuration today? | 20:52 |
| JayF | linggao: no, and I personally would consider in-band RAID management out of scope of Ironic and by extension IPA | 20:54 |
| linggao | JayF, okay, that's fair. thanks. | 20:55 |
| linggao | have to go.... | 20:55 |
| *** linggao has quit IRC | 20:55 | |
| *** pcrews has quit IRC | 20:59 | |
| *** pensu has quit IRC | 20:59 | |
| *** openstackgerrit has joined #openstack-ironic | 21:08 | |
| jroll | devananda: I've publicly said before that our initial deployment was ~1000 | 21:14 |
| * JayF has publically said he's paranoid and never remembers what was publically said | 21:14 | |
| jroll | and I believe all of our performance related patches have been upstreamed | 21:14 |
| JayF | :P | 21:14 |
| jroll | use the logs, luke | 21:14 |
| JayF | I don't want a cabin, Jim, I want answers! | 21:14 |
| jroll | :P | 21:15 |
| *** dprince has quit IRC | 21:25 | |
| *** jasondotstar has quit IRC | 21:25 | |
| *** greghaynes has quit IRC | 21:28 | |
| *** sbadia has quit IRC | 21:28 | |
| *** greghaynes has joined #openstack-ironic | 21:28 | |
| *** andreykurilin_ has joined #openstack-ironic | 21:30 | |
| *** sbadia has joined #openstack-ironic | 21:31 | |
| *** pcrews has joined #openstack-ironic | 21:38 | |
| *** marcoemorais has quit IRC | 21:43 | |
| *** marcoemorais has joined #openstack-ironic | 21:43 | |
| *** marcoemorais has quit IRC | 21:44 | |
| *** marcoemorais has joined #openstack-ironic | 21:44 | |
| *** marcoemorais has quit IRC | 21:44 | |
| *** marcoemorais has joined #openstack-ironic | 21:45 | |
| *** marcoemorais has quit IRC | 21:46 | |
| *** marcoemorais has joined #openstack-ironic | 21:47 | |
| *** marcoemorais has quit IRC | 21:47 | |
| *** marcoemorais has joined #openstack-ironic | 21:47 | |
| *** Nisha has quit IRC | 21:52 | |
| *** penick has joined #openstack-ironic | 21:53 | |
| NobodyCam | $#@-it Jim I'm an engineer not a technician | 21:54 |
| NobodyCam | oh wait that the wrong show. lol | 21:54 |
| jroll | lol | 21:54 |
| jroll | thought you were talking to me, I was wondering how I pissed you off :P | 21:55 |
| adam_g | devananda, https://review.openstack.org/#/c/122539/ your thoughts appreciated | 21:58 |
| openstackgerrit | Adam Gandelman proposed a change to openstack/ironic: Update devstack docs, require Ubuntu 14.04 https://review.openstack.org/122519 | 22:00 |
| NobodyCam | lol sorry nope | 22:04 |
| NobodyCam | just a joke | 22:04 |
| NobodyCam | :-p | 22:04 |
| NobodyCam | I saw the the use hte logs luke comment and was (trying) to play off that | 22:04 |
| rloo | hey, do we need this anymore: https://github.com/openstack/nova/blob/4ce3f55d169290015063131134f93fca236807ed/nova/tests/virt/test_ironic_api_contracts.py | 22:08 |
| JayF | That can be pulled when we vote on nova, right? | 22:09 |
| JayF | or is it OK since our driver gets unit test runs anyway | 22:09 |
| rloo | I hope our driver's unit tests are being run... | 22:10 |
| *** rloo is now known as rloo_afk | 22:12 | |
| *** penick has quit IRC | 22:28 | |
| *** penick has joined #openstack-ironic | 22:31 | |
| *** andreykurilin_ has quit IRC | 22:39 | |
| *** enterprisedc has quit IRC | 22:51 | |
| openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/ironic: Updated from global requirements https://review.openstack.org/122423 | 22:56 |
| openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/ironic-python-agent: Updated from global requirements https://review.openstack.org/114058 | 22:56 |
| *** penick has quit IRC | 22:59 | |
| *** Guest62135 is now known as annegentle | 23:13 | |
| *** annegentle is now known as annegentleznc | 23:13 | |
| *** annegentleznc is now known as annegentle | 23:14 | |
| *** marcoemorais has quit IRC | 23:14 | |
| *** marcoemorais has joined #openstack-ironic | 23:14 | |
| *** marcoemorais has quit IRC | 23:14 | |
| *** marcoemorais has joined #openstack-ironic | 23:14 | |
| NobodyCam | hummm seems ipmitool may be leaving nodes locked when it sets maintenance = True | 23:15 |
| JayF | NobodyCam: I'll check that behavior on our end real quick | 23:16 |
| JayF | NobodyCam: as I know we've had some boxes go into maint for that | 23:16 |
| NobodyCam | the issue I got reported was user entered BAD IP address | 23:17 |
| NobodyCam | fyi | 23:17 |
| jroll | we may have manually unlocked them on our end, though | 23:17 |
| NobodyCam | and thank you JayF :) | 23:17 |
| NobodyCam | jroll: just clear the field in the db? | 23:17 |
| jroll | yes | 23:17 |
| JayF | All the nodes we have in maint for the power status sync thing are unlocked | 23:17 |
| NobodyCam | hummm | 23:18 |
| *** marcoemorais1 has joined #openstack-ironic | 23:26 | |
| *** marcoemorais has quit IRC | 23:30 | |
| *** Haomeng|2 has joined #openstack-ironic | 23:46 | |
| *** Haomeng has quit IRC | 23:47 | |
| *** marcoemorais1 has quit IRC | 23:58 | |
| *** marcoemorais has joined #openstack-ironic | 23:58 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!