Wednesday, 2018-12-19

*** sgrasley_ has joined #openstack-helm00:30
*** sgrasley has quit IRC00:33
*** sthussey has quit IRC00:40
*** jklare has quit IRC01:08
*** jklare has joined #openstack-helm01:14
openstackgerritMatthew Heler proposed openstack/openstack-helm-infra master: [WIP] [CEPH] Support a troubleshooting option to reset PG metadata  https://review.openstack.org/62600101:16
openstackgerritMike Pham proposed openstack/openstack-helm master: WIP: Implement East-West EGRESS policies for the Cloud Core services  https://review.openstack.org/62331501:43
*** jklare has quit IRC01:49
*** jklare has joined #openstack-helm01:58
*** cfriesen has quit IRC02:12
*** sgrasley__ has joined #openstack-helm02:21
*** sgrasley_ has quit IRC02:24
*** sgrasley__ has quit IRC03:03
*** unicell has joined #openstack-helm03:05
*** unicell has quit IRC03:10
openstackgerritMatthew Heler proposed openstack/openstack-helm-infra master: [CEPH] Journal automation race fix  https://review.openstack.org/62609004:42
openstackgerritCliff Parsons proposed openstack/openstack-helm master: Implement east-west nw security policy for Cinder  https://review.openstack.org/61487305:00
*** evrardjp_ has joined #openstack-helm05:04
*** evrardjp has quit IRC05:07
*** krypto has joined #openstack-helm05:07
openstackgerritMatthew Heler proposed openstack/openstack-helm-infra master: [WIP] [CEPH] Support a troubleshooting option to reset PG metadata  https://review.openstack.org/62600105:08
openstackgerritMatthew Heler proposed openstack/openstack-helm-infra master: [CEPH] Support a troubleshooting option to reset PG metadata  https://review.openstack.org/62600105:10
openstackgerritMatthew Heler proposed openstack/openstack-helm-infra master: [DNM] [CEPH] Support a troubleshooting option to reset PG metadata  https://review.openstack.org/62600105:25
openstackgerritMatthew Heler proposed openstack/openstack-helm-infra master: [CEPH] Support a troubleshooting option to reset PG metadata  https://review.openstack.org/62600105:26
openstackgerritMatthew Heler proposed openstack/openstack-helm-infra master: [Ceph] Update rbd-provisioner and cephfs-provisioner  https://review.openstack.org/62540305:37
*** krypto has quit IRC05:52
*** evrardjp has joined #openstack-helm06:00
*** evrardjp_ has quit IRC06:03
*** iniazi has quit IRC06:20
*** dims has quit IRC06:27
*** dims has joined #openstack-helm06:28
*** dims has quit IRC06:33
*** dims has joined #openstack-helm06:34
*** unicell has joined #openstack-helm06:34
*** unicell has left #openstack-helm06:35
*** ianw is now known as ianw_pto06:43
openstackgerritYang Le proposed openstack/openstack-helm master: Remove duplicate cinder_task in rally yaml file.  https://review.openstack.org/62610107:10
openstackgerritYang Le proposed openstack/openstack-helm master: Add the CentOS system to Ceph related configuration file.  https://review.openstack.org/62610507:19
openstackgerritMerged openstack/openstack-helm master: Enable keystone brute-force protection by default  https://review.openstack.org/62568707:24
evrardjpfolks, the first periodics for OSH images are there, and all green! Thanks everyone!07:53
*** arunkant has joined #openstack-helm08:04
arunkantevrardjp, hi08:04
evrardjpportdirect: 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
evrardjparunkant: hello!08:04
evrardjparunkant: 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
evrardjpHowever I expect jayahn might have an opinion08:05
evrardjparunkant: for version mappings we can ask on #openstack-kolla08:06
*** gkadam has joined #openstack-helm08:11
openstackgerritJaesang Lee proposed openstack/openstack-helm master: Upgrade rally to 1.3.0  https://review.openstack.org/58678308:55
*** georgk has joined #openstack-helm09:21
jayahnsomeone summoned me. :)09:24
jayahnevrardjp: that was reverted because there were dependent ps (not merged) upgrading rally image to the right version09:25
jayahnevrardjp: 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-helm10:08
*** nick_kar has quit IRC10:13
*** nick_kar has joined #openstack-helm10:14
openstackgerritSeungkyu Ahn proposed openstack/openstack-helm-infra master: Supporting the local time zone  https://review.openstack.org/62580310:21
evrardjparunkant: ^10:32
evrardjpjayahn: it's very interesting, as arunkant has the same issue. What did you do at the end then?10:33
evrardjpI 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 IRC11:10
*** georgk has joined #openstack-helm11:37
*** phuoc_ has joined #openstack-helm11:43
*** phuoc has quit IRC11:47
*** phuoc_ has quit IRC11:47
*** phuoc_ has joined #openstack-helm11:47
*** georgk has quit IRC11:53
*** hogepodge has quit IRC12:29
*** hogepodge has joined #openstack-helm12:30
*** georgk has joined #openstack-helm12:38
*** sthussey has joined #openstack-helm13:56
openstackgerritMike Pham proposed openstack/openstack-helm master: WIP: Implement East-West EGRESS policies for the Cloud Core services  https://review.openstack.org/62331514:22
openstackgerritMohan V proposed openstack/openstack-helm-infra master: Do-not run nginx as root for Ingress helm chart  https://review.openstack.org/62235914:22
portdirectevrardjp: on phone so will be breif14:29
portdirectThe reason for revert was twofold14:29
portdirectPrimarily due to the reason I list in the revert commit message14:30
portdirectIn that it removed the ability to declare users for use with rally14:30
portdirectMaking it impossible to use with ldap backed keystone domains14:31
portdirectThe other being that it was broken with the version of rally used by all helm tests ATM, which that script is the driver for14:31
portdirectIf it only one of the above had been the case, I'd have gone for a fix14:32
portdirectBut with both in play, revert made more sense14:32
evrardjpI have no doubt that you took the decision that best applied to your situation :)14:47
portdirectJust pointing out that the reason for revert was in the commit ;)14:48
evrardjpI read that, but it's incomplete to me -- I don't see the reason why it was not possible with the image update14:49
evrardjpbut I am merely a messenger here14:49
evrardjpI think we are in the middle of two use cases -- ppl wanting a more recent image, and people wanting to continue passing existing tests14:49
evrardjpin those case there is no perfect approach but communication14:50
portdirectWait14:50
portdirectYour missing the point14:50
evrardjpprobably :)14:50
portdirectI'm 100% in favor of an updated image14:50
portdirectBut that ps did two things:14:50
portdirect1 update the image14:50
portdirect2 change the way users are managed in the tests14:51
portdirectIt 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
evrardjpI guess I will see when I will dig deeper with arunkant14:52
portdirectIf either one of the above points were broken in isolation, I'd have suggested a fix14:52
evrardjparunkant: is currently testing the rally part and has trouble making it work14:52
portdirectBut with both, it seemed like the ps had been merged prematurely14:52
evrardjpunderstood14:52
evrardjpI like the fact that in the future we might have the depends-on work that will help on this14:53
portdirectThere was no reason it could not have been used then, it's always been there14:53
evrardjpso images can be separately built, but also there is an explicit link between the work14:53
evrardjpportdirect: that's true14:54
evrardjpbut 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 table14:54
evrardjpsometimes it will be worse though, so we'll see14:54
evrardjpthanks for the clarifications14:55
evrardjparunkant: if you have questions, now would be the time14:55
openstackgerritjacky06 proposed openstack/openstack-helm-images master: Update the home-page to openstack-helm-images  https://review.openstack.org/62618414:58
openstackgerritjacky06 proposed openstack/openstack-helm-docs master: Update the home-page to openstack-helm-docs  https://review.openstack.org/62618615:00
*** jaypipes has quit IRC15:01
*** cfriesen has joined #openstack-helm15:11
*** jaypipes has joined #openstack-helm15:15
*** cfriesen has quit IRC15:17
openstackgerritJean-Philippe Evrard proposed openstack/openstack-helm-images master: Publish images on DockerHub and Quay.io optionally  https://review.openstack.org/62471315:20
openstackgerritjacky06 proposed openstack/openstack-helm-images master: Update the Ubuntu to 18.04  https://review.openstack.org/62620915:21
*** evgenyl has joined #openstack-helm15:26
*** cfriesen has joined #openstack-helm15:42
cfriesendoes the nova chart support multiple replicas for the "novncproxy" pod?15:43
*** georgk has quit IRC16:20
*** levmorgan has joined #openstack-helm16:24
*** levmorgan has quit IRC16:27
*** jistr has quit IRC16:41
*** jistr has joined #openstack-helm16:42
*** jistr_ has joined #openstack-helm16:46
*** jistr has quit IRC16:47
*** jistr_ has quit IRC16:51
*** jistr has joined #openstack-helm16:52
openstackgerritChris Wedgwood proposed openstack/openstack-helm-infra master: [Calico] Update to v3.2.4  https://review.openstack.org/62636817:28
*** unicell has joined #openstack-helm17:40
*** unicell has quit IRC17:43
*** gkadam has quit IRC17:46
openstackgerritChris Wedgwood proposed openstack/openstack-helm-infra master: [Calico] Show objects before they are applied in calico-settings (job)  https://review.openstack.org/62637117:51
*** aojea has quit IRC17:52
openstackgerritMike Pham proposed openstack/openstack-helm master: WIP: Implement East-West EGRESS policies for the Cloud Core services  https://review.openstack.org/62331518:42
*** levmorgan has joined #openstack-helm19:12
openstackgerritMike Pham proposed openstack/openstack-helm master: WIP: Implement East-West EGRESS policies for the Cloud Core services  https://review.openstack.org/62331519:48
*** unicell has joined #openstack-helm19:51
*** unicell has quit IRC20:42
openstackgerritbk160f proposed openstack/openstack-helm master: Keystone podSecurityContext  https://review.openstack.org/61401120:56
*** unicell has joined #openstack-helm22:34
openstackgerritRoy Tang proposed openstack/openstack-helm master: WIP: Replace bad nulls that break node/label overrides  https://review.openstack.org/62640322:37
*** levmorgan has quit IRC22:38
openstackgerritSteve Wilkerson proposed openstack/openstack-helm-infra master: WIP: Write libvirt logs to host  https://review.openstack.org/62640522:54
openstackgerritCraig Anderson proposed openstack/openstack-helm-infra master: Document node/nodelabel override capability  https://review.openstack.org/57492323:52

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