*** sgrasley_ has joined #openstack-helm | 00:30 | |
*** sgrasley has quit IRC | 00:33 | |
*** sthussey has quit IRC | 00:40 | |
*** jklare has quit IRC | 01:08 | |
*** jklare has joined #openstack-helm | 01:14 | |
openstackgerrit | Matthew Heler proposed openstack/openstack-helm-infra master: [WIP] [CEPH] Support a troubleshooting option to reset PG metadata https://review.openstack.org/626001 | 01:16 |
---|---|---|
openstackgerrit | Mike Pham proposed openstack/openstack-helm master: WIP: Implement East-West EGRESS policies for the Cloud Core services https://review.openstack.org/623315 | 01:43 |
*** jklare has quit IRC | 01:49 | |
*** jklare has joined #openstack-helm | 01:58 | |
*** cfriesen has quit IRC | 02:12 | |
*** sgrasley__ has joined #openstack-helm | 02:21 | |
*** sgrasley_ has quit IRC | 02:24 | |
*** sgrasley__ has quit IRC | 03:03 | |
*** unicell has joined #openstack-helm | 03:05 | |
*** unicell has quit IRC | 03:10 | |
openstackgerrit | Matthew Heler proposed openstack/openstack-helm-infra master: [CEPH] Journal automation race fix https://review.openstack.org/626090 | 04:42 |
openstackgerrit | Cliff Parsons proposed openstack/openstack-helm master: Implement east-west nw security policy for Cinder https://review.openstack.org/614873 | 05:00 |
*** evrardjp_ has joined #openstack-helm | 05:04 | |
*** evrardjp has quit IRC | 05:07 | |
*** krypto has joined #openstack-helm | 05:07 | |
openstackgerrit | Matthew Heler proposed openstack/openstack-helm-infra master: [WIP] [CEPH] Support a troubleshooting option to reset PG metadata https://review.openstack.org/626001 | 05:08 |
openstackgerrit | Matthew Heler proposed openstack/openstack-helm-infra master: [CEPH] Support a troubleshooting option to reset PG metadata https://review.openstack.org/626001 | 05:10 |
openstackgerrit | Matthew Heler proposed openstack/openstack-helm-infra master: [DNM] [CEPH] Support a troubleshooting option to reset PG metadata https://review.openstack.org/626001 | 05:25 |
openstackgerrit | Matthew Heler proposed openstack/openstack-helm-infra master: [CEPH] Support a troubleshooting option to reset PG metadata https://review.openstack.org/626001 | 05:26 |
openstackgerrit | Matthew Heler proposed openstack/openstack-helm-infra master: [Ceph] Update rbd-provisioner and cephfs-provisioner https://review.openstack.org/625403 | 05:37 |
*** krypto has quit IRC | 05:52 | |
*** evrardjp has joined #openstack-helm | 06:00 | |
*** evrardjp_ has quit IRC | 06:03 | |
*** iniazi has quit IRC | 06:20 | |
*** dims has quit IRC | 06:27 | |
*** dims has joined #openstack-helm | 06:28 | |
*** dims has quit IRC | 06:33 | |
*** dims has joined #openstack-helm | 06:34 | |
*** unicell has joined #openstack-helm | 06:34 | |
*** unicell has left #openstack-helm | 06:35 | |
*** ianw is now known as ianw_pto | 06:43 | |
openstackgerrit | Yang Le proposed openstack/openstack-helm master: Remove duplicate cinder_task in rally yaml file. https://review.openstack.org/626101 | 07:10 |
openstackgerrit | Yang Le proposed openstack/openstack-helm master: Add the CentOS system to Ceph related configuration file. https://review.openstack.org/626105 | 07:19 |
openstackgerrit | Merged openstack/openstack-helm master: Enable keystone brute-force protection by default https://review.openstack.org/625687 | 07:24 |
evrardjp | folks, the first periodics for OSH images are there, and all green! Thanks everyone! | 07:53 |
*** arunkant has joined #openstack-helm | 08:04 | |
arunkant | evrardjp, hi | 08:04 |
evrardjp | portdirect: I am curious about this revert: https://review.openstack.org/#/c/616282/ -- why not having the two different behaviours conditionally triggered based on rally's version? | 08:04 |
evrardjp | arunkant: hello! | 08:04 |
evrardjp | arunkant: portdirect is in holiday, so I doubt he will be able to help right now, and most of the AT&T people are in US timezones. | 08:05 |
evrardjp | However I expect jayahn might have an opinion | 08:05 |
evrardjp | arunkant: for version mappings we can ask on #openstack-kolla | 08:06 |
*** gkadam has joined #openstack-helm | 08:11 | |
openstackgerrit | Jaesang Lee proposed openstack/openstack-helm master: Upgrade rally to 1.3.0 https://review.openstack.org/586783 | 08:55 |
*** georgk has joined #openstack-helm | 09:21 | |
jayahn | someone summoned me. :) | 09:24 |
jayahn | evrardjp: that was reverted because there were dependent ps (not merged) upgrading rally image to the right version | 09:25 |
jayahn | evrardjp: the purpose of that ps was actually fixing rally chart to use the right one, kolla rally image (orginally used in osh chart) was not working. | 09:26 |
*** aojea has joined #openstack-helm | 10:08 | |
*** nick_kar has quit IRC | 10:13 | |
*** nick_kar has joined #openstack-helm | 10:14 | |
openstackgerrit | Seungkyu Ahn proposed openstack/openstack-helm-infra master: Supporting the local time zone https://review.openstack.org/625803 | 10:21 |
evrardjp | arunkant: ^ | 10:32 |
evrardjp | jayahn: it's very interesting, as arunkant has the same issue. What did you do at the end then? | 10:33 |
evrardjp | I think if something is broken for you, it doesn't make sense to simply revert (I am glad mattmceuen wrote why the revert was done in a comment, but I'd have preferred seeing that in commit message, or better: continue with the fix) | 10:34 |
*** georgk has quit IRC | 11:10 | |
*** georgk has joined #openstack-helm | 11:37 | |
*** phuoc_ has joined #openstack-helm | 11:43 | |
*** phuoc has quit IRC | 11:47 | |
*** phuoc_ has quit IRC | 11:47 | |
*** phuoc_ has joined #openstack-helm | 11:47 | |
*** georgk has quit IRC | 11:53 | |
*** hogepodge has quit IRC | 12:29 | |
*** hogepodge has joined #openstack-helm | 12:30 | |
*** georgk has joined #openstack-helm | 12:38 | |
*** sthussey has joined #openstack-helm | 13:56 | |
openstackgerrit | Mike Pham proposed openstack/openstack-helm master: WIP: Implement East-West EGRESS policies for the Cloud Core services https://review.openstack.org/623315 | 14:22 |
openstackgerrit | Mohan V proposed openstack/openstack-helm-infra master: Do-not run nginx as root for Ingress helm chart https://review.openstack.org/622359 | 14:22 |
portdirect | evrardjp: on phone so will be breif | 14:29 |
portdirect | The reason for revert was twofold | 14:29 |
portdirect | Primarily due to the reason I list in the revert commit message | 14:30 |
portdirect | In that it removed the ability to declare users for use with rally | 14:30 |
portdirect | Making it impossible to use with ldap backed keystone domains | 14:31 |
portdirect | The other being that it was broken with the version of rally used by all helm tests ATM, which that script is the driver for | 14:31 |
portdirect | If it only one of the above had been the case, I'd have gone for a fix | 14:32 |
portdirect | But with both in play, revert made more sense | 14:32 |
evrardjp | I have no doubt that you took the decision that best applied to your situation :) | 14:47 |
portdirect | Just pointing out that the reason for revert was in the commit ;) | 14:48 |
evrardjp | I read that, but it's incomplete to me -- I don't see the reason why it was not possible with the image update | 14:49 |
evrardjp | but I am merely a messenger here | 14:49 |
evrardjp | I think we are in the middle of two use cases -- ppl wanting a more recent image, and people wanting to continue passing existing tests | 14:49 |
evrardjp | in those case there is no perfect approach but communication | 14:50 |
portdirect | Wait | 14:50 |
portdirect | Your missing the point | 14:50 |
evrardjp | probably :) | 14:50 |
portdirect | I'm 100% in favor of an updated image | 14:50 |
portdirect | But that ps did two things: | 14:50 |
portdirect | 1 update the image | 14:50 |
portdirect | 2 change the way users are managed in the tests | 14:51 |
portdirect | It was because of 2 that a revert was done, in addition to the face that 1 broke all tests as the image had not been updated. | 14:51 |
evrardjp | I guess I will see when I will dig deeper with arunkant | 14:52 |
portdirect | If either one of the above points were broken in isolation, I'd have suggested a fix | 14:52 |
evrardjp | arunkant: is currently testing the rally part and has trouble making it work | 14:52 |
portdirect | But with both, it seemed like the ps had been merged prematurely | 14:52 |
evrardjp | understood | 14:52 |
evrardjp | I like the fact that in the future we might have the depends-on work that will help on this | 14:53 |
portdirect | There was no reason it could not have been used then, it's always been there | 14:53 |
evrardjp | so images can be separately built, but also there is an explicit link between the work | 14:53 |
evrardjp | portdirect: that's true | 14:54 |
evrardjp | but people sometimes fail to discern and aren't forced to split work in two, which would be the case when different repos are on the table | 14:54 |
evrardjp | sometimes it will be worse though, so we'll see | 14:54 |
evrardjp | thanks for the clarifications | 14:55 |
evrardjp | arunkant: if you have questions, now would be the time | 14:55 |
openstackgerrit | jacky06 proposed openstack/openstack-helm-images master: Update the home-page to openstack-helm-images https://review.openstack.org/626184 | 14:58 |
openstackgerrit | jacky06 proposed openstack/openstack-helm-docs master: Update the home-page to openstack-helm-docs https://review.openstack.org/626186 | 15:00 |
*** jaypipes has quit IRC | 15:01 | |
*** cfriesen has joined #openstack-helm | 15:11 | |
*** jaypipes has joined #openstack-helm | 15:15 | |
*** cfriesen has quit IRC | 15:17 | |
openstackgerrit | Jean-Philippe Evrard proposed openstack/openstack-helm-images master: Publish images on DockerHub and Quay.io optionally https://review.openstack.org/624713 | 15:20 |
openstackgerrit | jacky06 proposed openstack/openstack-helm-images master: Update the Ubuntu to 18.04 https://review.openstack.org/626209 | 15:21 |
*** evgenyl has joined #openstack-helm | 15:26 | |
*** cfriesen has joined #openstack-helm | 15:42 | |
cfriesen | does the nova chart support multiple replicas for the "novncproxy" pod? | 15:43 |
*** georgk has quit IRC | 16:20 | |
*** levmorgan has joined #openstack-helm | 16:24 | |
*** levmorgan has quit IRC | 16:27 | |
*** jistr has quit IRC | 16:41 | |
*** jistr has joined #openstack-helm | 16:42 | |
*** jistr_ has joined #openstack-helm | 16:46 | |
*** jistr has quit IRC | 16:47 | |
*** jistr_ has quit IRC | 16:51 | |
*** jistr has joined #openstack-helm | 16:52 | |
openstackgerrit | Chris Wedgwood proposed openstack/openstack-helm-infra master: [Calico] Update to v3.2.4 https://review.openstack.org/626368 | 17:28 |
*** unicell has joined #openstack-helm | 17:40 | |
*** unicell has quit IRC | 17:43 | |
*** gkadam has quit IRC | 17:46 | |
openstackgerrit | Chris Wedgwood proposed openstack/openstack-helm-infra master: [Calico] Show objects before they are applied in calico-settings (job) https://review.openstack.org/626371 | 17:51 |
*** aojea has quit IRC | 17:52 | |
openstackgerrit | Mike Pham proposed openstack/openstack-helm master: WIP: Implement East-West EGRESS policies for the Cloud Core services https://review.openstack.org/623315 | 18:42 |
*** levmorgan has joined #openstack-helm | 19:12 | |
openstackgerrit | Mike Pham proposed openstack/openstack-helm master: WIP: Implement East-West EGRESS policies for the Cloud Core services https://review.openstack.org/623315 | 19:48 |
*** unicell has joined #openstack-helm | 19:51 | |
*** unicell has quit IRC | 20:42 | |
openstackgerrit | bk160f proposed openstack/openstack-helm master: Keystone podSecurityContext https://review.openstack.org/614011 | 20:56 |
*** unicell has joined #openstack-helm | 22:34 | |
openstackgerrit | Roy Tang proposed openstack/openstack-helm master: WIP: Replace bad nulls that break node/label overrides https://review.openstack.org/626403 | 22:37 |
*** levmorgan has quit IRC | 22:38 | |
openstackgerrit | Steve Wilkerson proposed openstack/openstack-helm-infra master: WIP: Write libvirt logs to host https://review.openstack.org/626405 | 22:54 |
openstackgerrit | Craig Anderson proposed openstack/openstack-helm-infra master: Document node/nodelabel override capability https://review.openstack.org/574923 | 23:52 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!