chainloop-dev / chainloop

Chainloop is an Open Source evidence store for your Software Supply Chain attestations, SBOMs, VEX, SARIF, CSAF files, QA reports, and more.
https://docs.chainloop.dev
Apache License 2.0
358 stars 27 forks source link

Describe of workflow run errors out #872

Open javirln opened 3 months ago

javirln commented 3 months ago

When describing a workflow run by its id, it's failing due to an error when unmarshalling:

$ chainloop --debug wf run ls
DBG Telemetry enabled, to disable it use DO_NOT_TRACK=1
┌──────────────────────────────────────┬────────────────┬─────────────┬─────────────────────┬─────────────┐
│ ID                                   │ WORKFLOW       │ STATE       │ CREATED AT          │ RUNNER      │
├──────────────────────────────────────┼────────────────┼─────────────┼─────────────────────┼─────────────┤
│ a7e9a956-21c7-49dc-b346-0c9ac1613abd │ core/team-demo │ success     │ 03 Jun 24 12:37 UTC │ Unspecified │
│ 43b67a50-8f84-4b53-aec5-b8c130c15d38 │ core/team-demo │ success     │ 03 Jun 24 12:35 UTC │ Unspecified │
│ 76561265-1b44-41a5-a919-110a4fffdd9c │ core/team-demo │ success     │ 03 Jun 24 12:29 UTC │ Unspecified │
│ 731d9d0e-7a94-4388-95ec-00dac57ee169 │ core/team-demo │ initialized │ 03 Jun 24 12:22 UTC │ Unspecified │
│ 391a80b1-98db-4207-8bc5-80efd918a363 │ core/team-demo │ initialized │ 03 Jun 24 12:08 UTC │ Unspecified │
└──────────────────────────────────────┴────────────────┴─────────────┴─────────────────────┴─────────────┘
~/Development/projects/playground/demo                                                                                                     system 14:42:49
$ chainloop --debug wf run describe --id a7e9a956-21c7-49dc-b346-0c9ac1613abd
DBG Telemetry enabled, to disable it use DO_NOT_TRACK=1
ERR grpc: error while marshaling: string field contains invalid UTF-8

Double check if it has something to do with having a inline CAS rather than something set up for example S3 local cosign key and KMS.

migmartri commented 3 months ago

The issue seems to be related to a helm chart stored inline

 materials:{name:"chart" value:"\x1f\x8b\x08\x14\x00\x00\x00\x00\x00\xff)\x00+aHR0cHM6Ly95b3V0dS5iZS96OVV6MWljandyTQo=Helm\x00\xec}{\x7f\xd
b6\xb2h\xff֧\x98K\xbb'VjQ\x92_\xe9\xf1\xaew\x8f\xeb\xa4[ߦ\x89\xaf\x9d&wO\xb3\xa7\x85HHBM\x11,\x00\xca\xd1&\xbe\x9f\xfd\xfe\xf0\"\xc1\x97\x1e\xb6l'\xbb\xe6\xee/\xb5H`0\x18\x0c\xe6\x85\x01\x10\xd0Ʉ\xc6ݓ1b\u009f\xa1I\xf4\xd5\xfa\x9f^\xaf\xd7;\xd8\xdbS\xff\xed\xf5z\xe5\xff\xf6{;\xbb_\xf
5\xf7\xf6\xfb\xbb\xfb\xfd\xfe\xfeN\xef\xab^\x7fgo\x7f\xe7+\xe8\xdd\x01.\x95'\xe5\x02\xb1\xafz\xb7n\xabܹ/\xe4AqL\x05\x12\x84\xc6\xfc\xb0\x05\x10 \x81G\x94\xcd\x0e\xe14\x1e2\xc4\x05K\x03\x912\xdc\x02\x88H\x80c\x8e\xf9!\x1c'(\x18\xe3Ύ\xdfk\xa1\x84\xbcŌ\x13\x1a\x1f\xc2t\xa7\x85\x92$\xfb
\xb9\xe3\xf7w\xfd\xddV\x88y\xc0H\"ԻcxI\x06\x0c\xb1\x19\xfc\x80\xa3\t(\xae\x83!e0b4MH<\x02͏\x10\xd1\x11\t`\x80\xc5\x15\xc61\x0c\x88\x88ф@ \xcbs\xbf\x05\xf0fL\xb8\xfe\t\x84CL\x05\x848\x89\xe8\x0c\r\"\x0c\x83\x19\x10\xc1q4\xf4[c:\xc1\x870\x16\"\xe1\x87ݮ\x81\xe3\x07t\xd2\"\x81D\xa8\xe6K
7\xa4WqDQȻ\x11\x1dQn\xbfu&\x88]\xfaI<j]\xe2\xd9\x15e!?lu\x0c\xbe\xad\x0e\x8cq\x94`\xd6\xea\x80\xc0\x93$B\x02\xb7:0L\xe3@v\xbcձ]hM\x10\x89\x05\"1f\xaaz\x8c$~o\x7f\xbaB\x0co\xc3i\x1c\xc8Υ,\xca1\x1b\x111N\x07\n1\x03\xa3\xab\xc9\xd0\xd2u\r\x02\x9c\xa6,\xc0\n\xe6\xe2\x9ab\x96\xe0C\x88\x
f4P\xb4\xa6\xa5\x11{h\x96||\xee\xf11\xf2\x7f\x8a\xa2\x14\xf3;R\x00\x0b\xe4\x7fo\xe7\xe0\xa0,\xff\x9f\xed\xf4\x1e\xe5\xff}<\x1bpB\x93\x19#\xa3\xb1(\x8a\xa1\r\xb88{\xfe\x7f;/\xb5\xd48\x16dH0;\x84\xe3\xb3\xe3\x93\x1f^(\xf9\xdf\xda؀L\xb8hI\xb4\xb1\x01\xa7J*3\xfcGJ\x18\x0e\xa5<>9\xed\x1
9e<\x07Ai\xc4\x01\xc5!$\x8c\x06\x98s\xcc}Y\xfe\xbf\xf8%I\x00\x7f@\x93$\xc2o%+\xb666Z\xeeo+\xe7:J\x82=\x8a\xa8\xb5=f\xfe[\xa5Ż\xbf\xa2\xe1\x90\xc4D\x10\xcc}\x91\xacE\x1a̟\xff\xfd\x9d\xfd\xfd^i\xfe\xef\xef\xef\xee>\xce\xff\xfbx>~\xec>m5\x88\x80e\x04\xc0\xd3\xee\xf5u\xab%\xa1\xc0\x94L\
x0e\x81c\x01C\x12aib\x1cM$q\x831>\x84\xbcX\xeb\x1c\x8b\x94ŀ\x80ӡ\x80\x98\x86\xf8Xs\xdc\x0cB\xacY\x8fƭ\x8f\x1f\x81\xc4A\x94\x86\x18<ͣ\xbeØ\xb2\x16
javirln commented 3 months ago

Shall we move this issue to TODO ?

migmartri commented 3 months ago

Shall we move this issue to TODO ?

Good catch, thanks!