Pass sync_acls=True to sync_release (#2317)

Pass sync_acls=True to sync_release Followup of packit/packit#2214 RELEASE NOTES BEGIN N/A RELEASE NOTES END Reviewed-by: Nikola Forró
packit · Jan 25, 2024 · 0bbfd66 · 0bbfd66
2 parents 5257ccd + efeed0f
commit 0bbfd66
Show file tree

Hide file tree

Showing 5 changed files with 10 additions and 0 deletions.
diff --git a/packit_service/worker/handlers/distgit.py b/packit_service/worker/handlers/distgit.py
@@ -249,6 +249,7 @@ def sync_branch(
                     "release_monitoring_project_id"
                 ),
                 pr_description_footer=CHANGED_LOADING_BEHAVIOUR_IN_DISTGIT_MESSAGE,
+                sync_acls=True,
             )
         except PackitDownloadFailedException as ex:
             # the archive has not been uploaded to PyPI yet

diff --git a/tests/integration/test_new_hotness_update.py b/tests/integration/test_new_hotness_update.py
@@ -154,6 +154,7 @@ def test_new_hotness_update(new_hotness_update, sync_release_model):
         resolved_bugs=["rhbz#2106196"],
         release_monitoring_project_id=4181,
         pr_description_footer=CHANGED_LOADING_BEHAVIOUR_IN_DISTGIT_MESSAGE,
+        sync_acls=True,
     ).and_return(pr).once()
     flexmock(PackitAPI).should_receive("clean")
 

diff --git a/tests/integration/test_pr_comment.py b/tests/integration/test_pr_comment.py
@@ -2688,6 +2688,7 @@ def _get_project(url, *_, **__):
         resolved_bugs=["rhbz#123", "rhbz#124"],
         release_monitoring_project_id=None,
         pr_description_footer=CHANGED_LOADING_BEHAVIOUR_IN_DISTGIT_MESSAGE,
+        sync_acls=True,
     ).and_return(pr).once()
     flexmock(PackitAPI).should_receive("clean")
 

diff --git a/tests/integration/test_release_event.py b/tests/integration/test_release_event.py
@@ -184,6 +184,7 @@ def test_dist_git_push_release_handle(github_release_webhook, propose_downstream
         resolved_bugs=[],
         release_monitoring_project_id=None,
         pr_description_footer=CHANGED_LOADING_BEHAVIOUR_IN_DISTGIT_MESSAGE,
+        sync_acls=True,
     ).and_return(pr).once()
     flexmock(PackitAPI).should_receive("clean")
 
@@ -306,6 +307,7 @@ def test_dist_git_push_release_handle_multiple_branches(
             resolved_bugs=[],
             release_monitoring_project_id=None,
             pr_description_footer=CHANGED_LOADING_BEHAVIOUR_IN_DISTGIT_MESSAGE,
+            sync_acls=True,
         ).and_return(pr).once()
 
         flexmock(ProposeDownstreamJobHelper).should_receive(
@@ -438,6 +440,7 @@ def test_dist_git_push_release_handle_one_failed(
                 resolved_bugs=[],
                 release_monitoring_project_id=None,
                 pr_description_footer=CHANGED_LOADING_BEHAVIOUR_IN_DISTGIT_MESSAGE,
+                sync_acls=True,
             ).and_return(pr).once()
             flexmock(ProposeDownstreamJobHelper).should_receive(
                 "report_status_for_branch"
@@ -459,6 +462,7 @@ def test_dist_git_push_release_handle_one_failed(
                 resolved_bugs=[],
                 release_monitoring_project_id=None,
                 pr_description_footer=CHANGED_LOADING_BEHAVIOUR_IN_DISTGIT_MESSAGE,
+                sync_acls=True,
             ).and_raise(Exception, f"Failed {model.branch}").once()
             flexmock(ProposeDownstreamJobHelper).should_receive(
                 "report_status_for_branch"
@@ -699,6 +703,7 @@ def test_retry_propose_downstream_task(
         resolved_bugs=[],
         release_monitoring_project_id=None,
         pr_description_footer=CHANGED_LOADING_BEHAVIOUR_IN_DISTGIT_MESSAGE,
+        sync_acls=True,
     ).and_raise(
         PackitDownloadFailedException, "Failed to download source from example.com"
     ).once()
@@ -809,6 +814,7 @@ def test_dont_retry_propose_downstream_task(
         resolved_bugs=[],
         release_monitoring_project_id=None,
         pr_description_footer=CHANGED_LOADING_BEHAVIOUR_IN_DISTGIT_MESSAGE,
+        sync_acls=True,
     ).and_raise(
         PackitDownloadFailedException, "Failed to download source from example.com"
     ).once()

diff --git a/tests/unit/test_steve.py b/tests/unit/test_steve.py
@@ -168,6 +168,7 @@ def test_process_message(event, private, enabled_private_namespaces, success):
         resolved_bugs=[],
         release_monitoring_project_id=None,
         pr_description_footer=CHANGED_LOADING_BEHAVIOUR_IN_DISTGIT_MESSAGE,
+        sync_acls=True,
     ).and_return(pr).times(1 if success else 0)
     flexmock(shutil).should_receive("rmtree").with_args("")