StackStorm · cognifloyd · Dec 30, 2022 · Apr 13, 2021 · May 17, 2021 · May 18, 2021
diff --git a/CHANGELOG.rst b/CHANGELOG.rst
@@ -14,6 +14,7 @@ Added
   working on StackStorm, improve our security posture, and improve CI reliability thanks in part
   to pants' use of PEX lockfiles. This is not a user-facing addition.
   #5778 #5789 #5817 #5795 #5830 #5833 #5834 #5841 #5840 #5838 #5842 #5837 #5849 #5850
+  #5846
   Contributed by @cognifloyd
 
 * Added a joint index to solve the problem of slow mongo queries for scheduled executions. #5805

diff --git a/conf/BUILD b/conf/BUILD
@@ -0,0 +1,56 @@
+file(
+    name="st2client_sample_config",
+    source="st2rc.sample.ini",
+)
+
+file(
+    name="st2.conf.sample",
+    source="st2.conf.sample",
+)
+
+file(
+    name="logrotate",
+    source="logrotate.conf",
+)
+
+file(
+    name="nginx_sample_config",
+    source="nginx/st2.conf",
+)
+
+file(
+    name="st2_kvstore_demo_crypto_key",
+    source="st2_kvstore_demo.crypto.key.json",
+)
+
+files(
+    name="st2_tests_conf",
+    sources=[
+        "st2.tests*.conf",
+    ],
+    dependencies=[
+        "st2auth/conf:htpasswd",
+        "st2actions/conf:logging",
+        "st2reactor/conf:logging",
+        "st2tests/conf:other_logging_conf",
+    ],
+)
+
+file(
+    name="st2_dev_conf",
+    source="st2.dev.conf",
+    dependencies=[
+        ":st2_kvstore_demo_crypto_key",
+        "st2auth/conf:htpasswd",
+        "st2actions/conf:logging",
+        "st2api/conf:logging",
+        "st2auth/conf:logging",
+        "st2reactor/conf:logging",
+        "st2stream/conf:logging",
+    ],
+)
+
+file(
+    name="st2_package_conf",
+    source="st2.package.conf",
+)
diff --git a/contrib/chatops/tests/BUILD b/contrib/chatops/tests/BUILD
@@ -1,3 +1,10 @@
+files(
+    name="fixtures",
+    sources=["fixtures/*.json"],
+)
+
 python_tests(
+    name="tests",
+    dependencies=[":fixtures"],
     skip_pylint=True,
 )
