[parsing] - when generating tbpl urls, only use pgo for aurora and beta
jmaher opened this issue · 9 comments
an example is this-
we generate:
we want to generate:
https://tbpl.mozilla.org/?tree=Mozilla-Aurora&fromchange=caf1a925d7db&tochange=9b62596f6151&jobname=Windows%207%2032-bit%20mozilla-aurora%20pgo%20talos%20tp5o
but we don't have pgo for osx builds, so this is for branches = [Mozilla-Aurora, Mozilla-Beta] and platforms =[linux, linux64, winxp, win7, win8]
Hi,
Can you please elaborate a little bit more on this?
And I have a few things to discuss with you, so is it okay if I send you a mail at
jhamer@mozilla.com?
Thanks.
Hi,
I received your mail and have sent you one too. Please have a look. Also a little elaboration on this Issue would be great.
Thanks.
Hi Kaustabh,
At Mozilla we build opt (optimized), debug (with debug symbols, etc.) and pgo (program optimized). What we release as final releases is a PGO build. For our non trunk/integration branches we only build PGO.
With that background, we have two different branches for our builders, for inbound:
mozilla-inbound-non-pgo
mozilla-inbound
but this translates to the tbpl view as:
mozilla-inbound
mozilla-inbound-pgo
That is sort of flip flopped, but keep reading. On our aurora, beta, release branches we only build PGO, so the links to tbpl need to be mozilla-aurora-pgo and mozilla-beta-pgo.
For the branches that are listed in alert-manager database, if they have non-pgo in the branch name, then tbpl should load it fine as the branch name.
That is the background, the example above should help show what specific details need to be changed when the platform doesn't have a non-pgo in it.
Thank You.
And one more thing. Except for the url, the contents of the two links are the same. So how do I know that these are contents from two different URL?
everything will look to be the same except for the additional %20pgo in the case when we don't have a non-pgo.
Thanks. I am unto it. I will let you know if I face any more problems.
P.S. : Did you get my mail?
Hi,
I think I have solved the issue. I have created a pull request. Please do have a look.
Thanks.
this is fixed and deployed! Thanks for looking into this.