Skip to content

Commit

Permalink
Revert "fix: set limited permissions on package installs (#23683)" (#…
Browse files Browse the repository at this point in the history
…23851)

This reverts commit 8d5f0b5.
  • Loading branch information
jeffreyssmith2nd committed Nov 2, 2022
1 parent 6fc66ac commit 5b6fdbf
Show file tree
Hide file tree
Showing 4 changed files with 3 additions and 7 deletions.
4 changes: 2 additions & 2 deletions .circleci/package/control/postinst
Original file line number Diff line number Diff line change
Expand Up @@ -111,8 +111,8 @@ elif [[ -f /etc/debian_version ]]; then
# Moving these lines out of this if statement would make `rmp -V` fail after installation.
chown -R -L influxdb:influxdb $LOG_DIR
chown -R -L influxdb:influxdb $DATA_DIR
chmod 750 $LOG_DIR
chmod 750 $DATA_DIR
chmod 755 $LOG_DIR
chmod 755 $DATA_DIR

# Debian/Ubuntu logic
if command -v systemctl &>/dev/null; then
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,6 @@ KillMode=control-group
Restart=on-failure
Type=forking
PIDFile=/var/lib/influxdb/influxd.pid
UMask=0027

[Install]
WantedBy=multi-user.target
Expand Down
Empty file.
5 changes: 1 addition & 4 deletions .circleci/scripts/build-package
Original file line number Diff line number Diff line change
Expand Up @@ -58,10 +58,7 @@ function run_fpm()
--after-remove control/postrm \
`# package files` \
--chdir fs/ \
--package /artifacts \
--directories /var/lib/influxdb \
--rpm-defattrdir 750 \
--rpm-defattrfile 750
--package /artifacts

popd

Expand Down

0 comments on commit 5b6fdbf

Please sign in to comment.