| *** kaa_ has joined #refstack | 00:38 | |
| *** chandankumar has quit IRC | 06:38 | |
| *** tosky has joined #refstack | 07:38 | |
| *** chandankumar has joined #refstack | 07:47 | |
| chandankumar | arxcruz: please update this review https://review.openstack.org/567820 | 07:52 |
|---|---|---|
| *** kopecmartin has joined #refstack | 07:53 | |
| *** kaa_ has quit IRC | 08:05 | |
| arxcruz | tosky: chandankumar i'm gettin problems because i did both reviews one depending the other, the download cirros image fix and load basic config, now the laod basic confi patch was merged and when i try a git review i got error | 08:08 |
| arxcruz | http://paste.openstack.org/show/721068/ | 08:09 |
| arxcruz | you guys know how to fix? | 08:09 |
| tosky | arxcruz: you did not properly rebase again origin/master | 08:11 |
| tosky | git review -d <mynewreview> | 08:11 |
| tosky | git rebase origin/master | 08:11 |
| tosky | git review | 08:11 |
| openstackgerrit | Arx Cruz proposed openstack/python-tempestconf master: Fixing download of the cirros image https://review.openstack.org/567820 | 08:13 |
| arxcruz | tosky: https://memegenerator.net/instance/53068365/not-bad-obama-well-done-give-that-man-a-medal | 08:14 |
| arxcruz | i was rebasing the previous patch, i should just git rebase --skip | 08:14 |
| tosky | oh | 08:16 |
| tosky | well, it's fixed now :) | 08:16 |
| tosky | let's wait for the gates and then rekick +w (we need to wait anyway for the 1st run) | 08:17 |
| openstackgerrit | Martin Kopec proposed openstack/python-tempestconf master: Make tempestconf easier to use as an library https://review.openstack.org/562672 | 08:22 |
| chandankumar | kopecmartin: hello | 08:26 |
| chandankumar | kopecmartin: can you please take care of this bug also https://storyboard.openstack.org/#!/story/2001784 | 08:26 |
| chandankumar | arxcruz: hey, Since you are working on compute coverage is it possible to include this one also https://bugzilla.redhat.com/show_bug.cgi?id=1569613 | 08:28 |
| openstack | bugzilla.redhat.com bug 1569613 in python-tempestconf "compute.min_compute_nodes not discovered by tempestconf" [Unspecified,New] - Assigned to chkumar | 08:28 |
| chandankumar | ? | 08:28 |
| chandankumar | arxcruz: if not then we can move to next sprint | 08:28 |
| arxcruz | chandankumar: sure, i'll take a look, fixing some other stuff on oooq | 08:28 |
| chandankumar | arxcruz: kopecmartin I will plan for next sprint in advance for tempestconf | 08:29 |
| tosky | arxcruz, chandankumar : if it's a real bug/feature request not just for RH systems, a story on storyboard would be nice too | 08:34 |
| chandankumar | arxcruz: Can i create a card for stackviz stuff also? | 08:34 |
| chandankumar | tosky: correct copying th stuff to storyboard | 08:34 |
| chandankumar | *the | 08:34 |
| arxcruz | chandankumar: what's happening with stackviz ? | 09:08 |
| arxcruz | chandankumar: the cards are for our sprint | 09:08 |
| arxcruz | shit man, packstack is always getting can't find compute node :( | 09:09 |
| arxcruz | can't test nova stuff | 09:09 |
| openstackgerrit | Chandan Kumar proposed openstack/python-tempestconf master: Allow users to upload images to glance https://review.openstack.org/568252 | 09:12 |
| kopecmartin | chandankumar, I took a look at https://storyboard.openstack.org/#!/story/2001784 and there is nothing to do with that right now, i commented there | 09:44 |
| kopecmartin | chandankumar, i would close it, but i have no idea how the whole storyboard works | 09:45 |
| chandankumar | kopecmartin: https://review.openstack.org/#/c/536121/ | 09:45 |
| chandankumar | kopecmartin: this will solve your problem | 09:45 |
| chandankumar | kopecmartin: for current implementation we can use try except till this patch lands | 09:46 |
| chandankumar | with a fix me or we can wait till this patch lands | 09:46 |
| kopecmartin | chandankumar, we can wait | 09:47 |
| kopecmartin | kopecmartin, if we wait it's only one patch, if we implement it now, we will require another patch to remove v2 .. | 09:47 |
| openstackgerrit | Arx Cruz proposed openstack/python-tempestconf master: Fixing download of the cirros image https://review.openstack.org/567820 | 09:47 |
| chandankumar | kopecmartin: so from rocky v3 will be default | 09:49 |
| chandankumar | on glance side, I think image api v1 will be removed | 09:50 |
| arxcruz | so... ntp... | 09:52 |
| arxcruz | the big vilain | 09:52 |
| arxcruz | if you not set a ntp compute node even if it's all in one will not get registered | 09:52 |
| arxcruz | (╯°□°）╯︵ ┻━┻ | 09:52 |
| tosky | arxcruz: try to work with SSL stuff without ntp, and good luck | 09:53 |
| tosky | kopecmartin: re storyboard, is the question about the usage of Story: and Task:, or about the workflow for "closing" stories? | 09:59 |
| kopecmartin | tosky, i find it anything but not intuitive | 10:00 |
| kopecmartin | tosky, but I'll read a doc or somthing and figure it out | 10:00 |
| openstackgerrit | Martin Kopec proposed openstack/python-tempestconf master: Make tempestconf easier to use as an library https://review.openstack.org/562672 | 10:02 |
| tosky | kopecmartin: you work on the tasks, and they have states | 10:05 |
| tosky | kopecmartin: you can close automatically a certain task using (instead of Closes-Bug:) footer like Story: and Task: in the commit message | 10:05 |
| tosky | but yes, strongly suggested to read the documentation a bit | 10:05 |
| kopecmartin | tosky, what if a task or a whole story was resolved but there is no review for that? let's say it was resolved a different way than by a review | 10:08 |
| tosky | kopecmartin: you can manually change the status of the corresponding task | 10:08 |
| tosky | check it - there is a listbox for each task | 10:08 |
| tosky | a story is not active anymore when all the tasks (which may be just one) have a final state | 10:09 |
| tosky | apart from the fact that a story can have more tasks for the same project (repository), that's not much different than launchpad | 10:09 |
| kopecmartin | there are options like review, merged or invalid , so if there is no review, i can't choose review or merged, so only option is invalid :D | 10:10 |
| kopecmartin | but i get the rest | 10:10 |
| tosky | do you have an example of something that does not fit in those categories? | 10:11 |
| tosky | if the issue is fixed with some other fixes in other projects, then there should be a task for the other project | 10:11 |
| tosky | what I think it's missing is marking a story as duplicate | 10:12 |
| arxcruz | great, now i can't ssh into vm | 10:13 |
| * tosky bbl | 10:13 | |
| arxcruz | tosky: anything i need to do? security group are enabled properly | 10:13 |
| tosky | arxcruz: is console access available? | 12:00 |
| arxcruz | tosky: just get rid of everything and reinstalling again | 12:01 |
| tosky | ouch | 12:04 |
| arxcruz | tosky: have you ever install it on rdocloud ? | 12:05 |
| arxcruz | in a rdocloud vm i mean | 12:05 |
| tosky | I don't have an account there | 12:05 |
| openstackgerrit | Martin Kopec proposed openstack/python-tempestconf master: Use volume v3 service client as the default https://review.openstack.org/568827 | 12:37 |
| chandankumar | tosky: I need some help related to swift healthcheck api | 13:13 |
| kopecmartin | chandankumar, tosky when i hit `openstack service list` i can see for example volume, volumev2, volumev3 .. my question is, when we are gathering for example api_extensions for volume service, all of the extensions from volumev2 and v3 are going under volume-feature-enabled section? | 13:13 |
| chandankumar | tosky: first thing In order to make sure healthcheck api is available we need to hit swift service <public endpoint>/healthcheck na? | 13:14 |
| arxcruz | chandankumar: tosky to volume encryption set | 13:17 |
| arxcruz | can i just check if barbican is enabled and set to true? otherwise set to false? | 13:17 |
| chandankumar | arxcruz: I think barbican is enabled and a role creator is there then i think we can set it to true | 13:18 |
| arxcruz | chandankumar: role creator is just for non admin be able to create encrypted volume | 13:18 |
| arxcruz | tosky: https://review.openstack.org/#/c/567820/ +w ? | 13:28 |
| tosky | arxcruz: I was rechecking it | 13:31 |
| arxcruz | (╯°□°）╯︵ ┻━┻ | 13:31 |
| tosky | arxcruz: but if the directory exists, what happens? Isn't OSError raised? | 13:31 |
| arxcruz | tosky: no because it checks if directory exists first :) | 13:31 |
| tosky | oh, right | 13:32 |
| tosky | I probably even checked it before | 13:32 |
| tosky | luckily they fixed this complicated behavior in python3 | 13:32 |
| tosky | "exist_ok=True yes please, I don't care, no need to write tons of lines" | 13:33 |
| arxcruz | hehehehe | 13:33 |
| arxcruz | so, please, +w | 13:33 |
| arxcruz | gotta go to home now | 13:33 |
| arxcruz | so, the vm, cirros isn't getting dhcp, dont know why | 13:33 |
| tosky | cirros 0.3.4? 0.3.5? 0.4.0? | 13:34 |
| arxcruz | and then i can't test the volume attach, because attach the volume is returning is attached | 13:34 |
| arxcruz | but inside cirros isn't attaching | 13:34 |
| arxcruz | no idea let me check | 13:34 |
| arxcruz | 0.3.5 | 13:35 |
| arxcruz | if i set manually using virsh console and ip link bla bla bla | 13:35 |
| arxcruz | i'm able to connect through ssh with ip netns exec router | 13:35 |
| arxcruz | but that's a freaking workaround :/ | 13:35 |
| *** rmart04 has joined #refstack | 13:57 | |
| openstackgerrit | Martin Kopec proposed openstack/python-tempestconf master: Filter deprecated versions out https://review.openstack.org/568695 | 14:33 |
| openstackgerrit | Chandan Kumar proposed openstack/python-tempestconf master: Use healthcheck api to determine swift service https://review.openstack.org/568869 | 14:55 |
| chandankumar | arxcruz: kopecmartin tosky ^ Please have a look at this one | 14:56 |
| tosky | chandankumar: why the function is called "def check_object_storage_service_status(self):"? | 15:03 |
| tosky | can't we have a generic check_service_status in Service, inherited and reimplemented by each member? | 15:03 |
| tosky | this was exactly the reason for a generic base class for services: cover the common operations | 15:03 |
| openstackgerrit | Martin Kopec proposed openstack/python-tempestconf master: Store volumev2 and v3 extensions as well https://review.openstack.org/568876 | 15:04 |
| chandankumar | tosky: that would be doable, I will do it for swift | 15:08 |
| chandankumar | tosky: arxcruz kopecmartin regarding this one https://review.openstack.org/#/c/568252/ any feedback or comments | 15:09 |
| kopecmartin | chandankumar, I'm trying to understand what it does and why | 15:10 |
| tosky | chandankumar: I'm still unconvinced by the commit message vs the content | 15:11 |
| tosky | also, isn't catching Exception a bit broad? Isn't there a specific exception? | 15:11 |
| tosky | oh, now it's image only, not for flavors | 15:12 |
| kopecmartin | chandankumar, so you got rid of allow_creation and the exception informing that creation is not allowed -> it should be fine, It's the same like a user who doesn't have permissions to create an image but will pass --create (a user lied :D ) | 15:29 |
| kopecmartin | chandankumar, but i can't find, why is this code better, what does it solve? | 15:29 |
| chandankumar | kopecmartin: tosky In openstack deployment, an normal user can upload the image to glance but flavor is restricted | 15:34 |
| chandankumar | kopecmartin: tosky --create is only permitted when an user has admin related power | 15:35 |
| chandankumar | kopecmartin: tosky so for normal user, they need to ask admin user to create flavor and image, but above change will allow user to pass the image and use the falvor, tempestconf will take care of uploading the imahe and generarting imahe id | 15:37 |
| kopecmartin | i see, so the point here is to make it more OpenStack CLI like | 15:37 |
| chandankumar | for worse case, if an user does not have permission to even upload the image then that exception will be called | 15:38 |
| kopecmartin | chandankumar, if it's true what you said, they didn't have to as admin to upload the image there, they can do it via OpenStack CLI .. but i see the point ... if it's able via OpenStack CLI why not via python-tempetsconf | 15:39 |
| kopecmartin | *it's possible | 15:40 |
| chandankumar | kopecmartin: that is what i was intending to do | 15:42 |
| kopecmartin | chandankumar, great, get it .. i commented the change | 15:43 |
| *** rmart04 has quit IRC | 15:52 | |
| openstackgerrit | Chandan Kumar proposed openstack/python-tempestconf master: Allow non admin users to upload image to glance https://review.openstack.org/568252 | 15:53 |
| chandankumar | kopecmartin: tosky ^^ | 15:54 |
| tosky | chandankumar: isn't there a more specific exception than Exception? | 16:16 |
| openstackgerrit | Merged openstack/python-tempestconf master: Fixing download of the cirros image https://review.openstack.org/567820 | 16:17 |
| chandankumar | tosky: need help on that | 16:30 |
| chandankumar | tosky: kopecmartin I need to send one more patch for swift operator role | 16:31 |
| chandankumar | the implementation I think is something like that: | 16:32 |
| chandankumar | check for overrides values, if role, present set it, if nothing passed, get the default role from tempest.config and check for it, if not present create it if --create is passed and set the role to that user | 16:33 |
| chandankumar | the same goes for reseller one | 16:33 |
| chandankumar | what do you think? | 16:33 |
| * chandankumar headed home | 16:36 | |
| *** kopecmartin has quit IRC | 16:36 | |
| tosky | only if you have admin, of course | 17:12 |
| *** tosky has quit IRC | 19:20 | |
| *** tosky has joined #refstack | 19:24 | |
| *** jschlueter has quit IRC | 22:17 | |
| *** andreaf has quit IRC | 22:29 | |
| *** andreaf has joined #refstack | 22:29 | |
| *** jschlueter has joined #refstack | 22:34 | |
| *** jschlueter has quit IRC | 23:18 | |
| *** jschlueter has joined #refstack | 23:23 | |
| *** tosky has quit IRC | 23:34 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!