diff --git a/contrib/runners/action_chain_runner/action_chain_runner/BUILD b/contrib/runners/action_chain_runner/action_chain_runner/BUILD
@@ -1 +1,8 @@
-python_sources()
+resource(
+    name="runner_metadata",
+    source="runner.yaml",
+)
+
+python_sources(
+    dependencies=[":runner_metadata"],
+)
diff --git a/contrib/runners/announcement_runner/announcement_runner/BUILD b/contrib/runners/announcement_runner/announcement_runner/BUILD
@@ -1 +1,8 @@
-python_sources()
+resource(
+    name="runner_metadata",
+    source="runner.yaml",
+)
+
+python_sources(
+    dependencies=[":runner_metadata"],
+)
diff --git a/contrib/runners/http_runner/http_runner/BUILD b/contrib/runners/http_runner/http_runner/BUILD
@@ -1 +1,8 @@
-python_sources()
+resource(
+    name="runner_metadata",
+    source="runner.yaml",
+)
+
+python_sources(
+    dependencies=[":runner_metadata"],
+)
diff --git a/contrib/runners/inquirer_runner/inquirer_runner/BUILD b/contrib/runners/inquirer_runner/inquirer_runner/BUILD
@@ -1 +1,8 @@
-python_sources()
+resource(
+    name="runner_metadata",
+    source="runner.yaml",
+)
+
+python_sources(
+    dependencies=[":runner_metadata"],
+)
diff --git a/contrib/runners/local_runner/local_runner/BUILD b/contrib/runners/local_runner/local_runner/BUILD
@@ -1 +1,8 @@
-python_sources()
+resource(
+    name="runner_metadata",
+    source="runner.yaml",
+)
+
+python_sources(
+    dependencies=[":runner_metadata"],
+)
diff --git a/contrib/runners/noop_runner/noop_runner/BUILD b/contrib/runners/noop_runner/noop_runner/BUILD
@@ -1 +1,8 @@
-python_sources()
+resource(
+    name="runner_metadata",
+    source="runner.yaml",
+)
+
+python_sources(
+    dependencies=[":runner_metadata"],
+)
diff --git a/contrib/runners/orquesta_runner/orquesta_runner/BUILD b/contrib/runners/orquesta_runner/orquesta_runner/BUILD
@@ -1 +1,8 @@
-python_sources()
+resource(
+    name="runner_metadata",
+    source="runner.yaml",
+)
+
+python_sources(
+    dependencies=[":runner_metadata"],
+)
diff --git a/contrib/runners/python_runner/python_runner/BUILD b/contrib/runners/python_runner/python_runner/BUILD
@@ -1 +1,8 @@
-python_sources()
+resource(
+    name="runner_metadata",
+    source="runner.yaml",
+)
+
+python_sources(
+    dependencies=[":runner_metadata"],
+)
diff --git a/contrib/runners/remote_runner/remote_runner/BUILD b/contrib/runners/remote_runner/remote_runner/BUILD
@@ -1 +1,8 @@
-python_sources()
+resource(
+    name="runner_metadata",
+    source="runner.yaml",
+)
+
+python_sources(
+    dependencies=[":runner_metadata"],
+)
diff --git a/contrib/runners/winrm_runner/tests/unit/fixtures/BUILD b/contrib/runners/winrm_runner/tests/unit/fixtures/BUILD
@@ -1 +1,8 @@
-python_sources()
+files(
+    name="powershell",
+    sources=["*.ps1"],
+)
+
+python_sources(
+    dependencies=[":powershell"],
+)
diff --git a/contrib/runners/winrm_runner/winrm_runner/BUILD b/contrib/runners/winrm_runner/winrm_runner/BUILD
@@ -1 +1,8 @@
-python_sources()
+resource(
+    name="runner_metadata",
+    source="runner.yaml",
+)
+
+python_sources(
+    dependencies=[":runner_metadata"],
+)
diff --git a/scripts/github/BUILD b/scripts/github/BUILD
@@ -1 +1,11 @@
-shell_sources()
+files(
+    name="assets",
+    sources=[
+        "*.txt",
+        "*.conf",
+    ],
+)
+
+shell_sources(
+    dependencies=[":assets"],
+)
diff --git a/st2actions/conf/BUILD b/st2actions/conf/BUILD
@@ -0,0 +1,14 @@
+file(
+    name="logging_console",
+    source="console.conf",
+)
+
+files(
+    name="logging",
+    sources=["logging*.conf"],
+)
+
+files(
+    name="logging_syslog",
+    sources=["syslog*.conf"],
+)
diff --git a/st2api/conf/BUILD b/st2api/conf/BUILD
@@ -0,0 +1,19 @@
+file(
+    name="logging_console",
+    source="console.conf",
+)
+
+file(
+    name="logging",
+    source="logging.conf",
+)
+
+file(
+    name="logging_gunicorn",
+    source="logging.gunicorn.conf",
+)
+
+file(
+    name="logging_syslog",
+    source="syslog.conf",
+)
diff --git a/st2api/st2api/public/BUILD b/st2api/st2api/public/BUILD
@@ -0,0 +1,5 @@
+files(
+    sources=[
+        "images/*.png",
+    ],
+)
diff --git a/st2api/st2api/templates/BUILD b/st2api/st2api/templates/BUILD
@@ -0,0 +1 @@
+file(source="index.html")
diff --git a/st2api/tests/integration/BUILD b/st2api/tests/integration/BUILD
@@ -1,3 +1,6 @@
 python_tests(
     name="tests",
+    dependencies=[
+        "conf/st2.tests.conf:st2_tests_conf",
+    ],
 )
