Skip to content

Revert "Let the user give the pipeline URL directly"

Sergi Blanch Torné requested to merge sergi/ci-collate:revert_30 into main

This reverts commit e3bc1145.

An attempt to use the patch subcommand shows an issue with this commit:

ci-collate --namespace mesa --project mesa patch --pipeline-id 1135123 --local-clone . --branch-namespace sergi --branch nightly_run_expectation_changes
Traceback (most recent call last):
  File "/home/serguei/src/Collabora/ChromeOS-mesa/ci-collate.git/ci-collate.venv/bin/ci-collate", line 8, in <module>
    sys.exit(main())
             ^^^^^^
  File "/home/serguei/src/Collabora/ChromeOS-mesa/ci-collate.git/glcollate/main.py", line 84, in main
    if not collate_pipeline:
           ^^^^^^^^^^^^^^^^
UnboundLocalError: cannot access local variable 'collate_pipeline' where it is not associated with a value

I don't understand now why this definition of collate_pipeline is later not available when it likes to use it.

Merge request reports