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

Various fixes 12 2024 #956

Merged
merged 3 commits into from
Dec 24, 2024
Merged

Conversation

nuclearcat
Copy link
Member

1)pod memory allocation
2)scheduler fix due missing keys in node
3)trigger fix due tree returning tree during head retrieval

This will prevent pods from being evicted first.

Signed-off-by: Denys Fedoryshchenko <[email protected]>
Fixes crash:
```
12/23/2024 05:28:21 PM UTC [ERROR] Traceback (most recent call last):
  File "/home/kernelci/pipeline/src/base.py", line 69, in run
    status = self._run(context)
             ^^^^^^^^^^^^^^^^^^
  File "/home/kernelci/pipeline/./src/scheduler.py", line 296, in _run
    self._run_job(job, runtime, platform, input_node)
  File "/home/kernelci/pipeline/./src/scheduler.py", line 80, in _run_job
    node = self._api_helper.create_job_node(job_config, input_node,
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/site-packages/kernelci/api/helper.py", line 402, in create_job_node
    'kernel_revision': input_node['data']['kernel_revision'],
                       ~~~~~~~~~~~~~~~~~~^^^^^^^^^^^^^^^^^^^
KeyError: 'kernel_revision'
```

Signed-off-by: Denys Fedoryshchenko <[email protected]>
Fix crash on faulty tree, for example:
```
fatal: unable to access 'https://gitlab.collabora.com/google/chromeos-kernel.git/': The requested URL returned error: 502
12/22/2024 12:13:51 PM UTC [ERROR] Traceback (most recent call last):
  File "/home/kernelci/pipeline/src/base.py", line 69, in run
    status = self._run(context)
             ^^^^^^^^^^^^^^^^^^
  File "/home/kernelci/pipeline/./src/trigger.py", line 145, in _run
    self._iterate_build_configs(force, build_configs_list,
  File "/home/kernelci/pipeline/./src/trigger.py", line 120, in _iterate_build_configs
    self._run_trigger(config, force, timeout, trees)
  File "/home/kernelci/pipeline/./src/trigger.py", line 46, in _run_trigger
    head_commit = kernelci.build.get_branch_head(build_config)
                  ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/site-packages/kernelci/build.py", line 81, in get_branch_head
    head = shell_cmd(cmd)
           ^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/site-packages/kernelci/__init__.py", line 29, in shell_cmd
    return subprocess.check_output(cmd, shell=True).decode()
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/subprocess.py", line 466, in check_output
    return run(*popenargs, stdout=PIPE, timeout=timeout, check=True,
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/subprocess.py", line 571, in run
    raise CalledProcessError(retcode, process.args,
subprocess.CalledProcessError: Command 'git ls-remote https://gitlab.collabora.com/google/chromeos-kernel.git refs/heads/for-kernelci' returned non-zero exit status 128.
```

Signed-off-by: Denys Fedoryshchenko <[email protected]>
@nuclearcat nuclearcat force-pushed the various-fixes-12-2024 branch from 639a752 to 7dd41b3 Compare December 24, 2024 13:27
@nuclearcat
Copy link
Member Author

Tested on staging

@nuclearcat nuclearcat added this pull request to the merge queue Dec 24, 2024
Merged via the queue into kernelci:main with commit aa22d16 Dec 24, 2024
3 checks passed
@nuclearcat nuclearcat deleted the various-fixes-12-2024 branch December 24, 2024 13:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant