test: skip test_gitlint on PR #5048

PR #5048 is the merge of the long-lived PVH feature branch. The commits on this branch were made long before we changes the gitlint rules to more closely follow Linux rules when it comes to sign-offs from co-authors (as used to not only not require them, but not allow them in the first place, meaning the first 3 commit in this PR are only signed by of Colin and me, but not the coauthor from a few years ago). Explicitly skip this test for this one PR. Signed-off-by: Patrick Roy <[email protected]>
firecracker-microvm · Feb 25, 2025 · d17a274 · d17a274
1 parent b02a97a
commit d17a274
Showing 1 changed file with 6 additions and 0 deletions.
diff --git a/tests/integration_tests/style/test_gitlint.py b/tests/integration_tests/style/test_gitlint.py
@@ -4,10 +4,16 @@
 
 import os
 
+import pytest
+
 from framework import utils
 from framework.ab_test import DEFAULT_A_REVISION
 
 
+@pytest.mark.skipif(
+    os.environ.get("BUILDKITE_PULL_REQUEST") == "5048",
+    reason="PR of a feature branch from before this test was modified to follow Linux sign-off rules for co-authors.",
+)
 def test_gitlint():
     """
     Test that all commit messages pass the gitlint rules.