Wednesday, 2019-11-27

*** openstackgerrit has joined #openstack-horizon00:18
openstackgerritTin Lam proposed openstack/horizon master: Add policy_rules to flavors dashboard  https://review.opendev.org/69483700:18
*** btully has joined #openstack-horizon01:21
*** btully has quit IRC01:25
*** adriant has quit IRC01:39
*** spsurya has joined #openstack-horizon01:40
*** cloudnull has joined #openstack-horizon01:58
*** adriant has joined #openstack-horizon02:00
*** chenhaw has quit IRC03:00
*** btully has joined #openstack-horizon03:11
*** btully has quit IRC03:15
*** spatel has joined #openstack-horizon05:43
*** spatel has quit IRC05:47
*** threestrands has quit IRC05:51
*** chenhaw has joined #openstack-horizon06:27
*** pcaruana has joined #openstack-horizon06:34
*** jtomasek has joined #openstack-horizon06:39
*** btully has joined #openstack-horizon06:50
*** btully has quit IRC06:54
*** e0ne has quit IRC07:10
*** tmazur has joined #openstack-horizon07:51
*** tesseract has joined #openstack-horizon08:04
*** awalende has joined #openstack-horizon08:16
*** tkajinam has quit IRC08:21
wdoekesbrtknr: I don't get a discovery failure either (I'm using v3 keystone as backend). I do get a 403 on /project/ though (no openstack.services.compute perms I guess)08:22
wdoekes(I did get all dependencies the 17th of Oct. can't say if anything has changed in that regard (probably not))08:22
*** rdopiera has joined #openstack-horizon08:26
*** tosky has joined #openstack-horizon08:30
*** btully has joined #openstack-horizon08:38
*** btully has quit IRC08:42
*** rpittau|afk is now known as rpittau08:57
*** mkowalski has quit IRC09:27
openstackgerritOpenStack Proposal Bot proposed openstack/horizon master: Imported Translations from Zanata  https://review.opendev.org/69313909:30
*** openstackstatus has quit IRC09:30
*** e0ne has joined #openstack-horizon09:44
*** ChanServ sets mode: +o e0ne09:44
*** mkowalski has joined #openstack-horizon10:23
*** btully has joined #openstack-horizon10:27
*** rcernin has quit IRC10:31
*** btully has quit IRC10:31
*** rcernin has joined #openstack-horizon10:46
*** tkajinam has joined #openstack-horizon10:50
*** jtomasek has quit IRC11:23
*** jtomasek has joined #openstack-horizon11:25
*** rcernin has quit IRC11:31
*** nearyo has joined #openstack-horizon11:47
*** tkajinam has quit IRC12:11
*** tkajinam has joined #openstack-horizon12:15
*** tkajinam has quit IRC12:16
openstackgerritDmitriy Rabotyagov (noonedeadpunk) proposed openstack/horizon master: Add support to get disk_formats from glance  https://review.opendev.org/69587712:45
openstackgerritDmitriy Rabotyagov (noonedeadpunk) proposed openstack/horizon master: Add support to get disk_formats from glance  https://review.opendev.org/69587712:48
*** rdopiera has quit IRC12:51
wdoekesvishalmanchanda: your dev onboarding could use some work. getting trivial patches in has been a frustrating experience thusfar12:54
vishalmanchandawdoekes: what do you mean?13:03
*** rdopiera has joined #openstack-horizon13:09
openstackgerritDmitriy Rabotyagov (noonedeadpunk) proposed openstack/horizon master: Add support to get disk_formats from glance  https://review.opendev.org/69587713:14
openstackgerritMerged openstack/horizon master: Fix users integration test  https://review.opendev.org/68862613:36
*** btully has joined #openstack-horizon14:05
*** btully has quit IRC14:09
*** priteau has joined #openstack-horizon14:50
openstackgerritRadomir Dopieralski proposed openstack/horizon master: Fix use of ngettext in resgitry getName  https://review.opendev.org/69632115:09
*** awalende has quit IRC15:19
*** awalende has joined #openstack-horizon15:25
*** awalende has quit IRC15:29
*** hoonetorg has quit IRC15:51
wdoekesI mean that, after taking the effort of setting up git-review+account and making clean single-issue patches with a self-documenting commit message, instead of getting a "thanks for these fixes", I've been getting (a) failed build pipelines, (b) -2's to cherry-pick to master first, (c) requests to add (undocumented) "cherry-pick" messages, (d) and now (from you) requests to file bug reports15:54
wdoekesfirst.15:54
wdoekesI know it's not your fault. but it frustrates making the effort of contributing15:54
vishalmanchandawdoekes: ok i will update your PS and report a bug for the same.15:58
vishalmanchandawdoekes: I am not happy to give -1 but this is the process.15:58
vishalmanchandaif i will not give someone-else give you -1 for the same reason.15:59
amotokiwdoekes: regarding stable backports (b) and (c), they are covered in https://docs.openstack.org/project-team-guide/stable-branches.html16:04
*** hoonetorg has joined #openstack-horizon16:04
amotokiwdoekes: the stable policy requests us to merge a fix in the master branch first and then cherry-pick it.16:04
amotokiregrading "cherry-picked" line, we suggest to use "git cherry-pick -x $master_commit_id"16:05
amotokiit automatically adds the cherry-picked line.16:05
*** awalende has joined #openstack-horizon16:05
*** awalende has quit IRC16:10
*** nearyo has quit IRC16:12
amotokiwdoekes: rearding the bug report, is it unfair to request a bug report? we usually use issue tracker for detail bug details rather than to use a commit message to explain bug details.16:13
amotokiwdoekes: it might be different from your personal dev workflow, but if we use our-own preferred way it can be a mess...16:13
wdoekesamotoki: I'm sure it is documented. the whole openstack stuff isn't lacking in documentation (generally a good thing).16:36
wdoekesbut in this case, it feels like the amount of bureaucracy (= reading that has to be done, before being able to file a PR) is disproportionate to the amount of time that went into the fix.16:36
wdoekesthat's what I mean by "onboarding": getting new contributors up and running.16:36
wdoekes(and believe me, I know what's it like from your perspective: "why doesn't he just do one PR per feature?", "why didn't he write a sane commit message?", "why didn't he describe what the commit does, or why?", "why can't he just PEP8 the code beforehand?")16:36
wdoekesI'm sure you have a good reason for wanting bug reports. but I got no such requests for the other reviews, so this was the proverbial drop.16:36
wdoekes</vent>16:36
wdoekesand, vishalmanchanda: thanks!16:36
amotokiwdoekes: I totally understand your points and they are reasonable. I also feel same when I join a new dev community.16:39
openstackgerritRadomir Dopieralski proposed openstack/horizon master: Fix use of ngettext in resgitry getName  https://review.opendev.org/69632116:54
*** iurygregory has joined #openstack-horizon16:55
iurygregoryHello eveyrone, we are dropping py27 on ironic-ui but we are facing some problems with nodejs10-npm-run-test job https://review.opendev.org/#/c/695801/16:56
iurygregorydoes anyone has thoughts on how to solve this?16:57
vishalmanchandaiurygregory: ok i will help you to fix this.16:57
vishalmanchandaiurygregory: give me some time.16:57
iurygregoryvishalmanchanda, thanks =)16:58
vishalmanchandanp I think i have fix the same issue in designate-dashboard.16:59
iurygregorycool!17:00
vishalmanchandaiurygregory: If you don't mind Can i update patch set for this https://review.opendev.org/#/c/695801/17:00
iurygregoryone of the ironic cores said that maybe the job is running under py27 also17:01
vishalmanchandayeah right17:01
iurygregoryvishalmanchanda, you can push a change without problem if you want =)17:01
vishalmanchandaiurygregory: ok.17:01
*** tosky has quit IRC17:03
vishalmanchandaiurygregory: I have updated the PS..17:06
vishalmanchandaLet's see if it works.17:06
iurygregoryvishalmanchanda, tyvm! =)17:06
*** rdopiera has quit IRC17:50
*** tesseract has quit IRC17:53
*** david-lyle has joined #openstack-horizon17:54
*** ChanServ sets mode: +o david-lyle17:54
*** dklyle has quit IRC17:55
*** priteau has quit IRC17:55
*** e0ne has quit IRC18:02
*** rpittau is now known as rpittau|afk18:14
*** gouthamr has quit IRC18:20
*** gouthamr has joined #openstack-horizon18:21
*** tmazur has quit IRC18:49
*** e0ne has joined #openstack-horizon19:20
*** ChanServ sets mode: +o e0ne19:20
*** rdopiera has joined #openstack-horizon19:21
openstackgerritRadomir Dopieralski proposed openstack/horizon master: Fix use of ngettext in resgitry getName  https://review.opendev.org/69632119:23
*** spsurya has quit IRC19:30
*** btully has joined #openstack-horizon19:31
*** rdopiera has quit IRC19:34
*** btully has quit IRC19:35
*** e0ne has quit IRC19:36
*** e0ne has joined #openstack-horizon19:38
*** ChanServ sets mode: +o e0ne19:38
*** e0ne has quit IRC19:45
*** rdopiera has joined #openstack-horizon19:51
*** david-lyle has quit IRC20:04
*** dklyle has joined #openstack-horizon20:16
*** ChanServ sets mode: +o dklyle20:16
*** iurygregory has quit IRC20:48
*** e0ne has joined #openstack-horizon21:43
*** ChanServ sets mode: +o e0ne21:43
*** e0ne has quit IRC21:46
*** tsmith2 has quit IRC21:50
*** rcernin has joined #openstack-horizon21:56
*** pcaruana has quit IRC22:06

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