Thursday, 2019-09-26

*** aaronsheffield has quit IRC00:10
*** igordc has joined #openstack-helm00:44
*** stevthedev has joined #openstack-helm00:44
*** kranthikirang has joined #openstack-helm00:53
*** kranthikirang has quit IRC00:58
*** nicolasbock has quit IRC01:03
*** Liang__ has joined #openstack-helm01:14
*** Liang__ has left #openstack-helm01:15
*** roman_g has quit IRC02:33
*** jamesgu_ has quit IRC02:38
*** jamesgu has quit IRC02:38
*** kranthikirang has joined #openstack-helm02:42
*** kranthikirang has quit IRC02:46
*** cfriesen has quit IRC02:47
*** igordc has quit IRC03:18
*** igordc has joined #openstack-helm03:18
*** kranthikirang has joined #openstack-helm03:37
openstackgerritAndrii Ostapenko proposed openstack/openstack-helm master: Use service domain for Keystone test  https://review.opendev.org/68492103:45
*** kranthikirang has quit IRC03:52
*** markvoelker has quit IRC04:08
*** markvoelker has joined #openstack-helm05:05
*** dpawlik has joined #openstack-helm05:06
*** markvoelker has quit IRC05:11
*** kranthikirang has joined #openstack-helm05:25
*** kranthikirang has quit IRC05:29
*** roman_g has joined #openstack-helm05:42
*** roman_g has quit IRC06:00
*** vsaienk0 has joined #openstack-helm06:12
*** kranthikirang has joined #openstack-helm06:20
*** vsaienk0 has quit IRC06:22
*** pgaxatte has joined #openstack-helm06:23
*** kranthikirang has quit IRC06:24
*** vsaienk0 has joined #openstack-helm06:31
*** rchurch has quit IRC06:43
*** dmsimard has quit IRC06:44
*** jsuchome has joined #openstack-helm06:45
*** igordc has quit IRC06:53
*** dmsimard has joined #openstack-helm06:58
*** pgaxatte has quit IRC07:14
*** pgaxatte has joined #openstack-helm07:16
*** witek has joined #openstack-helm07:38
*** kranthikirang has joined #openstack-helm07:49
*** kranthikirang has quit IRC07:53
*** rchurch has joined #openstack-helm07:59
zhipeng[m]Hi all08:03
zhipeng[m]I have a rabbitmq issue08:03
zhipeng[m]After I upgrade openstack-helm-infra recently.08:04
zhipeng[m]keystone db sync failed due to RROR oslo.messaging._drivers.impl_rabbit [-] Connection failed: failed to resolve broker hostname08:04
zhipeng[m]I saw below code part removed in latest code of rabbitmq/values.yaml08:05
* zhipeng[m] sent a long message: < https://matrix.org/_matrix/media/v1/download/matrix.org/sazUaHLzaaEUQsdenXNpCJoM >08:05
* zhipeng[m] sent a long message: < https://matrix.org/_matrix/media/v1/download/matrix.org/OYTvetNNDqfzbsxAZmKEAtuU >08:06
zhipeng[m]In rabbitmq/templates/statefulset.yaml,   dicovery changed to internal in below part08:10
* zhipeng[m] sent a long message: < https://matrix.org/_matrix/media/v1/download/matrix.org/jGFrTbJbIHfUWflbnzFlYFlD >08:10
zhipeng[m]@por08:11
zhipeng[m] *08:11
zhipeng[m]portdirect: can you help?  I saw your comments in above code, thanks!08:11
itxakazhipeng[m]: we also encountered that, we had to resort to setting statefulset to null to keep the old naming, but this is deploying with airship AND with only 1 replica of rabbitmq08:17
itxakaour old hostname was: keystone-rabbitmq.openstack.svc.cluster.local but with this patch it changed to rabbitmq-rabbitmq-0.keystone-rabbitmq.openstack.svc.cluster.local08:17
itxakayou can probably tweak the endpoints.oslo_messaging in values.yaml to fit it to your deployment08:18
* zhipeng[m] sent a long message: < https://matrix.org/_matrix/media/v1/download/matrix.org/DWKYYrQzBzolWhqVftXSTeoK >08:19
zhipeng[m]this is the old one08:19
itxakathe patch that introduced this was  https://github.com/openstack/openstack-helm/commit/9bcf0df94c1a8455e17aa79f20efaf2736eed0f408:19
zhipeng[m]from my side, and  name is  rabbit@osh-openstack-rabbitmq-rabbitmq-0.osh-openstac-dsv-e50215.openstack.svc.cluster.loca08:19
zhipeng[m]now, it change to   rabbit@osh-openstack-rabbitmq-rabbitmq-0.rabbitmq.openstack.svc.cluster.local08:19
itxakacheck what do you have in endpoints.oslo_messaging.statefulset08:21
evrardjpitxaka it seems this change is breaking upgrades, yet there are no release notes ?08:21
itxakaside effct of not having releases I guess08:21
itxakayou dont have 2 nice points to check upgradability08:21
itxakazhipeng[m]: also check endpoints.oslo_messaging.hosts.default08:22
itxakabetween that and endpoints.oslo_messaging.statefulset the hostname is created08:22
zhipeng[m]recently, I'm upgrading both openstack-helm and openstack-helm-infra to latest commit version08:23
itxakaso from  endpoints.oslo_messaging.statefulset.name it comes the "rabbitmq-rabbitmq-X" part08:23
itxakaand from endpoints.oslo_messaging.hosts.default the other extra "rabbitmq"08:24
zhipeng[m]I have no this section08:24
zhipeng[m]endpoints.oslo_messaging.statefulset08:24
itxakathat config is done in the components08:25
itxakanot in the rabbitmq chart08:25
zhipeng[m]below is the old one08:26
* zhipeng[m] sent a long message: < https://matrix.org/_matrix/media/v1/download/matrix.org/oLlmkjlGAzlbLByHjCVUXjaE >08:26
zhipeng[m]you mean in rabbitmq code?08:26
itxakaIm well there is 2 things here, one is rabbitmq chart adn its creation of hostnames and one is the components creating thost hostnames to connect to rabbitmq08:29
itxakawhich one is failing here? the name creation in rabbit or the componentns not being able to connect to rabbit because they have the wrong hostname?08:30
itxakaI can help you with the second as I encountered that recently, for the first I cant as we are still using an older rabbitmq chart without the latest changes so no idea how that works :)08:30
zhipeng[m]OK, I guess it is second one08:32
zhipeng[m]btw if we want to upgrade openstack-helm, which commit version is prefered08:33
zhipeng[m]As I mentioned above, for our old version,  hostname  is08:35
zhipeng[m]osh-openstac-dsv-e50215.openstack.svc.cluster.loca08:35
zhipeng[m]now it was created as08:36
zhipeng[m]rabbitmq.openstack.svc.cluster.local08:36
zhipeng[m]then,  keystone db sync can not resolve it anymore08:37
zhipeng[m]From log, I can see rabbitmq started normally08:40
* zhipeng[m] sent a long message: < https://matrix.org/_matrix/media/v1/download/matrix.org/bRJdWMFngerJmKrfhgpywjCt >08:40
jsuchomeitxaka: once rabbitmq expert, always rabbitmq expert ....08:41
zhipeng[m]In keystone/values.yaml , it is below08:53
zhipeng[m] hosts:08:53
zhipeng[m]      default: keystone08:53
zhipeng[m]      internal: keystone-api08:53
zhipeng[m]in other component like nova is below08:55
* zhipeng[m] sent a long message: < https://matrix.org/_matrix/media/v1/download/matrix.org/FlQMvDQpnXNRobggSqiuRRXd >08:55
zhipeng[m]why we have no statefulset for keystone?08:56
zhipeng[m]sorry, missing it,  keystone has  statefulset as well09:02
zhipeng[m]portdirect:   Do I need to change rabbitmq-rabbitmq to rabbitmq  ?09:03
*** markvoelker has joined #openstack-helm09:08
*** markvoelker has quit IRC09:13
itxakammmh Im wondering where that osh-openstac-dsv-e50215 part in the hostname came from09:16
itxakamaybe portdirect has more info on that, sorry zhipeng[m]09:16
zhipeng[m]simply, so how can I change in keystone side to let it get name as09:16
zhipeng[m]rabbit@osh-openstack-rabbitmq-rabbitmq-0.rabbitmq.openstack.svc.cluster.local09:16
itxakaah, in keystone09:17
zhipeng[m]can you help on the second one :)09:17
zhipeng[m]above name I saw in rabbitmq log09:18
itxakaoverride the value  endpoints.oslo_messaging.statefulset.name to "osh-openstack-rabbitmq-rabbitmq"09:18
zhipeng[m]ok, I will try it, thanks09:19
itxakathat should make keystone try to connect to osh-openstack-rabbitmq-rabbitmq-0.rabbitmq.openstack.svc.cluster.local09:19
zhipeng[m]great!!!09:20
itxakaif that doesnt work let us know so other people can explain that better when they arrive (should be in around 3 hours I think)09:22
zhipeng[m]sure09:22
*** Emine has joined #openstack-helm09:24
*** bobmel has joined #openstack-helm09:37
*** kranthikirang has joined #openstack-helm09:37
zhipeng[m]itxaka: It works!09:41
*** roman_g has joined #openstack-helm09:41
itxakayay \o/09:41
zhipeng[m]thanks !!!09:41
itxakaIm glad I could help you :D09:41
*** kranthikirang has quit IRC09:42
*** bobmel has quit IRC09:49
pma78please review https://review.opendev.org/684197 (Handle neutron ML2 mechanism drivers)10:24
pma78please review https://review.opendev.org/684254 (Support neutron's baremetal mechanism driver)10:24
pma78please review https://review.opendev.org/683111 (Support override of ironic agent image)10:24
*** rezroo has quit IRC11:01
*** nicolasbock has joined #openstack-helm11:03
*** markvoelker has joined #openstack-helm11:17
*** markvoelker has quit IRC11:22
*** kranthikirang has joined #openstack-helm11:25
*** kranthikirang has quit IRC11:30
*** georgk5 has quit IRC11:57
*** rihabb has quit IRC11:57
*** georgk5 has joined #openstack-helm11:58
*** rihabb has joined #openstack-helm11:58
*** ianychoi has quit IRC12:00
*** ianychoi has joined #openstack-helm12:03
*** markvoelker has joined #openstack-helm12:07
*** Darcidride has joined #openstack-helm12:24
*** jsuchome has quit IRC12:29
*** bh526r has joined #openstack-helm12:54
*** Darcidride has quit IRC12:56
*** Darcidride has joined #openstack-helm12:56
*** vsaienk0 has quit IRC12:57
*** aaronsheffield has joined #openstack-helm13:04
*** dpawlik has quit IRC13:10
*** kranthikirang has joined #openstack-helm13:14
*** sthussey has joined #openstack-helm13:15
*** kranthikirang has quit IRC13:18
*** vsaienk0 has joined #openstack-helm13:28
*** dpawlik has joined #openstack-helm13:28
*** kranthikirang has joined #openstack-helm13:46
*** bobmel has joined #openstack-helm13:53
openstackgerritEvgeniy L proposed openstack/openstack-helm-infra master: Add RabbitMQ ingress Network Policy rules  https://review.opendev.org/68249013:57
zhipeng[m]Does latest build support Train?13:59
zhipeng[m]itxaka: However, this workaround is ugly, can we submit a patch to change it in values.yaml14:03
itxakazhipeng[m]: I think latests should support train, yes, I read that in the airshipit channel, havent tested it myself14:05
itxakaregarding the workaround, lets check with portdirect, as I would expect that in the latest version of osh and osh-infra the name should be automatically generated and would match, otherwise our jobs would fail14:06
itxakadid yoou upgrade an existing environment? Maybe thats the issue14:06
zhipeng[m]For openstack-helm upgrade, we have no release number, right?14:06
zhipeng[m]I just fetch the version according to latest commit id14:06
zhipeng[m]Do you have any proposal, thanks!14:06
*** nicolasbock has quit IRC14:06
itxakanope, you need to use another tool for that, we use airship with versions.yml to freeze the versions14:06
zhipeng[m]I saw last time, we fetch one openstack-helm according to one commit-id14:08
zhipeng[m]I'm not upgrading a existing env14:09
zhipeng[m]I come from StarlingX project team, like airship14:10
openstackgerritMerged openstack/openstack-helm master: [neutron] Handle ML2 mechanism drivers  https://review.opendev.org/68419714:13
openstackgerritMerged openstack/openstack-helm master: [neutron] Support baremetal mechanism driver  https://review.opendev.org/68425414:22
*** michael-beaver has joined #openstack-helm14:24
openstackgerritMerged openstack/openstack-helm-infra master: Mariadb: Run ingress error page server as nobody user  https://review.opendev.org/68480614:34
*** kranthikirang has quit IRC14:48
*** jamesgu has joined #openstack-helm15:05
*** jamesgu_ has joined #openstack-helm15:05
*** roman_g has quit IRC15:05
*** cfriesen has joined #openstack-helm15:11
*** vsaienk0 has quit IRC15:13
*** witek has quit IRC15:13
*** vsaienk0 has joined #openstack-helm15:17
*** vsaienk0 has quit IRC15:28
*** jamesgu has quit IRC15:28
*** jamesgu_ has quit IRC15:28
*** vsaienk0 has joined #openstack-helm15:30
*** dansmith has left #openstack-helm15:34
*** dansmith has joined #openstack-helm15:36
*** kranthikirang has joined #openstack-helm15:39
*** dpawlik has quit IRC15:43
*** dpawlik has joined #openstack-helm15:43
*** kranthikirang has quit IRC15:44
openstackgerritVasyl Saienko proposed openstack/openstack-helm master: [Glance] add job-metadefs-load  https://review.opendev.org/67996315:47
openstackgerritVasyl Saienko proposed openstack/openstack-helm master: [Glance] add job-metadefs-load  https://review.opendev.org/67996315:48
openstackgerritVasyl Saienko proposed openstack/openstack-helm-infra master: [helm-toolkit] add manifests.job_db_general  https://review.opendev.org/68147615:48
openstackgerritRADHIKA PAI proposed openstack/openstack-helm-infra master: Prometheus: Relabeling the node-exporter label  https://review.opendev.org/68511915:48
*** bh526r has quit IRC15:54
*** nicolasbock has joined #openstack-helm16:04
*** pgaxatte has quit IRC16:08
*** Emine has quit IRC16:11
*** igordc has joined #openstack-helm16:14
*** AlexNoskov has quit IRC16:28
*** kranthikirang has joined #openstack-helm16:29
*** markvoelker has quit IRC16:33
openstackgerritGage Hugo proposed openstack/openstack-helm-infra master: Define default netpol for openstack-exporter  https://review.opendev.org/68513016:35
*** markvoelker has joined #openstack-helm16:41
*** igordc has quit IRC16:43
*** Darcidride has quit IRC16:49
*** vsaienk0 has quit IRC16:53
openstackgerritGage Hugo proposed openstack/openstack-helm master: Add network policy nonvoting checks  https://review.opendev.org/68153916:57
*** itlinux has quit IRC17:04
*** vsaienk0 has joined #openstack-helm17:04
*** vsaienk0 has quit IRC17:14
openstackgerritPhil Sphicas proposed openstack/openstack-helm-infra master: helm-toolkit fix dependency resolver  https://review.opendev.org/68513617:23
openstackgerritPete Birley proposed openstack/openstack-helm master: Neutron: Update force purge of test project to be less aggressive  https://review.opendev.org/68513717:26
*** igordc has joined #openstack-helm17:28
openstackgerritPete Birley proposed openstack/openstack-helm master: Neutron: Update force purge of test project to be less aggressive  https://review.opendev.org/68513717:45
*** vsaienk0 has joined #openstack-helm17:54
*** dpawlik has quit IRC17:56
*** factor has joined #openstack-helm18:05
openstackgerritPete Birley proposed openstack/openstack-helm master: Neutron: Update force purge of test project to be less aggressive  https://review.opendev.org/68513718:06
openstackgerritchinasubbareddy mallavarapu proposed openstack/openstack-helm-infra master: [ceph-osd] Retry to create crush map for osd while ceph-mon service down for a while.  https://review.opendev.org/68514718:12
*** igordc has quit IRC18:29
openstackgerritPete Birley proposed openstack/openstack-helm master: Neutron: Ensure test project purge targets correct project  https://review.opendev.org/68515018:31
*** igordc has joined #openstack-helm18:32
*** vsaienk0 has quit IRC18:38
openstackgerritEvgeniy L proposed openstack/openstack-helm-infra master: Add RabbitMQ ingress Network Policy rules  https://review.opendev.org/68249018:52
openstackgerritEvgeniy L proposed openstack/openstack-helm-infra master: Add RabbitMQ ingress Network Policy rules  https://review.opendev.org/68249018:58
openstackgerritEvgeniy L proposed openstack/openstack-helm-infra master: Add RabbitMQ ingress Network Policy rules  https://review.opendev.org/68249019:02
openstackgerritchinasubbareddy mallavarapu proposed openstack/openstack-helm-infra master: [ceph-osd] Retry to create crush map for osd while ceph-mon service down for a while.  https://review.opendev.org/68514719:08
openstackgerritchinasubbareddy mallavarapu proposed openstack/openstack-helm-infra master: [ceph-osd] Retry to create crush map for osd while ceph-mon service down for a while.  https://review.opendev.org/68514719:25
openstackgerritEvgeniy L proposed openstack/openstack-helm-infra master: Add RabbitMQ ingress Network Policy rules  https://review.opendev.org/68249019:30
openstackgerritMerged openstack/openstack-helm master: Keystone: allow probes configuration  https://review.opendev.org/68322019:30
*** cfriesen has quit IRC19:38
*** vsaienk0 has joined #openstack-helm19:41
openstackgerritMerged openstack/openstack-helm master: Neutron: Update force purge of test project to be less aggressive  https://review.opendev.org/68513719:55
*** vsaienk0 has quit IRC20:01
openstackgerritSteven Fitzpatrick proposed openstack/openstack-helm-infra master: [WIP] Add mountpoints to node-exporter  https://review.opendev.org/68517220:09
*** kranthikirang has quit IRC20:21
*** kranthikirang has joined #openstack-helm20:22
*** kranthikirang has quit IRC20:27
openstackgerritSteven Fitzpatrick proposed openstack/openstack-helm-infra master: [WIP] Add extensible volume mounts to node-exporter  https://review.opendev.org/68517220:29
openstackgerritSteven Fitzpatrick proposed openstack/openstack-helm-infra master: [WIP] Add extensible volume mounts to node-exporter  https://review.opendev.org/68517220:30
openstackgerritMerged openstack/openstack-helm master: Neutron: Ensure test project purge targets correct project  https://review.opendev.org/68515020:34
openstackgerritMerged openstack/openstack-helm master: Don't use quotes in glance swift configuration  https://review.opendev.org/67578920:54
*** markvoelker has quit IRC21:34
*** kranthikirang has joined #openstack-helm21:35
*** rezroo has joined #openstack-helm21:38
*** kranthikirang has quit IRC21:40
*** rezroo has quit IRC21:44
*** michael-beaver has quit IRC21:50
*** markvoelker has joined #openstack-helm21:57
openstackgerritMerged openstack/openstack-helm-infra master: Grafana:Updated ES unassigned shards query  https://review.opendev.org/68322322:04
*** michael-beaver has joined #openstack-helm22:23
*** markvoelker has quit IRC22:27
*** jamesgu has joined #openstack-helm22:53
openstackgerritStas Egorov proposed openstack/openstack-helm-infra master: Fixed calico chart for hyperkube 1.12  https://review.opendev.org/68518623:11
*** aaronsheffield has quit IRC23:21
*** kranthikirang has joined #openstack-helm23:24
openstackgerritStas Egorov proposed openstack/openstack-helm-infra master: [TMP]: for testing  https://review.opendev.org/68518723:24
*** kranthikirang has quit IRC23:28
*** rezroo has joined #openstack-helm23:31
openstackgerritHari proposed openstack/openstack-helm-addons master: Include retry logic for checking status  https://review.opendev.org/68518823:38
*** sthussey has quit IRC23:43
openstackgerritStas Egorov proposed openstack/openstack-helm-infra master: Fixed calico chart for hyperkube 1.12  https://review.opendev.org/68518623:50
openstackgerritStas Egorov proposed openstack/openstack-helm-infra master: [TMP]: for testing  https://review.opendev.org/68518723:53

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