Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Scylla housekeeping with systemd, should use r mode for restart and d mode for daily check #1953

Closed
amnonh opened this issue Dec 21, 2016 · 2 comments
Closed

Scylla housekeeping with systemd, should use r mode for restart and d mode for daily check #1953

amnonh opened this issue Dec 21, 2016 · 2 comments
Assignees
@syuu1228
Labels
feature/enhancement
Milestone
2.0

Comments

@amnonh
Copy link
Contributor

amnonh commented Dec 21, 2016

Version 1.6 and up

In ubuntu 14, the timer script uses r mode for checking the version during restart and d mode for checking the version once a day.

This is the better behavior and should be the same for systemd support
@slivne
Copy link
Contributor

slivne commented Dec 27, 2016

@amnonh what is r mode and what is d mode - can you provide a bit more information

@slivne slivne added this to the Yellowfin milestone Dec 27, 2016
@amnonh
Copy link
Contributor Author

amnonh commented Dec 27, 2016

the housekeeping gets a mode parameter, it needs to get the right mode according to the state of the request

@slivne slivne modified the milestones: Yellowfin, 1.8 Jan 19, 2017
avikivity pushed a commit that referenced this issue Feb 5, 2017
@syuu1228 @avikivity
dist/common/systemd: introduce scylla-housekeeping restart mode
e4b3f02 
scylla-housekeeping requires to run 'restart mode' for check the version during
scylla-server restart, which wasn't called on systemd timer so added it.

Existing scylla-housekeeping.timer renamed to scylla-housekeeping-daily.timer,
since it is running 'daily mode'.

Fixes #1953

Signed-off-by: Takuya ASADA <syuu@scylladb.com>
Message-Id: <1486180031-18093-1-git-send-email-syuu@scylladb.com>
(cherry picked from commit e82932b)
benipeled pushed a commit to benipeled/scylla that referenced this issue May 31, 2022
@avikivity
Merge 'transport/system_ks: Add more info to system.clients' from J…
698012f 
…uliusz Stasiewicz

This patch fills the following columns in `system.clients` table:
* `connection_stage`
* `driver_name`
* `driver_version`
* `protocol_version`

It also improves:
* `client_type` - distinguishes cql from thrift just in case
* `username` - now it displays correct username iff `PasswordAuthenticator` is configured.

What is still missing:
* SSL params (I'll happily get some advice here)
* `hostname` - I didn't find it in tested drivers

Refs scylladb#6946

Closes scylladb#7349

* github.com:scylladb/scylla:
  transport: Update `connection_stage` in `system.clients`
  transport: Retrieve driver's name and version from STARTUP message
  transport: Notify `system.clients` about "protocol_version"
  transport: On successful authentication add `username` to system.clients

(cherry picked from commit e2a02f1)

Closes scylladb#1958

Ref scylladb#1953.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@syuu1228 syuu1228

Labels
feature/enhancement
Projects
None yet
Milestone
2.0
Development

No branches or pull requests
3 participants
@syuu1228 @amnonh @slivne