[prev in list] [next in list] [prev in thread] [next in thread]
List: subversion-dev
Subject: Re: Trunk buildbot broken
From: Daniel Sahlberg <daniel.l.sahlberg () gmail ! com>
Date: 2022-07-08 21:49:46
Message-ID: CAMHy98NHpauNpymVtkvzdN6qk2F2va5c_1kJiAACAA8oF94WLg () mail ! gmail ! com
[Download RAW message or body]
Den tors 30 juni 2022 kl 15:45 skrev Nathan Hartman <
hartman.nathan@gmail.com>:
> Recent breakage in the PO files wasn't detected by the buildbot so I
> went digging.
>
> The buildbot infrastructure changed recently. Builders are now listed at
> [1].
>
> [...]
> [1] all builders: https://ci2.apache.org/#/builders
>
>
Hijacking this thread, but in case anyone figure out the buildbots, I would
like to point out that the tools/dist/relase.py script (in
update_buildbot_config, called by the create-release-branch subcommand)
contains a reference to the old Infra repository for the buildbot
configurations [2] which, according to [3] is not used anymore.
The code seems to refer to a file subversion.conf which doesn't exist in
the new repository [4]. Looking at the code and the subversion.py file in
[4] I believe the could be as simple as
[[[
Index: dist/release.py
===================================================================
--- dist/release.py (revision 1902586)
+++ dist/release.py (working copy)
@@ -97,7 +97,7 @@
dist_release_url = dist_repos + '/release/subversion'
dist_archive_url = 'https://archive.apache.org/dist/subversion'
buildbot_repos = os.getenv('SVN_RELEASE_BUILDBOT_REPOS',
- '
https://svn.apache.org/repos/infra/infrastructure/buildbot/aegis/buildmaster
')
+ '
https://svn.apache.org/repos/infra/infrastructure/buildbot2/projects')
extns = ['zip', 'tar.gz', 'tar.bz2']
@@ -675,9 +675,9 @@
prev_ver = Version('1.%d.0' % (ver.minor - 1,))
next_ver = Version('1.%d.0' % (ver.minor + 1,))
- relpath = 'master1/projects/subversion.conf'
+ relpath = 'subversion.py'
edit_file(get_buildbot_wc_path(args.base_dir, relpath),
- r'(MINOR_LINES=\[.*%s)(\])' % (prev_ver.minor,),
+ r'(\s+MINOR_LINES = \[.*%s)(\])' % (prev_ver.minor,),
r'\1, %s\2' % (ver.minor,))
log_msg = '''\
]]]
I'll await some feedback on the above change, if someone feels an itch to
dig into the buildbots, feel free to commit the patch above if it is
reasonable.
Kind regards,
Daniel
[2]
https://svn.apache.org/repos/infra/infrastructure/buildbot/aegis/buildmaster
[3] https://svn.apache.org/repos/infra/infrastructure/buildbot/README
[4] https://svn.apache.org/repos/infra/infrastructure/buildbot2/projects/
Nathan recently made some inquiries regarding the buildbots. I would just
like to add this
[Attachment #3 (text/html)]
<div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div \
dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div \
dir="ltr">Den tors 30 juni 2022 kl 15:45 skrev Nathan Hartman <<a \
href="mailto:hartman.nathan@gmail.com">hartman.nathan@gmail.com</a>>:<br></div><div \
class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px \
0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Recent breakage in the \
PO files wasn't detected by the buildbot so I<br> went digging.<br>
<br>
The buildbot infrastructure changed recently. Builders are now listed at [1].<br>
<br></blockquote><div>[...]</div><div> </div><blockquote class="gmail_quote" \
style="margin:0px 0px 0px 0.8ex;border-left:1px solid \
rgb(204,204,204);padding-left:1ex">[1] all builders: <a \
href="https://ci2.apache.org/#/builders" rel="noreferrer" \
target="_blank">https://ci2.apache.org/#/builders</a><br> \
<br></blockquote><div><br></div><div>Hijacking this thread, but in case anyone figure \
out the buildbots, I would like to point out that the tools/dist/relase.py script (in \
update_buildbot_config, called by the create-release-branch subcommand) contains a \
reference to the old Infra repository for the buildbot configurations [2] which, \
according to [3] is not used anymore.</div><div><br></div><div>The code seems to \
refer to a file subversion.conf which doesn't exist in the new repository [4]. \
Looking at the code and the subversion.py file in [4] I believe the could be as \
simple as </div><div><br></div><div>[[[</div><div><div>Index: \
dist/release.py</div><div>===================================================================</div><div>--- \
dist/release.py (revision 1902586)</div><div>+++ dist/release.py \
(working copy)</div><div>@@ -97,7 +97,7 @@</div><div> dist_release_url = dist_repos \
+ '/release/subversion'</div><div> dist_archive_url = '<a \
href="https://archive.apache.org/dist/subversion">https://archive.apache.org/dist/subversion</a>'</div><div> \
buildbot_repos = os.getenv('SVN_RELEASE_BUILDBOT_REPOS',</div><div>- \
'<a href="https://svn.apache.org/repos/infra/infrastructure/buildbot/aegis/buildma \
ster">https://svn.apache.org/repos/infra/infrastructure/buildbot/aegis/buildmaster</a>')</div><div>+ \
'<a href="https://svn.apache.org/repos/infra/infrastructure/buildbot2/projects">ht \
tps://svn.apache.org/repos/infra/infrastructure/buildbot2/projects</a>')</div><div> \
extns = ['zip', 'tar.gz', \
'tar.bz2']</div><div><br></div><div><br></div><div>@@ -675,9 +675,9 \
@@</div><div> prev_ver = Version('1.%d.0' % (ver.minor - \
1,))</div><div> next_ver = Version('1.%d.0' % (ver.minor + \
1,))</div><div><br></div><div>- relpath = \
'master1/projects/subversion.conf'</div><div>+ relpath = \
'subversion.py'</div><div> \
edit_file(get_buildbot_wc_path(args.base_dir, relpath),</div><div>- \
r'(MINOR_LINES=\[.*%s)(\])' % (prev_ver.minor,),</div><div>+ \
r'(\s+MINOR_LINES = \[.*%s)(\])' % (prev_ver.minor,),</div><div> \
r'\1, %s\2' % (ver.minor,))</div><div><br></div><div> log_msg = \
'''\</div><div>]]]<br></div></div><div><br></div><div>I'll await some \
feedback on the above change, if someone feels an itch to dig into the buildbots, \
feel free to commit the patch above if it is \
reasonable.</div><div><br></div><div>Kind \
regards,</div><div>Daniel</div><div><br></div><div><br></div><div>[2] <a \
href="https://svn.apache.org/repos/infra/infrastructure/buildbot/aegis/buildmaster">ht \
tps://svn.apache.org/repos/infra/infrastructure/buildbot/aegis/buildmaster</a></div><div>[3] \
<a href="https://svn.apache.org/repos/infra/infrastructure/buildbot/README">https://svn.apache.org/repos/infra/infrastructure/buildbot/README</a></div><div>[4] \
<a href="https://svn.apache.org/repos/infra/infrastructure/buildbot2/projects/">https: \
//svn.apache.org/repos/infra/infrastructure/buildbot2/projects/</a></div><div><span \
style="color:rgb(0,0,0);white-space:pre-wrap"><br></span></div><div><br></div><div><br></div><div>Nathan \
recently made some inquiries regarding the buildbots. I would just like to add \
this</div><div><br></div><div> \
</div></div></div></div></div></div></div></div></div></div></div></div>
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic