* [igt-dev] [PATCH] Chamelium: Get Chamelium Logs on RPC failure.
@ 2023-02-01 18:12 Mark Yacoub
2023-02-01 18:24 ` [igt-dev] ✗ GitLab.Pipeline: warning for " Patchwork
2023-02-01 18:49 ` [igt-dev] ✗ Fi.CI.BAT: failure " Patchwork
0 siblings, 2 replies; 6+ messages in thread
From: Mark Yacoub @ 2023-02-01 18:12 UTC (permalink / raw)
To: igt-dev
Cc: petri.latvala, ihf, amstan, seanpaul, matthewtlam, markyacoub,
khaled.almahallawy
From: Mark Yacoub <markyacoub@chromium.org>
[Why]
Currently, Chamelium acts like a black box, we can't tell what is going on there when a failure occur. Seeing its logs will allow us to debug better.
[How]
On chamelium_rpc failure, print out the logs in debug so it wouldn't clutter good tests.
Signed-off-by: Mark Yacoub <markyacoub@chromium.org>
---
lib/igt_chamelium.c | 38 ++++++++++++++++++++++++--
lib/igt_chamelium.h | 1 +
tests/chamelium/kms_chamelium_helper.c | 5 ++++
3 files changed, 41 insertions(+), 3 deletions(-)
diff --git a/lib/igt_chamelium.c b/lib/igt_chamelium.c
index a235f3c8..65531671 100644
--- a/lib/igt_chamelium.c
+++ b/lib/igt_chamelium.c
@@ -658,6 +658,8 @@ static xmlrpc_value *chamelium_rpc(struct chamelium *chamelium,
xmlrpc_value *res;
va_list va_args;
int fsm_trials_left = 5;
+ bool did_fault_occur = false;
+ char fault_string[1024];
if (strcmp(method_name, "CaptureVideo") == 0
|| strcmp(method_name, "StartCapturingVideo") == 0) {
@@ -680,9 +682,23 @@ static xmlrpc_value *chamelium_rpc(struct chamelium *chamelium,
format_str, va_args);
va_end(va_args);
}
- igt_assert_f(!chamelium->env.fault_occurred,
- "Chamelium RPC call[%s] failed: %s\n", method_name,
- chamelium->env.fault_string);
+
+ did_fault_occur = chamelium->env.fault_occurred;
+ if (did_fault_occur) {
+ // Save the fault string before getting the logs which will
+ // clear the string if it is works.
+ strncpy(fault_string, chamelium->env.fault_string, 1024);
+ // To avoid recursive calls for logs, don't call for logs if we
+ // fail on getting the logs.
+ if (strcmp(method_name, "GetChameleondLogs") != 0) {
+ char *logs = chamelium_get_logs(chamelium);
+ igt_debug("CHAMELIUM LOGS:\n%s\n", logs);
+ free(logs);
+ }
+ }
+
+ igt_assert_f(!did_fault_occur, "Chamelium RPC call [%s] failed: %s\n",
+ method_name, fault_string);
return res;
}
@@ -737,6 +753,22 @@ void chamelium_assert_reachable(struct chamelium *chamelium, int timeout)
"Couldn't connect to Chamelium for %ds", timeout);
}
+char *chamelium_get_logs(struct chamelium *chamelium)
+{
+ xmlrpc_value *res;
+ const char *logs = NULL;
+
+ igt_debug(
+ "Calling GetChameleondLogs - Logs returned are from the last time "
+ "this was called.\n");
+
+ res = chamelium_rpc(chamelium, NULL, "GetChameleondLogs", "(s)", "IGT");
+ xmlrpc_read_string(&chamelium->env, res, &logs);
+ xmlrpc_DECREF(res);
+
+ return logs;
+}
+
/**
* chamelium_plug:
* @chamelium: The Chamelium instance to use
diff --git a/lib/igt_chamelium.h b/lib/igt_chamelium.h
index d979de4a..e9096519 100644
--- a/lib/igt_chamelium.h
+++ b/lib/igt_chamelium.h
@@ -159,6 +159,7 @@ chamelium_reset_state(igt_display_t *display,
bool chamelium_wait_reachable(struct chamelium *chamelium, int timeout);
void chamelium_assert_reachable(struct chamelium *chamelium, int timeout);
+char *chamelium_get_logs(struct chamelium *chamelium);
void chamelium_plug(struct chamelium *chamelium, struct chamelium_port *port);
void chamelium_unplug(struct chamelium *chamelium, struct chamelium_port *port);
bool chamelium_is_plugged(struct chamelium *chamelium,
diff --git a/tests/chamelium/kms_chamelium_helper.c b/tests/chamelium/kms_chamelium_helper.c
index 197d29be..f44cbec2 100644
--- a/tests/chamelium/kms_chamelium_helper.c
+++ b/tests/chamelium/kms_chamelium_helper.c
@@ -24,6 +24,7 @@
* Lyude Paul <lyude@redhat.com>
*/
+#include "igt_chamelium.h"
#include "igt_edid.h"
#include "kms_chamelium_helper.h"
@@ -47,6 +48,10 @@ void chamelium_init_test(chamelium_data_t *data)
/* we need to initalize chamelium after igt_display_require */
data->chamelium = chamelium_init(data->drm_fd, &data->display);
igt_require(data->chamelium);
+ // Get the logs so we can reset the chamelium logs at this cursor.
+ // The logs are then retrieved when any call fails so we can debug
+ // chamelium if needed.
+ free(chamelium_get_logs(data->chamelium));
data->ports = chamelium_get_ports(data->chamelium, &data->port_count);
--
2.39.1.456.gfc5497dd1b-goog
^ permalink raw reply related [flat|nested] 6+ messages in thread
* [igt-dev] ✗ GitLab.Pipeline: warning for Chamelium: Get Chamelium Logs on RPC failure.
2023-02-01 18:12 [igt-dev] [PATCH] Chamelium: Get Chamelium Logs on RPC failure Mark Yacoub
@ 2023-02-01 18:24 ` Patchwork
2023-02-01 18:49 ` [igt-dev] ✗ Fi.CI.BAT: failure " Patchwork
1 sibling, 0 replies; 6+ messages in thread
From: Patchwork @ 2023-02-01 18:24 UTC (permalink / raw)
To: Mark Yacoub; +Cc: igt-dev
== Series Details ==
Series: Chamelium: Get Chamelium Logs on RPC failure.
URL : https://patchwork.freedesktop.org/series/113567/
State : warning
== Summary ==
Pipeline status: FAILED.
see https://gitlab.freedesktop.org/gfx-ci/igt-ci-tags/-/pipelines/798228 for the overview.
build:tests-debian-meson has failed (https://gitlab.freedesktop.org/gfx-ci/igt-ci-tags/-/jobs/35709867):
../tests/chamelium/kms_chamelium_helper.c:338:7: warning: nested extern declaration of ‘drmModeGetPropertyBlob’ [-Wnested-externs]
drmModeGetPropertyBlob(data->drm_fd, edid_blob_id));
^~~~~~~~~~~~~~~~~~~~~~
../lib/igt_core.h:676:13: note: in definition of macro ‘igt_assert’
do { if (!(expr)) \
^~~~
../tests/chamelium/kms_chamelium_helper.c:337:23: warning: assignment to ‘drmModePropertyBlobPtr’ {aka ‘struct _drmModePropertyBlob *’} from ‘int’ makes pointer from integer without a cast [-Wint-conversion]
igt_assert(edid_blob =
^
../lib/igt_core.h:676:13: note: in definition of macro ‘igt_assert’
do { if (!(expr)) \
^~~~
cc1: some warnings being treated as errors
ninja: build stopped: subcommand failed.
section_end:1675275461:step_script
section_start:1675275461:cleanup_file_variables
Cleaning up project directory and file based variables
section_end:1675275461:cleanup_file_variables
ERROR: Job failed: exit code 1
build:tests-fedora has failed (https://gitlab.freedesktop.org/gfx-ci/igt-ci-tags/-/jobs/35709862):
../tests/chamelium/kms_chamelium_helper.c:338:7: warning: nested extern declaration of ‘drmModeGetPropertyBlob’ [-Wnested-externs]
338 | drmModeGetPropertyBlob(data->drm_fd, edid_blob_id));
| ^~~~~~~~~~~~~~~~~~~~~~
../lib/igt_core.h:676:13: note: in definition of macro ‘igt_assert’
676 | do { if (!(expr)) \
| ^~~~
../tests/chamelium/kms_chamelium_helper.c:337:23: warning: assignment to ‘drmModePropertyBlobPtr’ {aka ‘struct _drmModePropertyBlob *’} from ‘int’ makes pointer from integer without a cast [-Wint-conversion]
337 | igt_assert(edid_blob =
| ^
../lib/igt_core.h:676:13: note: in definition of macro ‘igt_assert’
676 | do { if (!(expr)) \
| ^~~~
cc1: some warnings being treated as errors
ninja: build stopped: subcommand failed.
section_end:1675275465:step_script
section_start:1675275465:cleanup_file_variables
Cleaning up project directory and file based variables
section_end:1675275466:cleanup_file_variables
ERROR: Job failed: exit code 1
build:tests-fedora-clang has failed (https://gitlab.freedesktop.org/gfx-ci/igt-ci-tags/-/jobs/35709866):
uint32_t bpp;
^
/usr/include/xf86drmMode.h:221:2: error: unknown type name 'uint32_t'
uint32_t depth;
^
/usr/include/xf86drmMode.h:223:2: error: unknown type name 'uint32_t'
uint32_t handle;
^
/usr/include/xf86drmMode.h:229:2: error: unknown type name 'uint32_t'
uint32_t id;
^
fatal error: too many errors emitted, stopping now [-ferror-limit=]
20 errors generated.
ninja: build stopped: subcommand failed.
section_end:1675275530:step_script
section_start:1675275530:cleanup_file_variables
Cleaning up project directory and file based variables
section_end:1675275532:cleanup_file_variables
ERROR: Job failed: exit code 1
build:tests-fedora-no-libdrm-nouveau has failed (https://gitlab.freedesktop.org/gfx-ci/igt-ci-tags/-/jobs/35709865):
../tests/chamelium/kms_chamelium_helper.c:338:7: warning: nested extern declaration of ‘drmModeGetPropertyBlob’ [-Wnested-externs]
338 | drmModeGetPropertyBlob(data->drm_fd, edid_blob_id));
| ^~~~~~~~~~~~~~~~~~~~~~
../lib/igt_core.h:676:13: note: in definition of macro ‘igt_assert’
676 | do { if (!(expr)) \
| ^~~~
../tests/chamelium/kms_chamelium_helper.c:337:23: warning: assignment to ‘drmModePropertyBlobPtr’ {aka ‘struct _drmModePropertyBlob *’} from ‘int’ makes pointer from integer without a cast [-Wint-conversion]
337 | igt_assert(edid_blob =
| ^
../lib/igt_core.h:676:13: note: in definition of macro ‘igt_assert’
676 | do { if (!(expr)) \
| ^~~~
cc1: some warnings being treated as errors
ninja: build stopped: subcommand failed.
section_end:1675275467:step_script
section_start:1675275467:cleanup_file_variables
Cleaning up project directory and file based variables
section_end:1675275468:cleanup_file_variables
ERROR: Job failed: exit code 1
build:tests-fedora-no-libunwind has failed (https://gitlab.freedesktop.org/gfx-ci/igt-ci-tags/-/jobs/35709863):
../tests/chamelium/kms_chamelium_helper.c:338:7: warning: nested extern declaration of ‘drmModeGetPropertyBlob’ [-Wnested-externs]
338 | drmModeGetPropertyBlob(data->drm_fd, edid_blob_id));
| ^~~~~~~~~~~~~~~~~~~~~~
../lib/igt_core.h:676:13: note: in definition of macro ‘igt_assert’
676 | do { if (!(expr)) \
| ^~~~
../tests/chamelium/kms_chamelium_helper.c:337:23: warning: assignment to ‘drmModePropertyBlobPtr’ {aka ‘struct _drmModePropertyBlob *’} from ‘int’ makes pointer from integer without a cast [-Wint-conversion]
337 | igt_assert(edid_blob =
| ^
../lib/igt_core.h:676:13: note: in definition of macro ‘igt_assert’
676 | do { if (!(expr)) \
| ^~~~
cc1: some warnings being treated as errors
ninja: build stopped: subcommand failed.
section_end:1675275465:step_script
section_start:1675275465:cleanup_file_variables
Cleaning up project directory and file based variables
section_end:1675275467:cleanup_file_variables
ERROR: Job failed: exit code 1
build:tests-fedora-oldest-meson has failed (https://gitlab.freedesktop.org/gfx-ci/igt-ci-tags/-/jobs/35709864):
../tests/chamelium/kms_chamelium_helper.c:338:7: warning: nested extern declaration of ‘drmModeGetPropertyBlob’ [-Wnested-externs]
338 | drmModeGetPropertyBlob(data->drm_fd, edid_blob_id));
| ^~~~~~~~~~~~~~~~~~~~~~
../lib/igt_core.h:676:13: note: in definition of macro ‘igt_assert’
676 | do { if (!(expr)) \
| ^~~~
../tests/chamelium/kms_chamelium_helper.c:337:23: warning: assignment to ‘drmModePropertyBlobPtr’ {aka ‘struct _drmModePropertyBlob *’} from ‘int’ makes pointer from integer without a cast [-Wint-conversion]
337 | igt_assert(edid_blob =
| ^
../lib/igt_core.h:676:13: note: in definition of macro ‘igt_assert’
676 | do { if (!(expr)) \
| ^~~~
cc1: some warnings being treated as errors
ninja: build stopped: subcommand failed.
section_end:1675275466:step_script
section_start:1675275466:cleanup_file_variables
Cleaning up project directory and file based variables
section_end:1675275467:cleanup_file_variables
ERROR: Job failed: exit code 1
== Logs ==
For more details see: https://gitlab.freedesktop.org/gfx-ci/igt-ci-tags/-/pipelines/798228
^ permalink raw reply [flat|nested] 6+ messages in thread
* [igt-dev] ✗ Fi.CI.BAT: failure for Chamelium: Get Chamelium Logs on RPC failure.
2023-02-01 18:12 [igt-dev] [PATCH] Chamelium: Get Chamelium Logs on RPC failure Mark Yacoub
2023-02-01 18:24 ` [igt-dev] ✗ GitLab.Pipeline: warning for " Patchwork
@ 2023-02-01 18:49 ` Patchwork
2023-02-01 21:02 ` Mark Yacoub
1 sibling, 1 reply; 6+ messages in thread
From: Patchwork @ 2023-02-01 18:49 UTC (permalink / raw)
To: Mark Yacoub; +Cc: igt-dev
[-- Attachment #1: Type: text/plain, Size: 3589 bytes --]
== Series Details ==
Series: Chamelium: Get Chamelium Logs on RPC failure.
URL : https://patchwork.freedesktop.org/series/113567/
State : failure
== Summary ==
CI Bug Log - changes from CI_DRM_12680 -> IGTPW_8428
====================================================
Summary
-------
**FAILURE**
Serious unknown changes coming with IGTPW_8428 absolutely need to be
verified manually.
If you think the reported changes have nothing to do with the changes
introduced in IGTPW_8428, please notify your bug team to allow them
to document this new failure mode, which will reduce false positives in CI.
External URL: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8428/index.html
Participating hosts (26 -> 25)
------------------------------
Missing (1): fi-snb-2520m
Possible new issues
-------------------
Here are the unknown changes that may have been introduced in IGTPW_8428:
### IGT changes ###
#### Possible regressions ####
* igt@kms_chamelium_frames@hdmi-crc-fast:
- fi-kbl-7567u: [PASS][1] -> [FAIL][2] +3 similar issues
[1]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12680/fi-kbl-7567u/igt@kms_chamelium_frames@hdmi-crc-fast.html
[2]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8428/fi-kbl-7567u/igt@kms_chamelium_frames@hdmi-crc-fast.html
#### Warnings ####
* igt@kms_chamelium_hpd@vga-hpd-fast:
- fi-kbl-7567u: [SKIP][3] ([fdo#109271]) -> [FAIL][4] +4 similar issues
[3]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12680/fi-kbl-7567u/igt@kms_chamelium_hpd@vga-hpd-fast.html
[4]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8428/fi-kbl-7567u/igt@kms_chamelium_hpd@vga-hpd-fast.html
Known issues
------------
Here are the changes found in IGTPW_8428 that come from known issues:
### IGT changes ###
#### Issues hit ####
* igt@i915_selftest@live@gt_heartbeat:
- fi-glk-j4005: [PASS][5] -> [DMESG-FAIL][6] ([i915#5334])
[5]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12680/fi-glk-j4005/igt@i915_selftest@live@gt_heartbeat.html
[6]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8428/fi-glk-j4005/igt@i915_selftest@live@gt_heartbeat.html
#### Possible fixes ####
* igt@i915_selftest@live@migrate:
- {bat-dg2-11}: [DMESG-WARN][7] ([i915#7699]) -> [PASS][8]
[7]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12680/bat-dg2-11/igt@i915_selftest@live@migrate.html
[8]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8428/bat-dg2-11/igt@i915_selftest@live@migrate.html
{name}: This element is suppressed. This means it is ignored when computing
the status of the difference (SUCCESS, WARNING, or FAILURE).
[fdo#109271]: https://bugs.freedesktop.org/show_bug.cgi?id=109271
[i915#5334]: https://gitlab.freedesktop.org/drm/intel/issues/5334
[i915#6311]: https://gitlab.freedesktop.org/drm/intel/issues/6311
[i915#7359]: https://gitlab.freedesktop.org/drm/intel/issues/7359
[i915#7699]: https://gitlab.freedesktop.org/drm/intel/issues/7699
[i915#7982]: https://gitlab.freedesktop.org/drm/intel/issues/7982
Build changes
-------------
* CI: CI-20190529 -> None
* IGT: IGT_7143 -> IGTPW_8428
CI-20190529: 20190529
CI_DRM_12680: 06086656e6f03cbcbb4b99273734a7943e923fc9 @ git://anongit.freedesktop.org/gfx-ci/linux
IGTPW_8428: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8428/index.html
IGT_7143: c7b12dcc460fc2348e1fa7f4dcb791bb82e29e44 @ https://gitlab.freedesktop.org/drm/igt-gpu-tools.git
== Logs ==
For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8428/index.html
[-- Attachment #2: Type: text/html, Size: 4073 bytes --]
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [igt-dev] ✗ Fi.CI.BAT: failure for Chamelium: Get Chamelium Logs on RPC failure.
2023-02-01 18:49 ` [igt-dev] ✗ Fi.CI.BAT: failure " Patchwork
@ 2023-02-01 21:02 ` Mark Yacoub
2023-02-03 10:10 ` Petri Latvala
0 siblings, 1 reply; 6+ messages in thread
From: Mark Yacoub @ 2023-02-01 21:02 UTC (permalink / raw)
To: Vudum, Lakshminarayana; +Cc: igt-dev
[-- Attachment #1: Type: text/plain, Size: 3336 bytes --]
Hi Lakshminarayana, the failure seems unrelated - can we get a rerun please?
On Wed, Feb 1, 2023 at 1:49 PM Patchwork <patchwork@emeril.freedesktop.org>
wrote:
> *Patch Details*
> *Series:* Chamelium: Get Chamelium Logs on RPC failure.
> *URL:* https://patchwork.freedesktop.org/series/113567/
> *State:* failure
> *Details:* https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8428/index.html CI
> Bug Log - changes from CI_DRM_12680 -> IGTPW_8428 Summary
>
> *FAILURE*
>
> Serious unknown changes coming with IGTPW_8428 absolutely need to be
> verified manually.
>
> If you think the reported changes have nothing to do with the changes
> introduced in IGTPW_8428, please notify your bug team to allow them
> to document this new failure mode, which will reduce false positives in CI.
>
> External URL:
> https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8428/index.html
> Participating hosts (26 -> 25)
>
> Missing (1): fi-snb-2520m
> Possible new issues
>
> Here are the unknown changes that may have been introduced in IGTPW_8428:
> IGT changes Possible regressions
>
> - igt@kms_chamelium_frames@hdmi-crc-fast:
> - fi-kbl-7567u: PASS
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12680/fi-kbl-7567u/igt@kms_chamelium_frames@hdmi-crc-fast.html>
> -> FAIL
> <https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8428/fi-kbl-7567u/igt@kms_chamelium_frames@hdmi-crc-fast.html>
> +3 similar issues
>
> Warnings
>
> - igt@kms_chamelium_hpd@vga-hpd-fast:
> - fi-kbl-7567u: SKIP
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12680/fi-kbl-7567u/igt@kms_chamelium_hpd@vga-hpd-fast.html>
> (fdo#109271 <https://bugs.freedesktop.org/show_bug.cgi?id=109271>)
> -> FAIL
> <https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8428/fi-kbl-7567u/igt@kms_chamelium_hpd@vga-hpd-fast.html>
> +4 similar issues
>
> Known issues
>
> Here are the changes found in IGTPW_8428 that come from known issues:
> IGT changes Issues hit
>
> - igt@i915_selftest@live@gt_heartbeat:
> - fi-glk-j4005: PASS
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12680/fi-glk-j4005/igt@i915_selftest@live@gt_heartbeat.html>
> -> DMESG-FAIL
> <https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8428/fi-glk-j4005/igt@i915_selftest@live@gt_heartbeat.html>
> (i915#5334 <https://gitlab.freedesktop.org/drm/intel/issues/5334>)
>
> Possible fixes
>
> - igt@i915_selftest@live@migrate:
> - {bat-dg2-11}: DMESG-WARN
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12680/bat-dg2-11/igt@i915_selftest@live@migrate.html>
> (i915#7699 <https://gitlab.freedesktop.org/drm/intel/issues/7699>)
> -> PASS
> <https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8428/bat-dg2-11/igt@i915_selftest@live@migrate.html>
>
> {name}: This element is suppressed. This means it is ignored when computing
> the status of the difference (SUCCESS, WARNING, or FAILURE).
> Build changes
>
> - CI: CI-20190529 -> None
> - IGT: IGT_7143 -> IGTPW_8428
>
> CI-20190529: 20190529
> CI_DRM_12680: 06086656e6f03cbcbb4b99273734a7943e923fc9 @ git://
> anongit.freedesktop.org/gfx-ci/linux
> IGTPW_8428: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8428/index.html
> IGT_7143: c7b12dcc460fc2348e1fa7f4dcb791bb82e29e44 @
> https://gitlab.freedesktop.org/drm/igt-gpu-tools.git
>
[-- Attachment #2: Type: text/html, Size: 4944 bytes --]
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [igt-dev] ✗ Fi.CI.BAT: failure for Chamelium: Get Chamelium Logs on RPC failure.
2023-02-01 21:02 ` Mark Yacoub
@ 2023-02-03 10:10 ` Petri Latvala
2023-02-03 16:59 ` Mark Yacoub
0 siblings, 1 reply; 6+ messages in thread
From: Petri Latvala @ 2023-02-03 10:10 UTC (permalink / raw)
To: Mark Yacoub; +Cc: igt-dev, Vudum, Lakshminarayana
On Wed, Feb 01, 2023 at 04:02:35PM -0500, Mark Yacoub wrote:
> Hi Lakshminarayana, the failure seems unrelated - can we get a rerun please?
The failure is very related. The failure is:
"GetChameleondLogs" is not supported
You need to check with chamelium_supports_method whether
GetChameleondLogs is there. Btw, what chameleond version does that
need?
--
Petri Latvala
>
> On Wed, Feb 1, 2023 at 1:49 PM Patchwork <patchwork@emeril.freedesktop.org>
> wrote:
>
> > *Patch Details*
> > *Series:* Chamelium: Get Chamelium Logs on RPC failure.
> > *URL:* https://patchwork.freedesktop.org/series/113567/
> > *State:* failure
> > *Details:* https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8428/index.html CI
> > Bug Log - changes from CI_DRM_12680 -> IGTPW_8428 Summary
> >
> > *FAILURE*
> >
> > Serious unknown changes coming with IGTPW_8428 absolutely need to be
> > verified manually.
> >
> > If you think the reported changes have nothing to do with the changes
> > introduced in IGTPW_8428, please notify your bug team to allow them
> > to document this new failure mode, which will reduce false positives in CI.
> >
> > External URL:
> > https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8428/index.html
> > Participating hosts (26 -> 25)
> >
> > Missing (1): fi-snb-2520m
> > Possible new issues
> >
> > Here are the unknown changes that may have been introduced in IGTPW_8428:
> > IGT changes Possible regressions
> >
> > - igt@kms_chamelium_frames@hdmi-crc-fast:
> > - fi-kbl-7567u: PASS
> > <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12680/fi-kbl-7567u/igt@kms_chamelium_frames@hdmi-crc-fast.html>
> > -> FAIL
> > <https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8428/fi-kbl-7567u/igt@kms_chamelium_frames@hdmi-crc-fast.html>
> > +3 similar issues
> >
> > Warnings
> >
> > - igt@kms_chamelium_hpd@vga-hpd-fast:
> > - fi-kbl-7567u: SKIP
> > <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12680/fi-kbl-7567u/igt@kms_chamelium_hpd@vga-hpd-fast.html>
> > (fdo#109271 <https://bugs.freedesktop.org/show_bug.cgi?id=109271>)
> > -> FAIL
> > <https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8428/fi-kbl-7567u/igt@kms_chamelium_hpd@vga-hpd-fast.html>
> > +4 similar issues
> >
> > Known issues
> >
> > Here are the changes found in IGTPW_8428 that come from known issues:
> > IGT changes Issues hit
> >
> > - igt@i915_selftest@live@gt_heartbeat:
> > - fi-glk-j4005: PASS
> > <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12680/fi-glk-j4005/igt@i915_selftest@live@gt_heartbeat.html>
> > -> DMESG-FAIL
> > <https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8428/fi-glk-j4005/igt@i915_selftest@live@gt_heartbeat.html>
> > (i915#5334 <https://gitlab.freedesktop.org/drm/intel/issues/5334>)
> >
> > Possible fixes
> >
> > - igt@i915_selftest@live@migrate:
> > - {bat-dg2-11}: DMESG-WARN
> > <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12680/bat-dg2-11/igt@i915_selftest@live@migrate.html>
> > (i915#7699 <https://gitlab.freedesktop.org/drm/intel/issues/7699>)
> > -> PASS
> > <https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8428/bat-dg2-11/igt@i915_selftest@live@migrate.html>
> >
> > {name}: This element is suppressed. This means it is ignored when computing
> > the status of the difference (SUCCESS, WARNING, or FAILURE).
> > Build changes
> >
> > - CI: CI-20190529 -> None
> > - IGT: IGT_7143 -> IGTPW_8428
> >
> > CI-20190529: 20190529
> > CI_DRM_12680: 06086656e6f03cbcbb4b99273734a7943e923fc9 @ git://
> > anongit.freedesktop.org/gfx-ci/linux
> > IGTPW_8428: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8428/index.html
> > IGT_7143: c7b12dcc460fc2348e1fa7f4dcb791bb82e29e44 @
> > https://gitlab.freedesktop.org/drm/igt-gpu-tools.git
> >
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [igt-dev] ✗ Fi.CI.BAT: failure for Chamelium: Get Chamelium Logs on RPC failure.
2023-02-03 10:10 ` Petri Latvala
@ 2023-02-03 16:59 ` Mark Yacoub
0 siblings, 0 replies; 6+ messages in thread
From: Mark Yacoub @ 2023-02-03 16:59 UTC (permalink / raw)
To: Petri Latvala; +Cc: igt-dev, Vudum, Lakshminarayana
[-- Attachment #1: Type: text/plain, Size: 4221 bytes --]
On Fri, Feb 3, 2023 at 5:10 AM Petri Latvala <adrinael@adrinael.net> wrote:
> On Wed, Feb 01, 2023 at 04:02:35PM -0500, Mark Yacoub wrote:
> > Hi Lakshminarayana, the failure seems unrelated - can we get a rerun
> please?
>
>
> The failure is very related. The failure is:
>
> "GetChameleondLogs" is not supported
>
> You need to check with chamelium_supports_method whether
> GetChameleondLogs is there. Btw, what chameleond version does that
> need?
>
It runs on V3 - I added a number of safeguards to make sure everything is
happy.
>
> --
> Petri Latvala
>
>
> >
> > On Wed, Feb 1, 2023 at 1:49 PM Patchwork <
> patchwork@emeril.freedesktop.org>
> > wrote:
> >
> > > *Patch Details*
> > > *Series:* Chamelium: Get Chamelium Logs on RPC failure.
> > > *URL:* https://patchwork.freedesktop.org/series/113567/
> > > *State:* failure
> > > *Details:*
> https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8428/index.html CI
> > > Bug Log - changes from CI_DRM_12680 -> IGTPW_8428 Summary
> > >
> > > *FAILURE*
> > >
> > > Serious unknown changes coming with IGTPW_8428 absolutely need to be
> > > verified manually.
> > >
> > > If you think the reported changes have nothing to do with the changes
> > > introduced in IGTPW_8428, please notify your bug team to allow them
> > > to document this new failure mode, which will reduce false positives
> in CI.
> > >
> > > External URL:
> > > https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8428/index.html
> > > Participating hosts (26 -> 25)
> > >
> > > Missing (1): fi-snb-2520m
> > > Possible new issues
> > >
> > > Here are the unknown changes that may have been introduced in
> IGTPW_8428:
> > > IGT changes Possible regressions
> > >
> > > - igt@kms_chamelium_frames@hdmi-crc-fast:
> > > - fi-kbl-7567u: PASS
> > > <
> https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12680/fi-kbl-7567u/igt@kms_chamelium_frames@hdmi-crc-fast.html
> >
> > > -> FAIL
> > > <
> https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8428/fi-kbl-7567u/igt@kms_chamelium_frames@hdmi-crc-fast.html
> >
> > > +3 similar issues
> > >
> > > Warnings
> > >
> > > - igt@kms_chamelium_hpd@vga-hpd-fast:
> > > - fi-kbl-7567u: SKIP
> > > <
> https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12680/fi-kbl-7567u/igt@kms_chamelium_hpd@vga-hpd-fast.html
> >
> > > (fdo#109271 <https://bugs.freedesktop.org/show_bug.cgi?id=109271
> >)
> > > -> FAIL
> > > <
> https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8428/fi-kbl-7567u/igt@kms_chamelium_hpd@vga-hpd-fast.html
> >
> > > +4 similar issues
> > >
> > > Known issues
> > >
> > > Here are the changes found in IGTPW_8428 that come from known issues:
> > > IGT changes Issues hit
> > >
> > > - igt@i915_selftest@live@gt_heartbeat:
> > > - fi-glk-j4005: PASS
> > > <
> https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12680/fi-glk-j4005/igt@i915_selftest@live@gt_heartbeat.html
> >
> > > -> DMESG-FAIL
> > > <
> https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8428/fi-glk-j4005/igt@i915_selftest@live@gt_heartbeat.html
> >
> > > (i915#5334 <https://gitlab.freedesktop.org/drm/intel/issues/5334
> >)
> > >
> > > Possible fixes
> > >
> > > - igt@i915_selftest@live@migrate:
> > > - {bat-dg2-11}: DMESG-WARN
> > > <
> https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_12680/bat-dg2-11/igt@i915_selftest@live@migrate.html
> >
> > > (i915#7699 <https://gitlab.freedesktop.org/drm/intel/issues/7699
> >)
> > > -> PASS
> > > <
> https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8428/bat-dg2-11/igt@i915_selftest@live@migrate.html
> >
> > >
> > > {name}: This element is suppressed. This means it is ignored when
> computing
> > > the status of the difference (SUCCESS, WARNING, or FAILURE).
> > > Build changes
> > >
> > > - CI: CI-20190529 -> None
> > > - IGT: IGT_7143 -> IGTPW_8428
> > >
> > > CI-20190529: 20190529
> > > CI_DRM_12680: 06086656e6f03cbcbb4b99273734a7943e923fc9 @ git://
> > > anongit.freedesktop.org/gfx-ci/linux
> > > IGTPW_8428:
> https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_8428/index.html
> > > IGT_7143: c7b12dcc460fc2348e1fa7f4dcb791bb82e29e44 @
> > > https://gitlab.freedesktop.org/drm/igt-gpu-tools.git
> > >
>
[-- Attachment #2: Type: text/html, Size: 7795 bytes --]
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2023-02-03 16:59 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2023-02-01 18:12 [igt-dev] [PATCH] Chamelium: Get Chamelium Logs on RPC failure Mark Yacoub
2023-02-01 18:24 ` [igt-dev] ✗ GitLab.Pipeline: warning for " Patchwork
2023-02-01 18:49 ` [igt-dev] ✗ Fi.CI.BAT: failure " Patchwork
2023-02-01 21:02 ` Mark Yacoub
2023-02-03 10:10 ` Petri Latvala
2023-02-03 16:59 ` Mark Yacoub
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox