-
Notifications
You must be signed in to change notification settings - Fork 123
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Print version on server startup #1556
Conversation
@moticless can you share the log of the server after your addition? In addition, I would like to verify that the appliance has this message as well in its log. Can you trigger an appliance build that has your change and add its log to the issue description? |
Thanks @moticless ! Can you please:
I'd like to see that in both cases it looks good. |
New INFO printout in other printouts context: DEBUG 2020/05/24 05:51:31 +0000 [pid=673] ^[[1m^[[36mSequel::Postgres::Database (0.5ms)^[[0m ^[[1mSELECT pg_advisory_unlock(767003715) AS "v" LIMIT 1^[[0m INFO 2020/05/24 05:51:31 +0000 [pid=727] CONJ00037I Conjur v1.6.0 starting up... DEBUG 2020/05/24 05:51:32 +0000 [pid=287] ^[[1m^[[35mSequel::Postgres::Database (0.4ms)^[[0m SELECT pg_try_advisory_lock(767003715) AS "v" LIMIT 1 |
@moticless please share the whole log of the startup so i can see the context. For example, on master the startup log looks like this:
To get this i ran |
Conjur logs along with CONJ000037I printout:
DAP logs along with CONJ000037I printout:
|
We would like to get the Conjur version from the logs to have a better supportability experience. Upon startup, we will write to the log the message: `Conjur v{0-conjur-version} starting up...`
58e6cc1
to
71dc05b
Compare
Code Climate has analyzed commit 71dc05b and detected 1 issue on this pull request. Here's the issue category breakdown:
The test coverage on the diff in this pull request is 100.0% (50% is the threshold). This pull request will bring the total coverage in the repository to 85.6% (0.0% change). View more on Code Climate. |
Connected to #1539
On Conjur startup, adding printout of "Conjur Version: XXXX", Based on file 'VERSION'. The new printout will be:
Conjur Version: 1.6.0
And in the context of other printouts: