diff mbox

[i-g-t] intel-ci: Remove kms_psr_sink_crc@psr_basic from BAT

Message ID 1476274380-2121-1-git-send-email-petri.latvala@intel.com (mailing list archive)
State New, archived
Headers show

Commit Message

Petri Latvala Oct. 12, 2016, 12:13 p.m. UTC
The test is producing a lot of CI noise.

Signed-off-by: Petri Latvala <petri.latvala@intel.com>
---

Will be pushed shortly. Visible in CI results tomorrow-ish.


tests/intel-ci/fast-feedback.testlist | 1 -
 1 file changed, 1 deletion(-)

Comments

Jani Nikula Oct. 12, 2016, 4:30 p.m. UTC | #1
On Wed, 12 Oct 2016, Petri Latvala <petri.latvala@intel.com> wrote:
> The test is producing a lot of CI noise.
>
> Signed-off-by: Petri Latvala <petri.latvala@intel.com>
> ---
>
> Will be pushed shortly. Visible in CI results tomorrow-ish.
>
>
> tests/intel-ci/fast-feedback.testlist | 1 -

I know this is the completely wrong patch to discuss this... but anyway,
why is BAT == fast-feedback.testlist, instead of, you know,
bat.testlist?

BR,
Jani.



>  1 file changed, 1 deletion(-)
>
> diff --git a/tests/intel-ci/fast-feedback.testlist b/tests/intel-ci/fast-feedback.testlist
> index ae94381..e2fc9ac 100644
> --- a/tests/intel-ci/fast-feedback.testlist
> +++ b/tests/intel-ci/fast-feedback.testlist
> @@ -208,7 +208,6 @@ igt@kms_pipe_crc_basic@read-crc-pipe-c-frame-sequence
>  igt@kms_pipe_crc_basic@suspend-read-crc-pipe-a
>  igt@kms_pipe_crc_basic@suspend-read-crc-pipe-b
>  igt@kms_pipe_crc_basic@suspend-read-crc-pipe-c
> -igt@kms_psr_sink_crc@psr_basic
>  igt@kms_setmode@basic-clone-single-crtc
>  igt@kms_sink_crc_basic
>  igt@pm_backlight@basic-brightness
Daniel Vetter Oct. 13, 2016, 7:40 a.m. UTC | #2
On Wed, Oct 12, 2016 at 2:13 PM, Petri Latvala <petri.latvala@intel.com> wrote:
> The test is producing a lot of CI noise.
>
> Signed-off-by: Petri Latvala <petri.latvala@intel.com>
> ---
>
> Will be pushed shortly. Visible in CI results tomorrow-ish.
>
>
> tests/intel-ci/fast-feedback.testlist | 1 -
>  1 file changed, 1 deletion(-)
>
> diff --git a/tests/intel-ci/fast-feedback.testlist b/tests/intel-ci/fast-feedback.testlist
> index ae94381..e2fc9ac 100644
> --- a/tests/intel-ci/fast-feedback.testlist
> +++ b/tests/intel-ci/fast-feedback.testlist
> @@ -208,7 +208,6 @@ igt@kms_pipe_crc_basic@read-crc-pipe-c-frame-sequence
>  igt@kms_pipe_crc_basic@suspend-read-crc-pipe-a
>  igt@kms_pipe_crc_basic@suspend-read-crc-pipe-b
>  igt@kms_pipe_crc_basic@suspend-read-crc-pipe-c
> -igt@kms_psr_sink_crc@psr_basic

This landed without an ack from Rodrigo and Jim as the engineers.
That's not good at all, because the point of this list was that we
have more visibility in what CI does. Adding Rodrigo and Jim as fyi.

More generally I think we need to have full-on review requirement for
igt patches too, to avoid this kind of fun. Sneaking stuff in like
this is a no-go.
-Daniel


>  igt@kms_setmode@basic-clone-single-crtc
>  igt@kms_sink_crc_basic
>  igt@pm_backlight@basic-brightness
> --
> 2.9.3
>
> _______________________________________________
> Intel-gfx mailing list
> Intel-gfx@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/intel-gfx
diff mbox

Patch

diff --git a/tests/intel-ci/fast-feedback.testlist b/tests/intel-ci/fast-feedback.testlist
index ae94381..e2fc9ac 100644
--- a/tests/intel-ci/fast-feedback.testlist
+++ b/tests/intel-ci/fast-feedback.testlist
@@ -208,7 +208,6 @@  igt@kms_pipe_crc_basic@read-crc-pipe-c-frame-sequence
 igt@kms_pipe_crc_basic@suspend-read-crc-pipe-a
 igt@kms_pipe_crc_basic@suspend-read-crc-pipe-b
 igt@kms_pipe_crc_basic@suspend-read-crc-pipe-c
-igt@kms_psr_sink_crc@psr_basic
 igt@kms_setmode@basic-clone-single-crtc
 igt@kms_sink_crc_basic
 igt@pm_backlight@basic-brightness