Skip to content
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

ffi: memory - increase duration to 50m #514

Closed
wants to merge 3 commits into from
Closed

ffi: memory - increase duration to 50m #514

wants to merge 3 commits into from

Conversation

dougsland
Copy link
Collaborator

avoid timeout

avoid timeout

Signed-off-by: Douglas Landgraf <[email protected]>
@Yarboa
Copy link
Collaborator

Yarboa commented Aug 23, 2024

I think the tests should be fixed,
We need to trigger exit status faster, no to increase duration to 50 minutes.
WDYT?

I would consider reducing autosd/ffi-tools image to slimmer version, to reduce image download time
WDYT?

@dougsland
Copy link
Collaborator Author

I think the tests should be fixed, We need to trigger exit status faster, no to increase duration to 50 minutes. WDYT?

I would consider reducing autosd/ffi-tools image to slimmer version, to reduce image download time WDYT?

agreed. I am testing, I was not convinced the 20m was the root cause and found other two problems.

Error: OCI runtime error: crun: the requested cgroup controller `pids` is not available
[ INFO  ] PASS: qm.container oom_score_adj value == 500
./test.sh: line 39: [: cat: /proc/0/oom_score_adj: No such file or directory: integer expression expected
[ INFO  ] FAIL: qm containers oom_score_adj != 750. Current value is cat: /proc/0/oom_score_adj: No such file or directory

@dougsland
Copy link
Collaborator Author

@Yarboa is going to take it, he have some ideas to solve it.

@Yarboa
Copy link
Collaborator

Yarboa commented Aug 27, 2024

@odra @dougsland
There is an issue with automation
see that https://quay.io/organization/centos-sig-automotive

quay.ioquay.io
Quay
Quay is the best place to build, store, and distribute your containers. Public repositories are always free.
3:40
45 G

Need to clean
https://gitlab.com/CentOS/automotive/container-images/ffi-tools/-/merge_requests/33

@Yarboa Yarboa mentioned this pull request Aug 27, 2024
@Yarboa
Copy link
Collaborator

Yarboa commented Aug 27, 2024

Closing #508 solved it with reduction of ffi-tools container

@Yarboa Yarboa closed this Aug 27, 2024
@dougsland dougsland deleted the timtout branch September 12, 2024 21:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working critical ffi jira
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants