XSF Editor Team - 2023-01-26


  1. emus has left

  2. moparisthebest has left

  3. moparisthebest has joined

  4. Tobi has joined

  5. soul has left

  6. Tobi has left

  7. Tobi has joined

  8. soul has joined

  9. moparisthebest has left

  10. moparisthebest has joined

  11. soul has left

  12. soul has joined

  13. emus has joined

  14. Millesimus has left

  15. Millesimus has joined

  16. wurstsalat has joined

  17. MSavoritias (fae,ve) has joined

  18. soul has left

  19. soul has joined

  20. Kev has joined

  21. emus has left

  22. emus has joined

  23. Kev

    Right, no sign of the published XEPs. Anyone know how they're deployed? The docs say https://hub.docker.com/r/xmppxsf/xeps/builds/ but that seems to not be true.

  24. jonas’

    indeed

  25. jonas’

    you need to build that image locally and push it to some obscure registry hosted on gitlab.com

  26. jonas’

    for which you probably don't have +w

  27. Kev

    Just found the repo in the create scripts, at least, ta.

  28. Kev

    Ok, I'll adapt that slightly, for the moment.

  29. jonas’

    sorry for that, that's an artificat of the failed attempt to streamline stuff over at gitlab.com

  30. jonas’

    sorry for that, that's an artifact of the failed attempt to streamline stuff over at gitlab.com

  31. Kev

    Not a problem, I just need to Do A Thing.

  32. Kev

    I don't see a lot of value in me fighting with a registry to upload a manual image when I could just upload directly to eos2, so I'm going to do that.

  33. jonas’

    how would you upload to eos2 directly?

  34. Kev

    docker save ... | gzip | pv | ssh eos2 docker load

  35. jonas’

    right

  36. jonas’

    I never do that because it doesn't dedupliate the lower layer

  37. jonas’

    which matters if you have 1 MB/s uplink :D

  38. Kev

    Maybe it's a stupid idea and I'm about to break everything. I guess we'll find out in a momen ;)

  39. Kev

    Maybe it's a stupid idea and I'm about to break everything. I guess we'll find out in a moment :)

  40. jonas’

    no, sounds about right

  41. Kev

    Alright, I now have a local script to do the build, upload and deploy.

  42. Kev

    Next step, I guess, working out if any cron or whatever on eos2 is going to overwrite it with the old version.

  43. Kev

    Oh, good, it looks like there's multiple things trying to serve xeps.

  44. jonas’

    yes, there is a cronjob

  45. jonas’

    I re-enabled it the other day

  46. Kev

    Worse, there's a cronjob that's serving xeps in a different manner.

  47. jonas’

    uhuh?

  48. jonas’

    did I mess that up?

  49. jonas’

    (maybe iteam@?

  50. jonas’

    (maybe iteam@?)

  51. Kev

    I don't know.

  52. jonas’

    (maybe move this to iteam@?)

  53. pep. has left

  54. pep. has joined

  55. soul has left

  56. soul has joined

  57. Kev

    Right, looks like I'm now set up to publish XEPs and attic in one go.

  58. pep. has left

  59. pep. has joined

  60. jonas’

    woop woop

  61. jonas’

    you probably won't continue my commit message history in the attic though

  62. Kev

    I did not.

  63. Kev

    I looked at it. I thought 'that's cute', and then continued to be lazy.

  64. me9 has joined

  65. Link Mauve has left

  66. Kev has left

  67. Kev has joined

  68. moparisthebest

    jonas’, Kev: should this eos2 instead run something like https://github.com/adnanh/webhook so merging an MR could tell it to pull and rebuild?

  69. Kev

    I'm entirely happy with doing the docker build locally, TBH.

  70. Kev

    It's one command I need to run to get it built and deployed.

  71. Kev

    Unless I'm missing something, the tooling once the PRs are ready to merge is pretty ok, it's all the stuff before that that's nuisanceful.

  72. Kev

    Next time I do a deploy I'll add a couple of extra lines to my local shell script, and everything'll be fine from that point of view.

  73. Kev

    All the checks when triaging is a nuisance.

  74. moparisthebest

    > Next time I do a deploy I'll add a couple of extra lines to my local shell script, and everything'll be fine from that point of view. But only for you, not for any other or future editors

  75. Kev

    I'll share my script ;)

  76. moparisthebest

    Plus all editors have to also be in iteam?

  77. Kev

    The deployment's automatic.

  78. Kev

    Anyway, I'm not saying we can't automate this bit further. Just that it's not the pain point.

  79. moparisthebest

    Right others seem higher priority, just seems a bit like death by a thousand cuts

  80. Kev

    At least for me, the deployment now of xeps and attics is fairly painless (as long as I don't discover gremlins next time).But the bit where we interact with people on PRs is...not painless :)

  81. moparisthebest

    If you want to review my triage.sh PR feel free :D

  82. Kev

    I've been mostly leaving MattJ for that, as he's got a better grip of where we're going with tooling, but I'll take a look, thanks.

  83. tmolitor has left

  84. tmolitor has joined

  85. jonas’

    moparisthebest, except for the missing approver thing, lgtm

  86. Link Mauve has joined

  87. Tobi has left

  88. Tobi has joined

  89. Tobi has left

  90. Tobi has joined

  91. Tobi has left

  92. Tobi has joined

  93. Tobi has left

  94. Tobi has joined

  95. stpeter has joined

  96. Tobi has left

  97. Tobi has joined

  98. Tobi has left

  99. Tobi has joined

  100. moparisthebest

    jonas’: trying to fix that in that other MR, applying approver to all of them, then this can just be a hard error

  101. moparisthebest

    Thanks for reviewing!

  102. Tobi has left

  103. Tobi has joined

  104. Kev

    I haven't looked, but does this error because there are XEPs that don't have approver, or error if a *changed* XEP doesn't have approver?

  105. Kev

    I think the latter would be ok, it's a forcing function.

  106. Zash has joined

  107. Zash

    Hello, I'd just like to be sure https://xmpp.org/extensions/inbox/xep-sla.html doesn't fall between the chair shuffle of the new council and editor constellation. https://logs.xmpp.org/council/2023-01-04?p=h#2023-01-04-90f98d9660c9c364

  108. soul has left

  109. stpeter has left

  110. soul has joined

  111. moparisthebest

    Kev: it does different things based on whether the approver of the changed xep is council or board, and has an ugly "unknown" branch that should be deleted imho

  112. moparisthebest

    But gotta make sure they all have them before that can be done

  113. stpeter has joined

  114. MSavoritias (fae,ve) has left

  115. stpeter has left

  116. me9 has left

  117. stpeter has joined

  118. Millesimus has left

  119. Millesimus has joined

  120. Millesimus has left

  121. Millesimus has joined

  122. Tobi has left

  123. stpeter has left

  124. Millesimus has left

  125. Tobi has joined

  126. Kev has left

  127. Millesimus has joined

  128. Tobi has left

  129. Daniel has left

  130. Millesimus has left

  131. Daniel has joined

  132. Millesimus has joined