diff --git a/st2auth/conf/BUILD b/st2auth/conf/BUILD
@@ -0,0 +1,29 @@
+file(
+    name="apache_sample_conf",
+    source="apache.sample.conf",
+)
+
+file(
+    name="htpasswd",
+    source="htpasswd_dev",
+)
+
+file(
+    name="logging_console",
+    source="console.conf",
+)
+
+file(
+    name="logging",
+    source="logging.conf",
+)
+
+file(
+    name="logging_gunicorn",
+    source="logging.gunicorn.conf",
+)
+
+file(
+    name="logging_syslog",
+    source="syslog.conf",
+)
diff --git a/st2client/tests/fixtures/BUILD b/st2client/tests/fixtures/BUILD
@@ -1 +1,13 @@
-python_sources()
+resources(
+    name="execution_fixtures",
+    sources=["execution*.json", "execution*.txt"],
+)
+
+resources(
+    name="st2client_ini",
+    sources=["*.ini"],
+)
+
+python_sources(
+    dependencies=[":execution_fixtures"],
+)
diff --git a/st2client/tests/unit/BUILD b/st2client/tests/unit/BUILD
@@ -1,6 +1,7 @@
 python_tests(
     name="tests",
     dependencies=[
+        "st2client/tests/fixtures:st2client_ini",
         # most files import tests.base which is ambiguous. Tell pants which one to use.
         "st2client/tests/base.py",
     ],

diff --git a/st2common/benchmarks/fixtures/json/BUILD b/st2common/benchmarks/fixtures/json/BUILD
@@ -0,0 +1,3 @@
+files(
+    sources=["*.json"],
+)
diff --git a/st2common/benchmarks/micro/BUILD b/st2common/benchmarks/micro/BUILD
@@ -1,4 +1,5 @@
 python_sources(
+    dependencies=["st2common/benchmarks/fixtures/json"],
     skip_pylint=True,
 )
 

diff --git a/st2common/st2common/BUILD b/st2common/st2common/BUILD
@@ -1 +1,11 @@
-python_sources()
+python_sources(
+    dependencies=[
+        ":openapi_spec",
+    ]
+)
+
+# These may be loaded with st2common.util.spec_loader
+resources(
+    name="openapi_spec",
+    sources=["openapi.yaml", "openapi.yaml.j2"],
+)
diff --git a/st2common/st2common/bootstrap/BUILD b/st2common/st2common/bootstrap/BUILD
@@ -1 +1,9 @@
-python_sources()
+python_sources(
+    overrides={
+        "policiesregistrar.py": {
+            "dependencies": [
+                "st2common/st2common/policies/meta:policies_meta",
+            ],
+        },
+    },
+)
diff --git a/st2common/st2common/conf/BUILD b/st2common/st2common/conf/BUILD
@@ -0,0 +1,5 @@
+# used in st2common.constants.logging.DEFAULT_LOGGING_CONF_PATH
+resource(
+    name="base.logging.conf",
+    source="base.logging.conf",
+)
diff --git a/st2common/st2common/constants/BUILD b/st2common/st2common/constants/BUILD
@@ -1 +1,5 @@
-python_sources()
+python_sources(
+    dependencies=[
+        "st2common/st2common/conf:base.logging.conf",
+    ]
+)
diff --git a/st2common/st2common/policies/meta/BUILD b/st2common/st2common/policies/meta/BUILD
@@ -0,0 +1,5 @@
+# These are loaded by st2common.bootstrap.policiesregistrar
+resources(
+    name="policies_meta",
+    sources=["*.yaml"],
+)
diff --git a/st2common/st2common/util/schema/BUILD b/st2common/st2common/util/schema/BUILD
@@ -1 +1,10 @@
-python_sources()
+python_sources(
+    dependencies=[
+        ":jsonschema",
+    ]
+)
+
+resources(
+    name="jsonschema",
+    sources=["*.json"],
+)
diff --git a/st2common/tests/integration/BUILD b/st2common/tests/integration/BUILD
@@ -2,4 +2,9 @@ python_sources()
 
 python_tests(
     name="tests",
+    dependencies=[
+        # used by test_register_content_script
+        "conf/st2.tests.conf:st2_tests_conf",
+        "conf/st2.tests1.conf:st2_tests_conf",
+    ],
 )
diff --git a/st2common/tests/unit/test_util_file_system.py b/st2common/tests/unit/test_util_file_system.py
@@ -34,6 +34,7 @@ def test_get_file_list(self):
             "mock_exception.py",
             "concurrency.py",
             "__init__.py",
+            "meta/BUILD",
             "meta/mock_exception.yaml",
             "meta/concurrency.yaml",
             "meta/__init__.py",
@@ -49,6 +50,6 @@ def test_get_file_list(self):
             "meta/__init__.py",
         ]
         result = get_file_list(
-            directory=directory, exclude_patterns=["*.pyc", "*.yaml", "BUILD"]
+            directory=directory, exclude_patterns=["*.pyc", "*.yaml", "*BUILD"]
         )
         self.assertItemsEqual(expected, result)