Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

bb.org: correct tarbuildnum #94

Closed
wants to merge 1 commit into from

Conversation

grooverdan
Copy link
Member

This probably causing errors like https://buildbot.mariadb.org/#/builders/219/builds/1255/steps/3/logs/stdio

sudo sh -c 'echo '\''deb [trusted=yes] https://ci.mariadb.org/1255/amd64-ubuntu-1804-deb-autobake/debs .'\'' >> /etc/apt/sources.list'
....
+ sudo apt-get update
E: Malformed entry 55 in list file /etc/apt/sources.list (Component)
E: The list of sources could not be read.

tarbuildnum is in the scripts (top of https://github.com/MariaDB/mariadb.org-tools/blob/master/buildbot.mariadb.org/scripts/deb-major-upgrade.sh#L7).

I'd like a quick check/deployment monitoring to ensure this is the only impact.

Copy link
Contributor

@vladbogo vladbogo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This seems to be the problem:

d['tarbuildnum'] = util.Interpolate("%(prop:buildnumber)s")

but I'm not sure if it has implications on other scripts.

f_tarball.addStep(steps.Trigger(schedulerNames=['s_upstream_all'], waitForFinish=False, updateSourceStamp=False,
set_properties={"tarbuildnum" : Property("buildnumber"), "mariadb_version" : Property("mariadb_version"), "master_branch" : Property("master_branch")}))
set_properties={"tarbuildnum" : Property("tarbuildnum"), "mariadb_version" : Property("mariadb_version"), "master_branch" : Property("master_branch")}))
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The tarbuildnum is correctly set without any changes. Based on this PR it will be undefined since the tarball builder doesn't have it set. So, we need to set tarbuildnum as buildnumber. This happens in the tarball builder and sets the property tarbuildnum for all triggered builders with the current buildnumer of the tarball builder.

While indeed in the log, there seems to be an issue, looking at the same build that you point, you can see that the tarbuildnum is correctly set (see below)

Screenshot 2022-01-06 at 16 09 26

So, I susptect to be an issue where the deb scripts looks at the buildnumber and not at the tarbuildnum

@grooverdan
Copy link
Member Author

thanks, I'll keep looking.

@grooverdan grooverdan closed this Jan 6, 2022
@grooverdan
Copy link
Member Author

Found 1a25399 and committed.

@grooverdan grooverdan deleted the tarbuildnum_prop branch January 10, 2022 04:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants