Monday, 2018-09-03

*** dingyichen has joined #openstack-infra00:29
*** bobh has quit IRC00:49
openstackgerritGhanshyam Mann proposed openstack-infra/devstack-gate master: Update grenade settings for stable/rocky  https://review.openstack.org/59159400:54
*** yamamoto has quit IRC01:04
*** yamamoto has joined #openstack-infra01:04
*** jiapei has joined #openstack-infra01:06
*** zhangfei has joined #openstack-infra01:09
*** slaweq has joined #openstack-infra01:11
*** bobh has joined #openstack-infra01:12
*** slaweq has quit IRC01:16
openstackgerritIan Wienand proposed openstack-infra/openstack-zuul-jobs master: Add warning job about readthedocs publishing  https://review.openstack.org/59926901:19
openstackgerritIan Wienand proposed openstack-infra/openstack-zuul-jobs master: Add warning job about readthedocs publishing  https://review.openstack.org/59926901:40
openstackgerritIan Wienand proposed openstack-infra/project-config master: Switch unmodified readthedocs jobs to warning job  https://review.openstack.org/59927001:40
*** bobh has quit IRC01:47
*** bobh has joined #openstack-infra01:57
*** bobh has quit IRC02:06
*** yamamoto_ has joined #openstack-infra02:11
*** yamamoto has quit IRC02:15
*** hongbin has joined #openstack-infra02:26
*** psachin has joined #openstack-infra02:42
*** vivsoni has quit IRC02:53
*** threestrands has quit IRC02:57
openstackgerritTristan Cacqueray proposed openstack-infra/zuul master: prometheus: add options to start the server and process collector  https://review.openstack.org/59920903:00
openstackgerritTristan Cacqueray proposed openstack-infra/zuul master: prometheus: add existing metrics  https://review.openstack.org/59921003:00
openstackgerritTristan Cacqueray proposed openstack-infra/zuul master: prometheus: add decorators to measure scheduler latency  https://review.openstack.org/59921103:00
*** slaweq has joined #openstack-infra03:11
*** slaweq has quit IRC03:16
*** hongbin has quit IRC03:19
stevebakerHey, the mirror set up in this change is returning 403s in our jobs https://review.openstack.org/#/c/592695 http://mirror.us-west-1.packethost.openstack.org:8080/cbs.centos/virt7-container-common-candidate/03:42
*** larainema has joined #openstack-infra03:44
*** jamesmcarthur has joined #openstack-infra03:49
*** yamamoto_ has quit IRC03:51
*** threestrands has joined #openstack-infra04:00
*** udesale has joined #openstack-infra04:02
ianwstevebaker: hrm04:03
ianwit looks like it hasn't been puppeted in there04:05
*** udesale has quit IRC04:06
stevebakerianw: some mirrors work http://mirror.mtl01.inap.openstack.org:8080/cbs.centos/virt7-container-common-candidate/04:06
*** udesale has joined #openstack-infra04:06
ianwstevebaker: yeah, it's something about our ansible/puppet deployment to that host ... having a look at logs now ...04:07
stevebakerok, thanks04:07
ianwit's not in emergency, so there's that04:08
*** udesale has joined #openstack-infra04:08
ianwi think we have the wrong key for it maybe04:14
*** rcernin has quit IRC04:18
*** rcernin has joined #openstack-infra04:18
*** yamamoto has joined #openstack-infra04:20
ianwwell it didn't have the root login permissions from bridge.o.o ... i do not know why04:24
ianwit being the packethost mirror04:24
AJaegerconfig-core, https://review.openstack.org/599013 removes a job and https://review.openstack.org/598591 updates descriptions - could you review these, please?04:25
*** jiapei has quit IRC04:26
AJaegerand a review request for infra-cores, topic:infra-retire is ready to retire 18 repos - next step is removing content, see https://review.openstack.org/#/q/status:open+++topic:infra-retire . If anybody could grab a couple, please...04:26
openstackgerritAndreas Jaeger proposed openstack-infra/openstack-zuul-jobs master: Add warning job about readthedocs publishing  https://review.openstack.org/59926904:28
*** yamamoto has quit IRC04:28
ianwAJaeger: ^ thanks :)  I actually got distracted by the warning message which was sub-optimal and fixing that in zuul-sphinx :)04:28
AJaeger;)04:29
AJaegerianw: you're welcome. Should we add some templates that use tox-docs instead of build-sphinx for rtd? With the pti changing, we should allow that and thus depreceate build-sphinx...04:30
ianwAJaeger: we could do.  someone brought up an interesting point along the way though that i'm not sure if RTD knows to look at docs/requirements.txt04:32
AJaegerianw: ah, yeah.... But we don't mandate docs/requirements.txt - just recommend. Oh fun...04:33
openstackgerritAndreas Jaeger proposed openstack-infra/openstack-zuul-jobs master: Add warning job about readthedocs publishing  https://review.openstack.org/59926904:34
AJaegerianw: fixed the linting ^ - named the task04:34
*** stakeda has joined #openstack-infra04:34
ianwit might be possible to get rtd to add it ... i dunno.  they have a github project04:34
AJaegerI see...04:35
AJaegerSorry, need to sign off again - day excursion with the family...04:35
ianwnp, have fun!04:37
*** jamesmcarthur has quit IRC04:41
*** ramishra has joined #openstack-infra04:41
*** jtomasek has joined #openstack-infra04:44
*** bhavikdbavishi has joined #openstack-infra04:54
*** yamamoto has joined #openstack-infra05:01
*** yamamoto has quit IRC05:05
*** yamamoto has joined #openstack-infra05:05
*** bhavikdbavishi has quit IRC05:10
*** slaweq has joined #openstack-infra05:11
*** slaweq has quit IRC05:15
*** snapiri has joined #openstack-infra05:24
*** zhangfei has quit IRC05:26
ianwclarkb: afaict the packethost mirror isn't getting picked up in ansible runs.  i have no idea right now why05:26
*** janki has joined #openstack-infra05:26
ianwi did possibly find a bug in the ansible json converter in the process, however05:26
*** hamzy has quit IRC05:32
*** hamzy has joined #openstack-infra05:32
openstackgerritAndreas Jaeger proposed openstack-infra/openstack-zuul-jobs master: Add warning job about readthedocs publishing  https://review.openstack.org/59926905:34
*** beagles has quit IRC05:35
*** quiquell has joined #openstack-infra05:41
*** aojea has joined #openstack-infra05:56
*** chkumar|pto is now known as chandankumar06:05
openstackgerritSimon Westphahl proposed openstack-infra/nodepool master: Update static nodes in Zookeeper on config change  https://review.openstack.org/59815606:05
*** snapiri has quit IRC06:12
*** shachar has joined #openstack-infra06:12
ianwstevebaker: well, the immediate issue is fixed as i did a manual run on the host06:13
*** CrayZee has joined #openstack-infra06:14
*** CrayZee is now known as Guest8111506:14
ianwafaict, it's still not picked up during the regular cron runs.  mystery?  i filed a bug/pull request for ansible-inventory anyway https://github.com/ansible/ansible/issues/4509806:15
ianwseems our keytabs make that explode06:15
ianwi doubt it is the right solution, but i'm hopeful someone will not be able to resist telling me how it's wrong and should be done :)06:16
*** shachar has quit IRC06:17
*** snapiri has joined #openstack-infra06:18
*** Guest81115 has quit IRC06:20
openstackgerritGhanshyam Mann proposed openstack-infra/devstack-gate master: Update grenade settings for stable/rocky  https://review.openstack.org/59159406:26
*** Dobroslaw has joined #openstack-infra06:31
*** threestrands has quit IRC06:48
*** jaosorior has joined #openstack-infra06:51
*** florianf has joined #openstack-infra06:59
*** Emine has joined #openstack-infra07:01
*** gnuoy has joined #openstack-infra07:05
*** dulek has joined #openstack-infra07:06
*** shardy has joined #openstack-infra07:09
*** slaweq has joined #openstack-infra07:11
*** rcernin has quit IRC07:11
*** slaweq has quit IRC07:15
*** gema has joined #openstack-infra07:19
*** alexchadin has joined #openstack-infra07:24
*** olivierb_ has joined #openstack-infra07:25
openstackgerritMarkus Hosch proposed openstack-infra/zuul master: Add a sanity check for all refs returned by Gerrit  https://review.openstack.org/59901107:25
openstackgerritMarkus Hosch proposed openstack-infra/zuul master: Add a sanity check for all refs returned by Gerrit  https://review.openstack.org/59901107:35
*** florianf is now known as florianf|afk07:35
*** tosky has joined #openstack-infra07:36
*** hashar has joined #openstack-infra07:38
*** jpena|off is now known as jpena07:43
*** apetrich has joined #openstack-infra07:43
*** jpich has joined #openstack-infra07:54
*** zhangfei has joined #openstack-infra07:55
*** psachin has quit IRC07:57
*** giblet_off has quit IRC08:09
*** holser_ has joined #openstack-infra08:10
*** ginopc has joined #openstack-infra08:12
*** pbourke has quit IRC08:23
*** sshnaidm|off is now known as sshnaidm08:23
*** pbourke has joined #openstack-infra08:24
*** pas-ha has joined #openstack-infra08:26
*** florianf|afk is now known as florianf08:33
*** derekh has joined #openstack-infra08:35
*** derekh has quit IRC08:36
*** derekh has joined #openstack-infra08:36
*** e0ne has joined #openstack-infra08:37
*** psachin has joined #openstack-infra08:41
*** ramishra_ has joined #openstack-infra08:41
*** ramishra has quit IRC08:43
openstackgerritOlivier Bourdon proposed openstack/diskimage-builder master: Fix CentOS image build failure when dib runs on debian based system and audit disabled  https://review.openstack.org/55948508:48
*** dingyichen has quit IRC08:54
*** jchhatbar has joined #openstack-infra08:56
*** janki has quit IRC08:56
*** snapiri has quit IRC08:58
*** janki has joined #openstack-infra08:59
*** jchhatbar has quit IRC08:59
*** gfidente has joined #openstack-infra09:02
*** r-mibu has joined #openstack-infra09:03
openstackgerritFabien Boucher proposed openstack-infra/zuul-jobs master: ensure-sphinx: do not attempt a gettext install if exists  https://review.openstack.org/59902809:05
*** pcaruana has joined #openstack-infra09:09
*** florianf has quit IRC09:11
*** florianf has joined #openstack-infra09:12
*** e0ne has quit IRC09:18
*** e0ne has joined #openstack-infra09:18
*** dtantsur|afk is now known as dtantsur09:19
*** slaweq has joined #openstack-infra09:20
*** owalsh has quit IRC09:20
*** owalsh has joined #openstack-infra09:33
*** electrofelix has joined #openstack-infra09:33
openstackgerritFabien Boucher proposed openstack-infra/zuul-jobs master: ensure-sphinx: do not attempt a gettext install if exists  https://review.openstack.org/59902809:37
*** snapiri has joined #openstack-infra09:38
*** snapiri has quit IRC09:38
openstackgerritMatthieu Huin proposed openstack-infra/nodepool master: Do not abort node launch if failed node cannot be deleted  https://review.openstack.org/58985409:46
*** jbadiapa has quit IRC09:49
*** stakeda has quit IRC09:56
*** jbadiapa has joined #openstack-infra10:01
*** shardy has quit IRC10:03
*** ramishra_ has quit IRC10:19
*** ramishra has joined #openstack-infra10:21
openstackgerritSimon Westphahl proposed openstack-infra/nodepool master: Fix node state checks after aquiring lock  https://review.openstack.org/59934310:24
*** alexchadin has quit IRC10:37
*** ccamacho has joined #openstack-infra10:41
*** ccamacho has quit IRC10:41
*** alexchadin has joined #openstack-infra10:41
*** alexchadin has quit IRC10:52
*** pbourke has quit IRC10:53
*** pbourke has joined #openstack-infra10:54
*** udesale has quit IRC10:56
*** dosaboy has joined #openstack-infra11:00
*** zhangfei has quit IRC11:01
*** yamamoto has quit IRC11:06
*** ccamacho has joined #openstack-infra11:06
*** yamamoto has joined #openstack-infra11:07
*** jpena is now known as jpena|lunch11:12
*** ramishra has quit IRC11:18
openstackgerritThierry Carrez proposed openstack/ptgbot master: ptgbot should still ask for identify-map cap  https://review.openstack.org/59935711:18
*** yboaron has joined #openstack-infra11:20
*** gibi has joined #openstack-infra11:23
yboaronHi folks,11:25
yboaronI need your help with https://review.openstack.org/#/c/595569/,11:25
yboaronAny idea why the requirements-integration gate fails?11:25
openstackgerritMerged openstack/ptgbot master: ptgbot should still ask for identify-map cap  https://review.openstack.org/59935711:33
*** cmurphy has joined #openstack-infra11:44
*** ccamacho has quit IRC11:50
*** ccamacho has joined #openstack-infra11:50
*** rfolco has joined #openstack-infra11:59
frickleryboaron: see http://lists.openstack.org/pipermail/openstack-dev/2018-September/134166.html11:59
*** zhangfei has joined #openstack-infra12:06
*** jpena|lunch is now known as jpena12:10
openstackgerritMerged openstack-infra/zuul-sphinx master: Warn when autoroles doesn't find a README.rst  https://review.openstack.org/59601412:10
*** yamamoto has quit IRC12:13
*** yamamoto has joined #openstack-infra12:13
*** dhill_ has joined #openstack-infra12:14
*** Bhujay has joined #openstack-infra12:14
openstackgerritMerged openstack-infra/nodepool master: Move sphinx + deps to doc/requirements.txt  https://review.openstack.org/59156512:22
yboaronfrickler, 10x for the information12:23
*** tpsilva has joined #openstack-infra12:29
*** jaosorior has quit IRC12:29
*** dtantsur is now known as dtantsur|brb12:32
*** udesale has joined #openstack-infra12:39
*** janki has quit IRC12:39
*** zul has joined #openstack-infra12:40
*** zhangfei has quit IRC12:43
sshnaidmpabelanger, do you know what's problem with cbs repos that we added recently? http://mirror.us-west-1.packethost.openstack.org:8080/cbs.centos/virt7-container-common-candidate/x86_64/os/repodata/repomd.xml: [Errno 14] HTTP Error 403 - Forbidden  http://logs.openstack.org/67/599267/2/check/tripleo-ci-centos-7-undercloud-containers/bd7f6af/logs/undercloud/home/zuul/repo_setup.log.txt.gz#_2018-09-03_02_37_0612:47
fungisshnaidm: see earlier ianw and stevebaker were discussing... for some reason that recent configuration change only ended up applied to some of our per-region mirror servers12:49
sshnaidmfungi, thanks, reading back12:49
fungimight be just a problem with mirror.us-west-1.packethost.openstack.org12:50
*** ramishra has joined #openstack-infra12:50
*** pcaruana has quit IRC12:52
*** shardy has joined #openstack-infra12:55
fungitl;dr, looks like it might be due to a bug upstream in ansible. ianw has filed a bug and pushed up this pr to fix it: https://github.com/ansible/ansible/pull/4509913:00
*** yboaron_ has joined #openstack-infra13:02
openstackgerritJan Kundrát proposed openstack-infra/git-review master: Support usernames that contain '@' and ssh Git URLs  https://review.openstack.org/42870013:03
*** yboaron has quit IRC13:04
*** jaosorior has joined #openstack-infra13:06
*** janki has joined #openstack-infra13:08
openstackgerritJoshua Hesketh proposed openstack-infra/zuul master: Serve project SSH keys and document  https://review.openstack.org/59906313:15
*** Emine has quit IRC13:28
*** dtantsur|brb is now known as dtantsur13:30
*** Emine has joined #openstack-infra13:30
*** psachin has quit IRC13:34
*** zhangfei has joined #openstack-infra13:44
*** jaosorior has quit IRC13:53
*** Bhujay has quit IRC13:58
*** bobh has joined #openstack-infra14:03
*** bobh has quit IRC14:03
openstackgerritMichel Peterson proposed openstack-infra/openstack-zuul-jobs master: Add project templates for openstack-tox-cover  https://review.openstack.org/59810414:05
*** r-daneel has joined #openstack-infra14:09
openstackgerritJan Kundrát proposed openstack-infra/git-review master: Support usernames that contain '@' and ssh Git URLs  https://review.openstack.org/42870014:15
*** quiquell has quit IRC14:23
*** larainema has quit IRC14:24
*** hashar has quit IRC14:26
*** r-daneel has quit IRC14:28
*** r-daneel has joined #openstack-infra14:30
lennybCores, pls review #link https://review.openstack.org/#/c/559731/  Currently stable/rocky devstack fails with tempest enabled14:33
*** bobh has joined #openstack-infra14:37
*** bobh has quit IRC14:39
*** yamamoto has quit IRC14:44
*** yamamoto has joined #openstack-infra14:45
*** yamamoto has quit IRC14:45
*** jiapei has joined #openstack-infra14:47
*** bobh has joined #openstack-infra14:49
*** bobh has quit IRC14:53
openstackgerritDean Troyer proposed openstack-infra/project-config master: Set up stx-docs and stx-specs publish jobs  https://review.openstack.org/59905414:53
*** yboaron_ has quit IRC14:54
*** yboaron_ has joined #openstack-infra14:54
*** zhangfei has quit IRC14:57
*** bobh has joined #openstack-infra14:58
*** bobh has quit IRC15:03
TheJuliaHey infra folks, could someone add ironic-core to virtualbmc-core?15:05
*** bobh has joined #openstack-infra15:10
*** bobh has quit IRC15:14
*** bobh has joined #openstack-infra15:18
*** r-mibu has quit IRC15:21
*** bobh has quit IRC15:22
fungiTheJulia: done!15:24
TheJuliafungi: thanks!15:24
*** yamamoto has joined #openstack-infra15:26
*** udesale has quit IRC15:26
clarkbfungi ianw I know with the move to bridge.o.o some hosts didnt get the new root key was that one of them?15:28
clarkber was packethost mirror one of them15:28
*** bobh has joined #openstack-infra15:29
*** dtantsur is now known as dtantsur|afk15:32
*** bobh has quit IRC15:33
fungii can sudo ssh to it successfully from bridge.o.o, so presumably not?15:35
fungiand it's in the inventory15:35
clarkbemergency file and whether kr not ansible says it is valid inventory entry are the other two things to check I think15:38
*** bobh has joined #openstack-infra15:39
*** bobh has quit IRC15:43
*** gibi has quit IRC15:46
*** e0ne has quit IRC15:48
*** bobh has joined #openstack-infra15:50
*** bharath_t has joined #openstack-infra15:50
clarkbnot in the emergency file15:53
*** bobh has quit IRC15:54
bharath_tHi, I need some advice regarding the project in openstack ecosystem. I got the permissions to work on the project openstack Meteos. As there is no one working on it, and I want take the responsibility of  taking it. I want to change the architecture of existing project, and there by I also want to change the name of the project. Can you please suggest is it better/easy to change the name of the project or create a new project and ramp up existi15:55
bharath_tng one?15:55
clarkbrenaming a project requires a gerrit downtime as we have to make changes to the database under gerrit. If you are not going to share history with the meteos code base then starting from scratch is probably easier15:57
bharath_tclarkb, yeah the code base will be totally new15:58
*** yamamoto has quit IRC15:58
*** yamamoto has joined #openstack-infra15:58
bharath_tclarkb, can you please point me to the docs to create new project under openstack eco system?15:59
clarkbbharath_t: https://docs.openstack.org/infra/manual/creators.html16:00
*** bobh has joined #openstack-infra16:01
*** jpich has quit IRC16:01
clarkbfungi: ianw from system-config/inventory/groups.yaml disabled: inventory_hostname.startswith('backup') or inventory_hostname.startswith('wiki') or inventory_hostname.startswith('puppetmaster') or inventory_hostname in ['ask-staging01.openstack.org', 'mirror01.sto2.citycloud.openstack.org', 'mirror02.us-west-1.packethost.openstack.org', 'ask-staging.openstack.org', 'review.openstack.org']16:02
fungioh!16:02
clarkbfungi: ianw I do not know why mirror02.us-west-1.packethost.openstack.org is in that list but I think it can be removed from that list as I can asnible to it directly. Patch in a moment16:02
fungiit's in our indefinite disable list, not our emergency disable list16:02
bharath_tthanks clarkb16:02
fungiyeah, i think it may have been offline or something initially when bridge.o.o was being set up16:03
openstackgerritThomas Morin proposed openstack-infra/project-config master: Add openstack/neutron-inter to neutron  https://review.openstack.org/59942916:03
openstackgerritClark Boylan proposed openstack-infra/system-config master: Run puppet on mirror02 in packethost  https://review.openstack.org/59943016:04
clarkbfungi: ianw ^ I think that will correct the issue16:04
clarkbit is looking quiet around here otherwise so maybe I should take that as a clue to go enjoy the holiday too :)16:05
*** bobh has quit IRC16:06
clarkbfungi: I was hoping to get a list of groups given a hostname from ansible but can't figure that out, I ended up just reading our inventory config16:06
*** bobh has joined #openstack-infra16:07
*** hashar has joined #openstack-infra16:07
*** bharath_t has quit IRC16:08
*** bobh has quit IRC16:14
AJaegerconfig-core, https://review.openstack.org/599013 removes a job and https://review.openstack.org/598591 updates descriptions - could you review these, please?16:18
*** ccamacho has quit IRC16:20
*** yboaron_ has quit IRC16:23
*** bobh has joined #openstack-infra16:27
*** tonyb has quit IRC16:31
*** bobh has quit IRC16:31
*** bobh has joined #openstack-infra16:38
*** dhellmann has quit IRC16:39
*** dhellmann has joined #openstack-infra16:39
*** yamamoto has quit IRC16:41
*** bobh has quit IRC16:42
*** patrickeast has left #openstack-infra16:43
*** jpena is now known as jpena|off16:43
*** dhellmann has quit IRC16:44
*** armax has joined #openstack-infra16:45
*** dhellmann has joined #openstack-infra16:45
openstackgerritsebastian marcet proposed openstack-infra/openstackid-resources master: Updated PUT/POST affiliation  https://review.openstack.org/59943516:49
*** electrofelix has quit IRC16:50
*** bobh has joined #openstack-infra16:50
openstackgerritMerged openstack-infra/openstackid-resources master: Updated PUT/POST affiliation  https://review.openstack.org/59943516:51
*** aojea has quit IRC16:51
*** dhellmann has quit IRC16:54
*** bobh has quit IRC16:54
*** dhellmann has joined #openstack-infra16:54
*** derekh has quit IRC17:00
*** bobh has joined #openstack-infra17:00
*** bobh has quit IRC17:04
*** _Cyclone_ has quit IRC17:07
*** _Cyclone_ has joined #openstack-infra17:08
*** bobh has joined #openstack-infra17:08
*** janki has quit IRC17:10
*** holser_ has quit IRC17:11
*** armax has quit IRC17:11
*** r-daneel has quit IRC17:13
*** bobh has quit IRC17:13
*** r-daneel has joined #openstack-infra17:13
*** bobh has joined #openstack-infra17:18
*** yamamoto has joined #openstack-infra17:20
*** bobh has quit IRC17:22
*** bobh has joined #openstack-infra17:22
*** hongbin has joined #openstack-infra17:28
*** hongbin has quit IRC17:31
*** hongbin has joined #openstack-infra17:33
*** jiapei has quit IRC17:37
*** florianf has quit IRC17:40
openstackgerritMerged openstack-infra/openstack-zuul-jobs master: Remove obsolete legacy-tempest-dsvm-full-kafka  https://review.openstack.org/59901317:41
*** ramishra has quit IRC17:44
*** dave-mccowan has quit IRC17:47
*** e0ne has joined #openstack-infra17:48
*** pcrews has joined #openstack-infra17:48
*** e0ne_ has joined #openstack-infra17:50
ssbarneaAJaegerfungi : if you could review https://review.openstack.org/#/c/598486/ it would be great as it would prevent wasting time running jobs with conflicts in the future.17:52
AJaegerssbarnea: AJaegerfungi is interesting ;)17:53
AJaegerssbarnea: I'm not reviewing devstack-gate...17:53
*** e0ne has quit IRC17:54
ssbarneaAJaeger: my client displayed them separated :)17:54
*** gfidente is now known as gfidente|afk17:56
AJaegerssbarnea: double checking with eavesdrop - see http://eavesdrop.openstack.org/irclogs/%23openstack-infra/latest.log.html#t2018-09-03T17:52:58 - I think your client is broken18:02
*** tosky has quit IRC18:04
ssbarneaAJaeger: i never said Riot would not be broken in many places https://s3.sbarnea.com/ss/180903-Monosnap_.png18:05
ssbarneai will keep in mind to keep separators18:05
clarkbssbarnea: I left a review, I don't think that change will necessarily improve things18:18
ssbarneaclarkb: pip install does *not* check for conflicts, ever. there is real risk that after doing any pip install, you may have conflicts. (pip is not yum, ....)18:19
*** gfidente|afk has quit IRC18:20
clarkbssbarnea: that is not true, it checks for conflicst within the packages being installed18:20
clarkbssbarnea: which is why it is redundant, since we only install one set of packages (if we ignore the dep less pbr)18:20
ssbarneaclarkb: also, last time I checked, virtualenv had its own copy of pip inside it and it was not installing latest pip, in fact it was quite an outdated version.18:21
clarkbwhat it does not do is check for conflicts between existing installs and the new package installs18:21
clarkbssbarnea: current modern virtuaelnv (for a couple years now) installs pip setuptools and wheel at venv creation time18:21
clarkbssbarnea: very old virtualenv does what you describe18:21
ssbarneahttps://github.com/pypa/pip/issues/988 is still open.18:22
clarkbssbarnea: yes I filed that bug :)18:22
clarkbpip does a breadth first check of its deps and first instance wins18:23
clarkbwhich means if first packages says install foo===1.0 and second package says foo<1.0 it will fail18:23
clarkbwhat it doesn't address is if first package says foo >0.8 installs 1.1 then second packages says foo<1.0 results in 0.9 being installed18:24
clarkbit will instead fail18:24
clarkbbecause 1.1 is installed and second package does not want that version18:24
ssbarneaanyway, I have nothing against removing the pip>=10.0 if you know is redundant. but i know from experience that I often found conflicts  and adding a check was a good way to catch them.18:24
clarkbthe actual issues here are I think ara didn't specify a minimum supported ansible version in its deps and we aren't set -e there18:25
clarkbso even if it had failed we would've continued on in the specific case you are thinking of, but it didn't because nothing in the requirements lists specified a conflict18:25
clarkbif you want to improve this particular case what we need is to check the return code of the main pip install and exit if non zero18:26
clarkbI think18:26
ssbarneadon't we run with bash strict mode?18:26
clarkbnot in devstack gate18:27
ssbarneano CI job should ever run without strict :) .... OMG, in this case  you are right, there are other things to fix first.18:27
clarkbthis is mostly off the top of my head so should be double checked but I believe that is the case18:27
ssbarneain this case I will probably make a CR to make it strict fist.18:28
*** ansmith has joined #openstack-infra18:29
ssbarneai know that virtualenv activate it notorious of not being compatible with full strict mode, I tried for (almost years) to fix it (PRs) but got nowhere.18:29
ssbarneaclarkb: maybe you have some inside contacts in pypa team, so we can pull some ropes.18:30
*** olivierb_ has quit IRC18:32
ssbarneajust one example (which was recommended for getting other fixed in) : https://github.com/pypa/virtualenv/pull/1087 -- soon to be one year old18:33
clarkbI don't think we ever activate the venv there fwiw18:35
clarkbwe just execute commands out of it18:36
AJaegerianw: seems that readthedocs handles docs/requirements, see https://github.com/rtfd/readthedocs.org/blob/master/docs/guides/specifying-dependencies.rst18:39
AJaegerssbarnea, clarkb, this is also devstack-gate - which is obsolete. The new Zuul v3 jobs don't need it. So, I wonder what'S the proper place for such a chnage is.18:40
openstackgerritsebastian marcet proposed openstack-infra/openstackid-resources master: Added new endpoints get organizations  https://review.openstack.org/59944518:42
*** e0ne_ has quit IRC18:43
openstackgerritMerged openstack-infra/openstackid-resources master: Added new endpoints get organizations  https://review.openstack.org/59944518:44
ssbarneaAJaeger: indeed, adding deps into reqs.txt file would on my mind. regarding obsolete part, i rememeber that few days ago we got a job that run this code, if is obsolete and not use the code should be removed. i am afraid is still used. anyway I need to go now, i will try to make a CR tomorrow, after dealing with more pressing issues.18:44
*** e0ne has joined #openstack-infra18:47
AJaegerssbarnea: devstack-gate is still used - but newly written jobs should not use it anymore. All jobs using it today, should be converted...18:51
sshnaidmpabelanger, hi, do you know if zuul rebases patches when it clones them to workspace?18:56
clarkbsshnaidm: f9r changes they are all rebased on the target branch or speculative future state foryou18:59
clarkbrefs are used as is (like post and release jobs)18:59
sshnaidmclarkb, sorry, not sure I understood - if I have patch A and master 4 commits ahead, when I run job for A - will I have rebased code with these 4 commits?19:00
clarkbyes, rebased on current master because that is target branch19:06
clarkbbut could be stable/rocky if that was the target for example19:06
*** Gorian has quit IRC19:10
*** vkmc has quit IRC19:10
*** kencjohnston_ has quit IRC19:10
*** lamt has quit IRC19:10
*** csatari has quit IRC19:10
*** mrhillsman has quit IRC19:10
*** kmalloc has quit IRC19:10
*** betherly has quit IRC19:10
*** xyang has quit IRC19:10
*** simonmcc has quit IRC19:10
*** jbryce has quit IRC19:10
*** yujunz has quit IRC19:10
*** Ng has quit IRC19:10
*** mrmartin has quit IRC19:10
*** fkautz has quit IRC19:10
*** mark-burnett has quit IRC19:10
*** lsell has quit IRC19:10
*** kencjohnston has joined #openstack-infra19:10
*** Gorian has joined #openstack-infra19:12
sshnaidmclarkb, thanks19:14
*** vkmc has joined #openstack-infra19:14
*** Ng has joined #openstack-infra19:15
*** kmalloc has joined #openstack-infra19:15
sshnaidmclarkb, and when I press "rebase" button in the review.. then it will be rebased as well? So no difference?19:25
sshnaidmwhy to have the button then19:25
*** e0ne has quit IRC19:33
clarkbthe button is a gerrit feature and actually rebases the change19:36
clarkbthe other behavior is zuul specific since zuul aims to test the mergeable state19:36
clarkbbut ya, many people suggest unnecessary rebases to make ci work. Shouldnt be required with our zuul19:37
clarkbthe exception is to fix merge conflicta19:37
*** hongbin has quit IRC19:44
openstackgerritChris Dent proposed openstack-infra/project-config master: Remove job entry for microversion-parse  https://review.openstack.org/59945319:45
*** bobh has quit IRC19:46
*** bobh has joined #openstack-infra19:49
*** bobh has quit IRC19:54
*** bobh has joined #openstack-infra19:58
stevebakerianw: thanks19:58
*** olivierb has joined #openstack-infra20:02
*** bobh has quit IRC20:02
clarkbstevebaker: ianw fyi https://review.openstack.org/#/c/599430/ is the fix, I think lack of reviews is mostly due to the holiday, but when ianw wakes hopefully we can get that in20:03
stevebakerclarkb: ok, do you think other mirrors are affected? If its possible, we'll hold off on using the new mirror for now20:05
clarkbstevebaker: no, I think that is the only one. The other mirrors listed on that line are for cloud regions that are not currently active20:06
*** bobh has joined #openstack-infra20:11
stevebakerclarkb: ok, thanks20:15
*** bobh has quit IRC20:15
*** bobh has joined #openstack-infra20:16
*** bobh has quit IRC20:18
*** bobh has joined #openstack-infra20:18
*** bobh has quit IRC20:22
*** hongbin has joined #openstack-infra20:25
*** bobh has joined #openstack-infra20:27
*** bobh has quit IRC20:31
*** bobh has joined #openstack-infra20:40
*** hashar has quit IRC20:41
*** tonyb has joined #openstack-infra20:51
*** imacdonn has quit IRC21:02
*** imacdonn has joined #openstack-infra21:02
*** bobh has quit IRC21:16
*** agopi has joined #openstack-infra21:17
*** fried_rice1 has joined #openstack-infra21:19
*** Emine has quit IRC21:20
*** fried_rice has quit IRC21:20
*** fried_rice1 is now known as fried_rice21:20
ianwclarkb: ahh, thanks.  i didn't look deep enough in the config files :)21:21
*** fried_rice1 has joined #openstack-infra21:29
*** fried_rice has quit IRC21:30
*** fried_rice has joined #openstack-infra21:31
*** fried_rice1 has quit IRC21:33
openstackgerritMerged openstack-infra/system-config master: Run puppet on mirror02 in packethost  https://review.openstack.org/59943021:39
*** SpamapS has quit IRC21:39
*** fried_rice has quit IRC21:57
*** auristor has quit IRC21:57
*** tpsilva has quit IRC21:57
*** tobiash has quit IRC22:06
*** jesusaur has quit IRC22:08
*** fried_rice has joined #openstack-infra22:22
prometheanfirezzzeek: I'm not sure https://github.com/openstack/oslo.db/commit/df6bf3401266f42271627c1e408f87c71a06cef7#diff-52043ef9a440ade65cc878df26b10604R1054 is working right or something22:31
*** hongbin has quit IRC22:32
*** armax has joined #openstack-infra22:33
*** hongbin has joined #openstack-infra22:34
*** armax has quit IRC22:36
*** rcernin has joined #openstack-infra22:37
*** hongbin has quit IRC22:50
*** threestrands has joined #openstack-infra23:03
*** tosky has joined #openstack-infra23:03
*** threestrands has quit IRC23:03
*** threestrands has joined #openstack-infra23:04
*** threestrands has quit IRC23:05
*** threestrands has joined #openstack-infra23:05
*** threestrands has quit IRC23:06
*** threestrands has joined #openstack-infra23:07
*** threestrands has quit IRC23:08
*** threestrands has joined #openstack-infra23:08
prometheanfirezzzeek: as far as I can tell, that's the first usage of moves.moved_property23:12
*** r-daneel has quit IRC23:15
*** d0ugal has quit IRC23:28
*** slaweq_ has joined #openstack-infra23:31
*** slaweq has quit IRC23:31
*** elod has quit IRC23:41
*** jesusaur has joined #openstack-infra23:44
*** auristor has joined #openstack-infra23:44
*** tosky has quit IRC23:52
*** d0ugal has joined #openstack-infra23:56

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