From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from smtp39.i.mail.ru (smtp39.i.mail.ru [94.100.177.99]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by dev.tarantool.org (Postfix) with ESMTPS id A71EB469710 for ; Wed, 10 Jun 2020 15:39:21 +0300 (MSK) Date: Wed, 10 Jun 2020 15:39:07 +0300 From: Alexander Turenko Message-ID: <20200610123907.tp5gskw66cacy23l@tkn_work_nb> References: <20200609114036.89486-1-arkholga@tarantool.org> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline In-Reply-To: <20200609114036.89486-1-arkholga@tarantool.org> Subject: Re: [Tarantool-patches] [PATCH v2] build: don't start example instance in postinstall List-Id: Tarantool development patches List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Olga Arkhangelskaia Cc: tarantool-patches@dev.tarantool.org LGTM. Pushed to master and 2.4. CCed Kirill. WBR, Alexander Turenko. On Tue, Jun 09, 2020 at 02:40:36PM +0300, Olga Arkhangelskaia wrote: > After tarantool installation on Debian/Ubuntu from repo, example instance > was automatically started on 3301 port. At the same time example instance > on RHEL/CentOS is started manually. Patch does the same for Debian/Ubuntu. Nit: Over 72 symbols. > > If tarantool is installed not for the first time, > do "apt-get purge tarantool-common" before installation to get rid of link in > /etc/tarantool/instances.enabled. Note: There is no such sentence in the commit on the branch. However the commit is okay without it too, so I would not bother much. > > Closes #4507 > --- > Brabch: gh-4507-disable-start-example-instance-full-ci I rebased and force-pushed it to verify again (just in case). https://gitlab.com/tarantool/tarantool/-/pipelines/154589256 release_asan_clang8 fail is the same as on master, so is not related to the patch. Other jobs are passed. Removed the branch. > @ChangeLog: > - disabled start of example instance after installing from Debian/Ubuntu > repo. Nit: Don't forget to include (gh-xxxx) at the end. > If there is no installed version of tarantool-common, one should do > "apt-get install tarantool-common". 'tarantool' package depends on 'tarantool-common', so it is not necessary. Moreover, when 'tarantool-common' is installed as the dependency for 'tarantool' package it is marked as 'auto' package and will be removed by `apt-get autoremove` (or `debfoster`) after removal of 'tarantool' package. See [1] for more information about autimatic dependencies. [1]: https://books.google.com/books?id=xmfTCgAAQBAJ&pg=PA115 > If tarantool-common was installed before, and even removed, > installation or upgrade won't take effect, because the link to example > instance still present in /etc/tarantool/instances.enabled. > Do following steps to get rid from example instance starting every time: > "apt-get purge tarantool-common" > "apt-get install tarantool-common" In order to avoid explicit (re)installation of tarantool-common I would suggest to stop the instance and remove the symlink manually: | # systemctl stop tarantool@example | # rm /etc/tarantool/instances.enabled/example.lua Having everything said above in the mind I added the following entry to the release notes: @ChangeLog - Don't start 'example' instance after installing tarantool (gh-4507). Before this release tarantool package for Debian and Ubuntu automatically enable and start 'example' instance, which listens on the TCP port 3301. Starting from this release the instance file is installed to /etc/tarantool/instances.available/example.lua, but is not enabled by default and not started anymore. One may perform the following actions to enable and start it: ``` # ln -s /etc/tarantool/instances.available/example.lua \ /etc/tarantool/instances.enabled/example.lua # systemctl start tarantool@example ``` Existing configuration will not be updated automatically at package update, so manual actions are required to stop and disable the instance (if it is not needed, of course): ``` # systemctl stop tarantool@example # rm /etc/tarantool/instances.enabled/example.lua ```