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.

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

gce: drop rsyslog #469

Merged
merged 1 commit into from
Aug 14, 2023
Merged

gce: drop rsyslog #469

merged 1 commit into from
Aug 14, 2023

Conversation

syuu1228
Copy link
Contributor

On AWS and Azure Ubuntu Minimal image we don't have rsyslog, it just uses persistent journal log, but only GCE image has rsyslog. We want to align image configuration between clouds, let's drop rsyslog from GCE image.

Related scylladb/scylla-enterprise#3080

On AWS and Azure Ubuntu Minimal image we don't have rsyslog, it just
uses persistent journal log, but only GCE image has rsyslog.
We want to align image configuration between clouds, let's drop rsyslog
from GCE image.

Related scylladb/scylla-enterprise#3080
@yaronkaikov
Copy link
Collaborator

@syuu1228 Let's run https://jenkins.scylladb.com/job/scylla-master/job/releng-testing/job/gce-image/ from your branch to verify nothing is broken

@yaronkaikov
Copy link
Collaborator

@mykaul
Copy link

mykaul commented Aug 14, 2023

@d-helios - do you need rsyslog (IIRC, you were going to use Vector as log shipper?)

@d-helios
Copy link

d-helios commented Aug 14, 2023

@d-helios - do you need rsyslog (IIRC, you were going to use Vector as log shipper?)

we do not use rsyslog now
and yes we are going to use vector for scylla as well

@yaronkaikov
Copy link
Collaborator

https://jenkins.scylladb.com/job/scylla-master/job/releng-testing/job/gce-image/80/ . let's see if this will pass

Verification passed

@yaronkaikov yaronkaikov merged commit a63350d into scylladb:next Aug 14, 2023
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants