artifactCleanup - protect artifacts from being deleted matching a regular expression #399
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
commit 97ff199 Merge: 56c8308 28a9252 Author: Martin Migasiewicz <[email protected]> Date: Fri Oct 1 11:56:36 2021 +0200 Merge pull request #15 from tt-gf/bugfix/undefined-var fix: several fixes around the keepArtifactsRegex feature commit 28a9252 Author: Martin Migasiewicz <[email protected]> Date: Fri Oct 1 11:35:04 2021 +0200 fix: compile string as Pattern since casting a string stored in json to Pattern is not possible. rename job to include the plugin name for easier searching in logs avoid using null for repos in case it is not set as otherwise all repositories will be cleaned up align default regular expressions, document how to pass an url-encoded regex and add fix for reading the regex in the cleanup execution add example explaining a bit the regular expression commit 56c8308 Merge: 02465a4 012edea Author: Martin Migasiewicz <[email protected]> Date: Wed Jul 21 14:45:29 2021 +0200 Merge pull request #14 from tt-gf/bugfix/add-missing-docs chore(artifactCleanup): document new keep artifacts feature. commit 012edea Author: Martin Migasiewicz <[email protected]> Date: Wed Jul 21 13:10:20 2021 +0200 chore(artifactCleanup): document new keep artifacts feature. commit 02465a4 Merge: e96ed42 2d2f2a7 Author: Martin Migasiewicz <[email protected]> Date: Wed Jul 21 12:37:39 2021 +0200 Merge pull request #13 from tt-gf/new-master Sync with upstream repository commit 2d2f2a7 Merge: 31d2423 c633061 Author: Martin Migasiewicz <[email protected]> Date: Wed Jul 21 12:07:50 2021 +0200 Merge pull request #12 from tt-gf/feature/merge-upstream Pre-merge of modifications in artifactCleanup and cleanDockerImages user plugins to see whether conflicts were merged wrongly with upstream commit c633061 Merge: e96ed42 31d2423 Author: Martin Migasiewicz <[email protected]> Date: Tue Jul 20 15:30:57 2021 +0200 Merge remote-tracking branch 'upstream/master' into feature/merge-upstream # Conflicts: # cleanup/artifactCleanup/artifactCleanup.groovy # cleanup/cleanDockerImages/README.md # cleanup/cleanDockerImages/cleanDockerImages.groovy # cleanup/cleanDockerImages/cleanDockerImages.properties # webhook/README.md # webhook/webhook.groovy commit e96ed42 Merge: 135df82 e60b0be Author: Maikel van den Hurk <[email protected]> Date: Wed Jan 20 09:31:32 2021 +0100 Merge pull request #10 from NorbertGebicki-TomTom/NAV-32635_fix_for_docker_cleanup Nav 32635 fix for docker cleanup commit e60b0be Author: Norbert Gebicki <[email protected]> Date: Mon Jan 18 11:12:54 2021 +0100 update for README file commit 453da5f Author: Norbert Gebicki <[email protected]> Date: Thu Jan 14 13:57:41 2021 +0100 fixed PR issues commit bdec65e Author: Norbert Gebicki <[email protected]> Date: Wed Jan 13 10:55:35 2021 +0100 added new functionality to cleanDockerImages commit 0afd960 Author: Norbert Gebicki <[email protected]> Date: Tue Dec 15 15:11:44 2020 +0100 added cleanDockerImages plugin commit 719ef2c Author: Norbert Gebicki <[email protected]> Date: Thu Nov 12 14:26:11 2020 +0100 fixed artifactory plugin for 7.x version commit 135df82 Merge: c303f3b 7adb7f0 Author: Maikel van den Hurk <[email protected]> Date: Mon Sep 14 16:54:17 2020 +0200 Merge pull request #6 from NorbertGebicki-TomTom/patch-2 Update artifactCleanup.groovy commit 7adb7f0 Author: Norbert Gebicki <[email protected]> Date: Mon Sep 14 16:43:45 2020 +0200 fixed counting of artifacts commit 26b172d Author: NorbertGebicki-TomTom <[email protected]> Date: Tue Sep 8 15:45:13 2020 +0200 Update artifactCleanup.groovy commit b5d3b40 Author: NorbertGebicki-TomTom <[email protected]> Date: Tue Sep 8 15:25:09 2020 +0200 Update artifactCleanup.groovy fixed duplicated count commit c303f3b Merge: 5cc0c52 bd78d74 Author: Maikel van den Hurk <[email protected]> Date: Mon Sep 7 21:14:49 2020 +0200 Merge pull request #7 from tt-gf/webhook-copy feat: Add webhook plugin commit bd78d74 Author: Maikel van den Hurk <[email protected]> Date: Mon Sep 7 12:53:57 2020 +0200 feat: Add webhook plugin This plugin is coming directly from https://github.com/jfrog/artifactory-user-plugins commit 72fb322 Author: NorbertGebicki-TomTom <[email protected]> Date: Wed Aug 19 13:56:00 2020 +0200 Update artifactCleanup.groovy commit 5cc0c52 Merge: bae045c 195d694 Author: wotd <[email protected]> Date: Fri Jun 1 14:05:31 2018 +0200 Merge pull request #5 from tt-gf/fix_for_correct_artifacts_list fix for removing images only from specific registry commit 195d694 Author: Norbert Gebicki <[email protected]> Date: Wed May 30 14:42:11 2018 +0200 fix for removing images only from specific registry commit bae045c Merge: 3b4a1b5 45b89c2 Author: wotd <[email protected]> Date: Tue May 29 14:42:24 2018 +0200 Merge pull request #4 from tt-gf/feature/NAVCD-399-Retention-policy-for-docker-images-script Feature/retention policy for docker images script commit 45b89c2 Author: Norbert Gebicki <[email protected]> Date: Tue May 29 13:22:11 2018 +0200 added properties file for retention policy for docker commit b6b6590 Author: Norbert Gebicki <[email protected]> Date: Fri May 11 14:34:09 2018 +0200 plugin for depricating artifacts introduced commit 3b4a1b5 Merge: e632b56 4726381 Author: wotd <[email protected]> Date: Tue Apr 10 13:45:11 2018 +0200 Merge pull request #2 from tt-gf/refactoring_and_fixing_code_issues fixed Martins comments and small code refactoring commit 4726381 Author: Norbert Gebicki <[email protected]> Date: Tue Apr 10 12:44:06 2018 +0200 changed variables names: keepRelease, releaseRegex to keepArtifacts and keepArtifactsRegex commit bcba633 Author: Norbert Gebicki <[email protected]> Date: Tue Apr 10 11:39:33 2018 +0200 removed commented command commit 89aaaad Author: Norbert Gebicki <[email protected]> Date: Mon Apr 9 16:02:38 2018 +0200 fixed Martins comments and small code refactoring commit e632b56 Merge: c193576 f5b225d Author: wotd <[email protected]> Date: Fri Apr 6 15:18:43 2018 +0200 Merge pull request #1 from tt-gf/feature/cleanup_plugin_fix_for_retention added possibility to delete artifact with specified name pattern commit f5b225d Author: Norbert Gebicki <[email protected]> Date: Fri Apr 6 14:28:43 2018 +0200 added possibility to delete artifact with specified name pattern
|
Unfortunately this feature has no unit tests since there are no guidelines on how to setup easily an dev environment (the user-plugins-dev-env repo is outdated) |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR is another attempt to contribute this feature and supersedes PR #204.
The main purpose is to protect artifacts from being deleted that match a certain regular expression (
keepArtifactsRegex)Credits go mainly to @wotd, @NorbertGebicki-TomTom
This feature could as well be used to avoid the deletion of metadata files (eg.
.timestamporindex.jsonfor Conan artifacts #319)