Tags: google/copybara
Tags
Scrub credentials in CredentialFileHandler logs Also changed the log level for unmatched credential lines from WARNING to INFO, as it is expected to have unmatched lines when the credentials file contains entries for multiple repositories. Improved the log message to be clearer. Added unit tests to verify scrubbing of credentials with different usernames. NO_BUG: No bug specified GWSQ_IGNORES: chriscampos@google.com PiperOrigin-RevId: 930724211 Change-Id: Ibbba6be17bb1eecb278eea351c0dc368d71704aa
Improve patch folder name inference. Previously, we inferred the QUILT_PATCHES variable, which determines the name of the "patches" folder in Quilt, as the entire folder path present in the "series" parameter, except the final "/series". However, this causes issues if the "series" parameter contains nested paths, as Quilt's output path in the working directory would now be linked to this input path in the config file system. With this change, only the last folder name is parsed, not the entire path. Regardless of the input folder depth (e.g. `someFolder/patchesFolderName/series`), the Quilt output will be in `checkoutDirectory/directory/patchesFolderName/series` and it's up the config/user to move all content in the root of the checkoutDirectory to other directories matching the config location like `someFolder`, if desired. Also fixed a bug with using an empty patches folder name, e.g. just `series` that caused the output to still create a `patches` folder anyway, as QUILT_PATCHES being an empty string caused quilt to fall back to its default. Explicitly used `.` as the folder name for this case. Also shifted the additional argument validation happening during folder parsing into one central location at the beginning. Also updated documentation to explain the path handling better, and made the `series` file name requirement an explicit validation check. BUG: 481299219 GWSQ_IGNORES: mschlaffer@google.com PiperOrigin-RevId: 923374892 Change-Id: I605a15594cf2aa31d0c6c8ddcb8f04a6c51ff411
Allow customization of patch validation. Allows authors to control patch validation strictness through the patch transformation definition. Generalizes the idea of the --patch-validate-on-load parameter to Starlark, and makes --patch-validate-on-load a CLI override mechanism with no default anymore. Effective default validation behavior is unchanged to before. patch.quilt_apply also exits early if no series or patches could be found and validation is bypassed, ensuring no superfluous quilt setup is done in this case. refactored the code a bit to achieve this in a readable way. Also cleaned up code a bit: used multiline string literals where appropriate, used newlines consistently for unit test structure. BUG: 475203147 GWSQ_IGNORES: mschlaffer@google.com PiperOrigin-RevId: 917910800 Change-Id: Id32281e12399613f6c859ef4862fd70ed279893b
Allow QuiltTransformation to run on existing patches folders The QuiltTransformation now warns instead of failing if a 'patches' directory already exists in the destination. It will overwrite the 'series' file and any patch files being imported, while preserving other files in the 'patches' directory. This allows for re-running the transformation without manual cleanup. It also allows it to run in-place for after-move transforms. BUG=481299219 GWSQ_IGNORES: no-reply@google.com PiperOrigin-RevId: 906331061 Change-Id: I8885f5aef3c192f5ae096b4cca538740d03bfedd
Add retry logic to Gerrit change lookup before submitting. When submitting a Gerrit change, Copybara will now retry querying Gerrit for the change ID up to 3 times with exponential backoff (1s, 2s) if the change is not found immediately. This addresses issues where Gerrit might not immediately return the change after it has been pushed. BUG=472404588 GWSQ_IGNORES: no-reply@google.com PiperOrigin-RevId: 904563462 Change-Id: I06235cb8030d6d2c30cb387eee6751f136958ea7
PreviousNext