Merged pull request influxdata#1660 from influxdata/nc-tar-root

Force tar files to be owner by root and group root.

CHANGELOG.md

@@ -45,6 +45,7 @@
 - [#1630](https://github.com/influxdata/kapacitor/issues/1630): Remove the pidfile after the server has exited.
 - [#1641](https://github.com/influxdata/kapacitor/issues/1641): Logs API writes multiple http headers.
 - [#1657](https://github.com/influxdata/kapacitor/issues/1657): Fix missing dependency in rpm package.
+- [#1660](https://github.com/influxdata/kapacitor/pull/1660): Force tar owner/group to be root.
 
 ## v1.3.3 [2017-08-11]
 

build.py

@@ -242,7 +242,7 @@ def package_python_udf(version, dist_dir):
     fname = "python-kapacitor_udf-{}.tar.gz".format(version)
     outfile = os.path.join(dist_dir, fname)
 
-    tar_cmd = ['tar', '-cz', '-C', './udf/agent/py', '--transform', 's/^./kapacitor_udf-{}/'.format(version), '-f']
+    tar_cmd = ['tar', '-cz', '-C', './udf/agent/py', '--owner=root', '--group=root', '--transform', 's/^./kapacitor_udf-{}/'.format(version), '-f']
     tar_cmd.append(outfile)
     exclude_list = ['*.pyc', '*.pyo', '__pycache__']
     for e in exclude_list:
@@ -724,7 +724,7 @@ def package(build_output, pkg_name, version, nightly=False, iteration=1, static=
                                                             package_arch)
                         current_location = os.path.join(os.getcwd(), current_location)
                         if package_type == 'tar':
-                            tar_command = "cd {} && tar -cvzf {}.tar.gz ./*".format(package_build_root, name)
+                            tar_command = "cd {} && tar -cvzf {}.tar.gz --owner=root --group=root ./*".format(package_build_root, name)
                             run(tar_command, shell=True)
                             run("mv {}.tar.gz {}".format(os.path.join(package_build_root, name), current_location), shell=True)
                             outfile = os.path.join(current_location, name + ".tar.gz")