public inbox for intel-gfx@lists.freedesktop.org
 help / color / mirror / Atom feed
* [PATCH i-g-t] intel-ci: Remove kms_psr_sink_crc@psr_basic from BAT
@ 2016-10-12 12:13 Petri Latvala
  2016-10-12 16:30 ` Jani Nikula
  2016-10-13  7:40 ` Daniel Vetter
  0 siblings, 2 replies; 3+ messages in thread
From: Petri Latvala @ 2016-10-12 12:13 UTC (permalink / raw)
  To: intel-gfx

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
 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

^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [PATCH i-g-t] intel-ci: Remove kms_psr_sink_crc@psr_basic from BAT
  2016-10-12 12:13 [PATCH i-g-t] intel-ci: Remove kms_psr_sink_crc@psr_basic from BAT Petri Latvala
@ 2016-10-12 16:30 ` Jani Nikula
  2016-10-13  7:40 ` Daniel Vetter
  1 sibling, 0 replies; 3+ messages in thread
From: Jani Nikula @ 2016-10-12 16:30 UTC (permalink / raw)
  To: Petri Latvala, intel-gfx

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

-- 
Jani Nikula, Intel Open Source Technology Center
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH i-g-t] intel-ci: Remove kms_psr_sink_crc@psr_basic from BAT
  2016-10-12 12:13 [PATCH i-g-t] intel-ci: Remove kms_psr_sink_crc@psr_basic from BAT Petri Latvala
  2016-10-12 16:30 ` Jani Nikula
@ 2016-10-13  7:40 ` Daniel Vetter
  1 sibling, 0 replies; 3+ messages in thread
From: Daniel Vetter @ 2016-10-13  7:40 UTC (permalink / raw)
  To: Petri Latvala, Rodrigo Vivi, Jim Bride; +Cc: intel-gfx

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



-- 
Daniel Vetter
Software Engineer, Intel Corporation
+41 (0) 79 365 57 48 - http://blog.ffwll.ch
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2016-10-13  7:40 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-10-12 12:13 [PATCH i-g-t] intel-ci: Remove kms_psr_sink_crc@psr_basic from BAT Petri Latvala
2016-10-12 16:30 ` Jani Nikula
2016-10-13  7:40 ` Daniel Vetter

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox