-
Notifications
You must be signed in to change notification settings - Fork 99
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
a bytes-like object is required, not 'str - VM unpacking #1578
Comments
Thanks for the detailed report! You are quite sure that VM scanning was working before? Which OS/arch are you running? Can you paste more of the log (from the UI)? Things unpack OK for here:
|
@AlexBaranowski can you enter a separate issue for the airgap config? |
@pombredanne -> Look at the log itself. 0 seconds for rest of pipeline. I'm very positive that is reproducible, please check the WebUI to see the problem. You can compare with older versions. I created #1579 -> I'm willing to provide PR myself, but I think that current bug is a little showstopper and I won't be able to test it. Give me 30 minutes and I will provide the diff on the python packages and system packages. I already ruled out that using older debian slim python do not fix the issue. |
With the The full lists are in the attachments. The diffs
Now the system packages:
As the python itself is not present - it's compiled from source there is small difference in python version - I might try to debug further but it's getting very code specific. |
Image used for tests - https://cdimage.debian.org/images/cloud/bookworm/latest/debian-12-genericcloud-amd64.qcow2 -> it's much much smaller about 332M -> so faster to test. RUN WITH SINGLE WORKER! WITHOUT Login into container to do changes
Restarting worker after changes:
First test - normalSecond test downgrading some packagesOn the worker execute
then restart worker. It still fails! The problem is that scancodeio was not downgraded to the 34.9.1 from 34.9.4 :(. Third try downgrading more packagesThese two packages have special place in my heart xD. lxml==5.3.0 and beautifulsoup4==4.12.3
WORKED! I stopped execution :). Forth try - downgrading only lxml and beautifulsoup4Only lxml==5.3.0 and beautifulsoup4==4.12.3 Recreate the everything with the:
Firstly I rerun test test that will fail (no changes). Then on worker I executed
And restarted it
It worked :). End noteWith forcefull downgarde lxml and beautifulsoup4 it is working once more. I might I also do not wait for full scan to finish as I do not have time. It's 12:30 in my timezone ;). Best, |
Thanks for all the details! Need some time to munch through it all. |
Thanks for reporting this @AlexBaranowski, I was facing the same issue too from last week. |
@pombredanne @AyanSinhaMahapatra current master fixes the issue. I'm closing this issue. |
For some time—probably about a week—the scancode.io instances have not been working anymore. I thought it was my fault and debugged it for a while, as this bug/change coincided with changes in the runtime environment I'm using. After a while, I found that the issue was the original scancode.io repository, not my environment.
The VM image that I'm trying to scan:
It will result in the following error after the worker tries to unpack the VM image. Unfortunately, this is an extremely short and not helpful error message:
You can replicate the not-working environment and working environment with a slightly older version and images.
Installation method 1 - Original - not working
Installation method 2 - older images + fixes on offline deployment
Note that I'm using my personal repository, which contains the fixes for docker-compose offline. I'm using an older version because the Redis environment changed 😸
How this could be helpful
I think that checking the:
could be helpful, so I'm providing the working images :). I won't bottom out this problem because I do not have more time to debug it as I have spent two days fixing the nonexistent bug in my environment :((( The second installation option is a good workaround for my needs.
Airgap not working as expected
Well, TBH, when it's fixed (the Docker images are once more working), I could update the offline installation docs, makefile, and docker-compose-offline.yml because they are outdated and not working ;)
Related issues
#1577
Best,
Alex
The text was updated successfully, but these errors were encountered: