Friday, 2019-08-16

*** goldyfruit has quit IRC00:30
*** goldyfruit has joined #openstack-doc00:31
*** goldyfruit has quit IRC00:55
*** brinzhang_ has joined #openstack-doc01:00
*** brinzhang has quit IRC01:03
*** brinzhang has joined #openstack-doc01:16
*** brinzhang_ has quit IRC01:20
*** goldyfruit has joined #openstack-doc02:04
*** tinwood has quit IRC02:12
*** tinwood has joined #openstack-doc02:13
AJaegerasettle: the tripleo-docs change is https://review.opendev.org/#/c/569185/ - for some history ;)05:41
*** kopecmartin|off is now known as kopecmartin06:53
*** brinzhang_ has joined #openstack-doc06:56
*** brinzhang has quit IRC06:59
*** brinzhang_ has quit IRC07:17
*** brinzhang_ has joined #openstack-doc07:18
asettleThanks AJaeger08:34
AJaegerasettle: could you put  https://review.opendev.org/676483 on your review queue, please? This is a functional change that I think is beneficial...08:50
AJaegergood morning, asettle ;)08:50
*** alexmcleod has joined #openstack-doc08:55
asettleAJaeger, morning!10:02
asettleAnd yep, it's in10:02
asettleLooking through10:02
*** brinzhang_ has quit IRC10:20
AJaegerstephenfin: could you also put  https://review.opendev.org/676483 on your review queue, please? This is a functional change that I think is beneficial...10:45
AJaegerthanks, asettle10:45
stephenfinsure10:46
asettleYou got it!10:47
AJaegerthanks, stephenfin10:57
AJaegerstephenfin: also https://review.opendev.org/676739 needs a +1 for the release of openstack-doc-tools10:57
stephenfinyou got it (y)10:58
AJaeger;)10:58
openstackgerritMerged openstack/api-site master: Switch publishing to publish-docs/html  https://review.opendev.org/67657510:59
openstackgerritMerged openstack/openstack-doc-tools master: Remove some commented obsolete lines  https://review.opendev.org/67623811:00
AJaegerone more cleanup change for review: https://review.opendev.org/#/c/676751/ ...11:01
openstackgerritMerged openstack/openstack-manuals master: Use opendev-tox-docs job  https://review.opendev.org/67648311:07
openstackgerritAndreas Jaeger proposed openstack/openstack-manuals master: Fix queue for promote job  https://review.opendev.org/67691311:41
*** hemna has joined #openstack-doc11:44
openstackgerritMerged openstack/openstack-manuals master: Fix queue for promote job  https://review.opendev.org/67691312:13
*** goldyfruit has quit IRC12:32
AJaegeryeah, this is working now ^12:43
AJaegerstephenfin, asettle, here's a "normal" change - updating the glossary: https://review.opendev.org/67624812:49
asettleOhhh fun fun12:49
asettleDo you want +2a AJaeger ?12:50
asettleDoesn't look complicated12:50
openstackgerritMerged openstack/training-guides master: Publish to publish-docs/html  https://review.opendev.org/67657612:52
openstackgerritMerged openstack/security-doc master: Use publish-docs/html  https://review.opendev.org/67657412:59
AJaegerasettle: your call...13:03
asettleAll yours dude :D13:03
asettleIf you have any weight in any of these projects AJaeger - Some +2 action would be helpful on some of these: https://review.opendev.org/#/q/topic:fix-url-404+(status:open+OR+status:merged)13:04
openstackgerritAndreas Jaeger proposed openstack/openstack-manuals stable/ocata: Switch output to publish-docs/html  https://review.opendev.org/67693113:04
AJaegerasettle: oh, you have been fixing some as well? So did I ;)13:04
asettleOh yeah, been a real joy13:05
asettleHhahaha13:05
asettleNeed to fix them for the rocky docs we're using for suse.com/documentation... but also cause they're fundamentally broken :p13:05
asettleI've got a weird one with Sahara docs atm and I'm not sure I know anyone working on Sahara. Do you?13:06
AJaegerasettle: I did more like https://review.opendev.org/67572613:06
AJaegerasettle: but can't help with those reviews13:06
AJaegerasettle: tosky works on sahara - I just went to the channels and pinged friendly13:07
asettleAJaeger, ah that's because you're smarter than me ;)13:07
asettleOh yes of course he does!13:07
asettleokay, I'll hop over there now13:07
AJaegerinsistent ;)13:08
asettleOh that's what we're calling it? :p13:08
AJaegerYeah!13:10
*** goldyfruit has joined #openstack-doc13:10
asettle\o/13:11
openstackgerritMerged openstack/openstack-manuals master: Glossary: Add OpenDev, update Zuul  https://review.opendev.org/67624813:16
openstackgerritAndreas Jaeger proposed openstack/training-guides master: Switch to new promote-openstack-manuals job  https://review.opendev.org/67693313:18
AJaegerasettle: could you review this as well, please? ^13:18
openstackgerritAndreas Jaeger proposed openstack/openstack-manuals stable/newton: Switch output to publish-docs/html  https://review.opendev.org/67693513:19
openstackgerritAndreas Jaeger proposed openstack/openstack-manuals stable/ocata: Switch output to publish-docs/html  https://review.opendev.org/67693113:20
openstackgerritOpenStack Proposal Bot proposed openstack/security-doc master: Updated from openstack-manuals  https://review.opendev.org/67693613:20
openstackgerritAndreas Jaeger proposed openstack/openstack-manuals stable/newton: Switch output to publish-docs/html  https://review.opendev.org/67693513:23
openstackgerritAndreas Jaeger proposed openstack/openstack-manuals stable/mitaka: Switch output to publish-docs/html  https://review.opendev.org/67694213:28
openstackgerritAndreas Jaeger proposed openstack/openstack-manuals stable/newton: Switch output to publish-docs/html  https://review.opendev.org/67693513:35
openstackgerritMerged openstack/training-guides master: Switch to new promote-openstack-manuals job  https://review.opendev.org/67693313:37
openstackgerritMerged openstack/security-doc master: Updated from openstack-manuals  https://review.opendev.org/67693613:41
openstackgerritAndreas Jaeger proposed openstack/openstack-manuals stable/mitaka: Switch output to publish-docs/html  https://review.opendev.org/67694213:45
asettleAJaeger, I see you got that one :p13:46
AJaegerasettle: yeah... wanted to get it all fixed...13:47
AJaegernow fighting with old stable branches ;(13:47
asettleAw isn't that fun13:50
openstackgerritAndreas Jaeger proposed openstack/openstack-manuals stable/newton: Switch output to publish-docs/html  https://review.opendev.org/67693513:57
openstackgerritAndreas Jaeger proposed openstack/openstack-manuals stable/mitaka: Switch output to publish-docs/html  https://review.opendev.org/67694214:01
*** goldyfruit_ has joined #openstack-doc14:03
openstackgerritAndreas Jaeger proposed openstack/openstack-manuals stable/ocata: Switch output to publish-docs/html  https://review.opendev.org/67693114:04
openstackgerritAndreas Jaeger proposed openstack/openstack-manuals stable/mitaka: Switch output to publish-docs/html  https://review.opendev.org/67694214:05
openstackgerritAndreas Jaeger proposed openstack/openstack-manuals stable/newton: Switch output to publish-docs/html  https://review.opendev.org/67693514:05
*** goldyfruit has quit IRC14:06
openstackgerritAndreas Jaeger proposed openstack/openstack-manuals master: Cleanup jobs and config  https://review.opendev.org/67695614:20
openstackgerritAndreas Jaeger proposed openstack/training-guides master: Rename docs job  https://review.opendev.org/67695714:21
openstackgerritAndreas Jaeger proposed openstack/openstack-manuals master: Remove checkbuild job  https://review.opendev.org/67695814:22
openstackgerritAndreas Jaeger proposed openstack/training-guides master: Rename docs job  https://review.opendev.org/67695714:25
openstackgerritAndreas Jaeger proposed openstack/openstack-manuals stable/newton: Switch output to publish-docs/html  https://review.opendev.org/67693514:34
openstackgerritAndreas Jaeger proposed openstack/openstack-manuals stable/mitaka: Switch output to publish-docs/html  https://review.opendev.org/67694214:36
AJaegerasettle: sorry, making one more change ...14:36
asettlenp link when you're done14:36
AJaegerwill do...14:38
openstackgerritAndreas Jaeger proposed openstack/openstack-manuals stable/ocata: Switch output to publish-docs/html  https://review.opendev.org/67693114:38
AJaegerasettle:if those three pass, they are good ;)14:41
openstackgerritAndreas Jaeger proposed openstack/openstack-manuals master: Simplify publishdocs et al  https://review.opendev.org/67696514:47
AJaegerasettle: want to review https://review.opendev.org/676957 as well, please?14:51
asettlegot it14:52
AJaegeryeah, thanks14:52
openstackgerritAndreas Jaeger proposed openstack/security-doc master: Cleanup jobs  https://review.opendev.org/67696614:56
openstackgerritAndreas Jaeger proposed openstack/api-site master: Cleanup jobs  https://review.opendev.org/67696714:59
AJaegerasettle: sorry, two more ^ - that's it - I hope at least ;)14:59
openstackgerritAndreas Jaeger proposed openstack/api-site master: Cleanup jobs  https://review.opendev.org/67696715:00
asettleokay :D15:00
asettleCan I ask a favor though AJaeger? :D15:00
AJaegersure15:01
asettleCould you backport https://review.opendev.org/675667 and https://review.opendev.org/675667 to stein and rocky? :D15:01
AJaegerthat's twice the same URL...15:01
asettle... damn15:01
asettleThat's what I meant15:02
asettleI don't know why I put it twice15:02
asettleI meant just the once :p15:02
AJaegerlet me try...15:02
hemnaso, last night I tried running the commands listed in the patch: https://review.opendev.org/#/c/664555/2/roles/build-pdf-docs/tasks/main.yaml15:02
hemnaagainst Cinder, and it did work15:03
hemnamake seemed to output lots of warnings/errors15:03
hemnabut it did generated a Cinder.pdf15:03
AJaegerstephenfin: want to review any of my mass cleanups for the repos? Otherwise I'll self-approve again to get this through15:03
stephenfinAJaeger: I'm okay with you self-approving. I trust you :)15:03
AJaegerhemna: latex is noisy ;) But if you have a Cinder.pdf, let's celebrate ;)15:03
AJaegerthanks, stephenfin15:03
stephenfinassuming that's okay with you15:03
asettleAJaeger, I can go through and +2a if you want15:04
hemnaI was going to try os-brick next15:04
hemnaand then cinderclient15:04
asettlehemna, powah that's pretty good news15:04
hemnahttp://paste.openstack.org/show/757496/15:04
hemnathat's all the noise15:04
*** kopecmartin is now known as kopecmartin|off15:06
AJaegerasettle: let me +A myself - in proper order to see that everything works - need to schedule in a project-config change as well ;) Thanks15:06
AJaegerasettle: backported to stein ;)15:06
asettleGood call, AJaeger15:07
asettleAnd yay! Thanks :D15:07
asettleAJaeger, possible to do Rocky? ;)15:07
AJaegertrying...15:07
AJaegerlet's see what cmurphy will say about those...15:07
asettleOh,. she suggested :p15:08
AJaegerasettle: far too many changes in that tree - half of the changes don#t apply in rocky ;(15:08
AJaegerah, good!15:08
asettleAJaeger, I had: https://review.opendev.org/#/c/676908/ and her response was for hte backports.15:08
asettleAhhh dangit15:08
asettleI'd be happy to keep this patch for Rocky then15:08
AJaegerso, giving up on rocky ;(15:09
asettleSAD NEWS15:10
asettleOkay, I'll respond to the review15:10
AJaegerstephenfin, asettle, one more open request: https://review.opendev.org/676965 , please15:13
asettleAJaeger, so demanding :p15:14
AJaegeryes - want this down before the weekend. If you leave - so be it ;)15:14
AJaegers/down/done/ - I need weekend ;)15:15
*** hemna has quit IRC15:18
asettleAJaeger, hahaha I'll be here to help :D15:19
asettleLet me know if you want +2A on some of them15:19
asettleAnd sounds like you do need a weekend! Thanks for all your work on this15:19
*** goldyfruit___ has joined #openstack-doc15:27
*** goldyfruit_ has quit IRC15:28
AJaegerthanks15:29
*** goldyfruit_ has joined #openstack-doc15:29
*** hemna has joined #openstack-doc15:29
openstackgerritAndreas Jaeger proposed openstack/openstack-doc-tools master: Remove openstack-jsoncheck and openstack-indexpage  https://review.opendev.org/67697715:31
AJaegerstephenfin: here's the delete request ^15:31
AJaegerlet me add a releasenote...15:32
*** goldyfruit___ has quit IRC15:32
openstackgerritAndreas Jaeger proposed openstack/openstack-doc-tools master: Remove openstack-jsoncheck and openstack-indexpage  https://review.opendev.org/67697715:33
openstackgerritMerged openstack/openstack-manuals master: Cleanup jobs and config  https://review.opendev.org/67695615:33
AJaegerstephenfin: if you +2A 676977, I'll update the release request and we get this in as well...15:35
openstackgerritAndreas Jaeger proposed openstack/openstack-manuals master: [doc-contrib-guide] Remove jsoncheck  https://review.opendev.org/67697815:37
hemnaI tried os-brick and python-cinderclient and they both work15:43
AJaegergreat!15:45
AJaegerasettle, stephenfin, Cinder repos look fine ^15:46
openstackgerritMerged openstack/openstack-doc-tools master: Cleanup doc-tools-check-languages  https://review.opendev.org/67675115:46
hemnacinderlib failed to build15:48
hemnahttp://paste.openstack.org/show/757971/15:48
hemnathere are some embedded images in the rst files15:49
hemnaand sphinx-build puked on it15:49
AJaegerhemna: amotoki wrote someting in the mailing list about an rst2svg sphinx extension, you might need that as wel15:49
openstackgerritAndreas Jaeger proposed openstack/openstack-doc-tools master: Remove openstack-jsoncheck and openstack-indexpage  https://review.opendev.org/67697715:50
openstackgerritMerged openstack/training-guides master: Rename docs job  https://review.opendev.org/67695716:06
openstackgerritMerged openstack/api-site master: Cleanup jobs  https://review.opendev.org/67696716:08
openstackgerritMerged openstack/security-doc master: Cleanup jobs  https://review.opendev.org/67696616:17
hemnaAJaeger: have a pointer to the message?16:20
*** goldyfruit_ has quit IRC16:32
AJaegerhemna: https://review.opendev.org/#/c/676730/2/doc/source/conf.py is the change16:32
AJaegerneeds a requirements update as well16:32
* AJaeger couldn't find the mail quickly - just the change ;)16:33
*** goldyfruit_ has joined #openstack-doc16:53
openstackgerritMerged openstack/openstack-manuals master: Remove checkbuild job  https://review.opendev.org/67695816:57
openstackgerritMerged openstack/openstack-manuals master: Simplify publishdocs et al  https://review.opendev.org/67696516:58
*** cmorpheus has joined #openstack-doc17:14
cmorpheusAJaeger: i just noticed that https://review.opendev.org/#/c/676969/1/doc/source/admin/identity-concepts.rst doesn't work17:14
cmorpheushttps://docs.openstack.org/keystone/latest/admin/manage_services17:14
openstackgerritMerged openstack/openstack-manuals stable/ocata: Switch output to publish-docs/html  https://review.opendev.org/67693117:23
openstackgerritMerged openstack/openstack-manuals stable/mitaka: Switch output to publish-docs/html  https://review.opendev.org/67694217:24
openstackgerritMerged openstack/openstack-manuals stable/newton: Switch output to publish-docs/html  https://review.opendev.org/67693517:24
*** alexmcleod has quit IRC17:26
openstackgerritAndreas Jaeger proposed openstack/openstack-manuals master: Add file list for promote-openstack-manuals-lang  https://review.opendev.org/67699317:31
*** pkovar has joined #openstack-doc17:55
*** pkovar has quit IRC18:01
openstackgerritMerged openstack/openstack-manuals master: Add file list for promote-openstack-manuals-lang  https://review.opendev.org/67699318:03
*** pcaruana has quit IRC20:57
*** KeithMnemonic has quit IRC21:02
*** vesper11 has quit IRC23:32

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