Marcelo Giles

Results 22 comments of Marcelo Giles

I'm going to move it to _Ready for review_, to see if it passes the automatic online build process, since it may just fail on my local system.

Hi @kranurag7. I've run several tests on my system, including setting up different stub resolvers and DNS configurations based on the error that I get during the `checkPhase` of the...

I've been testing different solutions. I think that skipping the `trivy` test is less disruptive than introducing those Go variables that I proposed before. I can confirm that the `k8sgpt`...

They solved the `lisp-modules` issue. Now the `k8sgpt` package builds fine. See build log file: [k8sgpt-x86_64-linux.log](https://github.com/user-attachments/files/17454664/k8sgpt-x86_64-linux.log) ## `nixpkgs-review` result Generated using [`nixpkgs-review`](https://github.com/Mic92/nixpkgs-review). Command: `nixpkgs-review pr 348479` --- ### `x86_64-linux` :white_check_mark:...

Hey @kranurag7, thanks for checking. I tried to fix the merge error myself (based on the error that you got when merging) by moving the package to [`pkgs/by-name`](https://github.com/NixOS/nixpkgs/blob/master/pkgs/by-name/README.md) and made...

> **nixpkgs-merge-bot**: merge not permitted > pkgs/applications/networking/cluster/k8sgpt/default.nix is not in pkgs/by-name/ I'm giving the merge issue another try (I'm stubborn). Seems like all checks have passed, we should be able...

@NixOS/nixpkgs-merge-bot merge

Same thing... Oh, well. I'll rollback the changes. At least I gave a try. Thanks for checking @kranurag7

I think I've found out why the PR merge run by the [`nixpks-merge-bot`](https://github.com/NixOS/nixpkgs-merge-bot) fails. The error message is quite obvious but I didn't understand why it didn't fail for other...

Thank you for your comments @gepbird! They helped me to better understand the process. Now I can see more clearly which of my assumptions were correct and which were wrong....