Stream: t-compiler/wg-prioritization/alerts

Topic: I-prioritize #77410 regression 1.46 -> 1.47 big-endian back…


triagebot (Oct 01 2020 at 17:09, on Zulip):

@WG-prioritization/alerts issue #77410 has been requested for prioritization.

Procedure

LeSeulArtichaut (Oct 01 2020 at 17:11, on Zulip):

Those are all tier 2 platforms

Joshua Nelson (Oct 01 2020 at 17:12, on Zulip):

I'd say P-high maybe?

LeSeulArtichaut (Oct 01 2020 at 17:13, on Zulip):

Seems too high for me but maybe I’m underestimating the importance of backtraces

Camelid (Oct 01 2020 at 18:35, on Zulip):

I've had issues running UI tests on macOS x64... maybe I should open an issue

LeSeulArtichaut (Oct 01 2020 at 18:50, on Zulip):

@Camelid Failures on macOS should have been caught by bors though

Camelid (Oct 01 2020 at 18:51, on Zulip):

Hmm...

Camelid (Oct 01 2020 at 18:52, on Zulip):

I'm running a build right now to see what the exact tests were

Camelid (Oct 01 2020 at 19:59, on Zulip):

@LeSeulArtichaut Yep, two tests failed. Opened #77424.

Camelid (Oct 01 2020 at 20:02, on Zulip):

LeSeulArtichaut said:

Camelid Failures on macOS should have been caught by bors though

I think macOS builds were turned off.

LeSeulArtichaut (Oct 01 2020 at 20:03, on Zulip):

Oh

Camelid (Oct 01 2020 at 20:04, on Zulip):

Yeah :/

LeSeulArtichaut (Oct 01 2020 at 20:04, on Zulip):

Weren’t they disabled only on GitHub Actions and enabled on Azure, or something like that?

LeSeulArtichaut (Oct 01 2020 at 20:05, on Zulip):

Well we can probably ask someone from T-infra I think, cc @simulacrum

Camelid (Oct 01 2020 at 20:06, on Zulip):

See #77423's checks; I only see Ubuntu:

image.png

LeSeulArtichaut (Oct 01 2020 at 20:23, on Zulip):

Camelid said:

See #77423's checks; I only see Ubuntu:

image.png

That's only CI checks, only bors runs the full testsuite on all tier 1 targets when merging the PR

Camelid (Oct 01 2020 at 21:20, on Zulip):

I learned something new about bors today!

Camelid (Oct 01 2020 at 21:21, on Zulip):

Yeah, it does seem that macOS is being run in Azure...

DPC (Oct 02 2020 at 01:25, on Zulip):

yeah CI is "minimal", while bors at the end runs the entire testsuite across all targets which is why PRs take 2-3 hours to get merged :smile:

apiraino (Oct 07 2020 at 10:42, on Zulip):

Should the T-libs team be pinged about this? Or perhaps someone else?

Santiago Pastorino (Oct 07 2020 at 19:08, on Zulip):

this one needs prioritization

Santiago Pastorino (Oct 07 2020 at 19:08, on Zulip):

I'd say P-medium?

apiraino (Oct 07 2020 at 19:33, on Zulip):

do we have an idea about the right priority? ping someone, maybe? or maybe give a P-high/I-nominate to have the team briefly talk about that?

Camelid (Oct 07 2020 at 19:37, on Zulip):

I think P-high and a nominate sounds good

Camelid (Oct 07 2020 at 19:38, on Zulip):

Thoughts @Santiago Pastorino?

Santiago Pastorino (Oct 07 2020 at 19:39, on Zulip):

my first reaction was similar to @LeSeulArtichaut's one

Santiago Pastorino (Oct 07 2020 at 19:40, on Zulip):

but anyway, if 2 of us think this is P-high and 2 others thinks it is P-medium I'd rather label it as P-high

Santiago Pastorino (Oct 07 2020 at 19:41, on Zulip):

because if you people are right and we label this one as P-medium we won't be ringing the right alarms by labelling with a lower priority

Santiago Pastorino (Oct 07 2020 at 19:42, on Zulip):

@Camelid if you can go ahead and tag it with P-high then :+1:

Camelid (Oct 07 2020 at 19:42, on Zulip):

I don't know much about the backtrace code, I just think it might make sense to have higher priority so someone who knows looks into it :smile:

Camelid (Oct 07 2020 at 19:42, on Zulip):

Okay, sure

Camelid (Oct 07 2020 at 19:42, on Zulip):

Nominate too?

Santiago Pastorino (Oct 07 2020 at 19:42, on Zulip):

yeah :+1:

triagebot (Oct 07 2020 at 19:43, on Zulip):

Issue #77410's prioritization request has been removed.

Santiago Pastorino (Oct 07 2020 at 19:43, on Zulip):

we may need to see how much stuff we have in the agenda but still nominations go last so ... :+1:

apiraino (Oct 08 2020 at 15:07, on Zulip):

So, the T-compiler didnt have a lot to say about this issue. @Santiago Pastorino I see the issue is tagged with O-PowerPC O-SPARC O-SystemZ. Who else should be ping more relevant to the issue scope?

Santiago Pastorino (Oct 08 2020 at 15:18, on Zulip):

we can leave it nominated for next week

apiraino (Oct 08 2020 at 15:24, on Zulip):

ok

Last update: Apr 11 2021 at 19:30UTC