Skip to content

Commit 1ccde65

Browse files
committed
cl/logging: bump all shadow link state mutating request logs to info
1 parent 290e8f8 commit 1ccde65

File tree

1 file changed

+4
-6
lines changed

1 file changed

+4
-6
lines changed

src/v/redpanda/admin/services/shadow_link/shadow_link.cc

Lines changed: 4 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -32,7 +32,6 @@ shadow_link_service_impl::shadow_link_service_impl(
3232
ss::future<proto::admin::create_shadow_link_response>
3333
shadow_link_service_impl::create_shadow_link(
3434
serde::pb::rpc::context, proto::admin::create_shadow_link_request req) {
35-
vlog(sllog.trace, "create_shadow_link: {}", req);
3635
auto redirect_node = redirect_to(model::controller_ntp);
3736
if (redirect_node) {
3837
vlog(
@@ -45,7 +44,7 @@ shadow_link_service_impl::create_shadow_link(
4544
*redirect_node)
4645
.create_shadow_link(serde::pb::rpc::context{}, std::move(req));
4746
}
48-
47+
vlog(sllog.info, "create_shadow_link: {}", req);
4948
auto md = convert_create_to_metadata(std::move(req));
5049
auto get_resp = _service->local().get_cluster_link(md.name);
5150
if (get_resp.has_value()) {
@@ -65,7 +64,6 @@ shadow_link_service_impl::create_shadow_link(
6564
ss::future<proto::admin::delete_shadow_link_response>
6665
shadow_link_service_impl::delete_shadow_link(
6766
serde::pb::rpc::context ctx, proto::admin::delete_shadow_link_request req) {
68-
vlog(sllog.trace, "delete_shadow_link: {}", req);
6967
auto redirect_node = redirect_to(model::controller_ntp);
7068
if (redirect_node) {
7169
vlog(
@@ -78,7 +76,7 @@ shadow_link_service_impl::delete_shadow_link(
7876
*redirect_node)
7977
.delete_shadow_link(ctx, std::move(req));
8078
}
81-
79+
vlog(sllog.info, "delete_shadow_link: {}", req);
8280
handle_error(
8381
co_await _service->local().delete_cluster_link(
8482
cluster_link::model::name_t{req.get_name()}, req.get_force()));
@@ -147,7 +145,6 @@ shadow_link_service_impl::list_shadow_links(
147145
ss::future<proto::admin::update_shadow_link_response>
148146
shadow_link_service_impl::update_shadow_link(
149147
serde::pb::rpc::context ctx, proto::admin::update_shadow_link_request req) {
150-
vlog(sllog.trace, "update_shadow_link: {}", req);
151148
auto redirect_node = redirect_to(model::controller_ntp);
152149
if (redirect_node) {
153150
vlog(
@@ -161,6 +158,7 @@ shadow_link_service_impl::update_shadow_link(
161158
.update_shadow_link(ctx, std::move(req));
162159
}
163160

161+
vlog(sllog.info, "update_shadow_link: {}", req);
164162
auto link_name = cluster_link::model::name_t{
165163
req.get_shadow_link().get_name()};
166164

@@ -187,7 +185,6 @@ shadow_link_service_impl::update_shadow_link(
187185
ss::future<proto::admin::fail_over_response>
188186
shadow_link_service_impl::fail_over(
189187
serde::pb::rpc::context ctx, proto::admin::fail_over_request req) {
190-
vlog(sllog.trace, "fail_over_request: {}", req);
191188
auto redirect_node = redirect_to(model::controller_ntp);
192189
if (redirect_node) {
193190
vlog(
@@ -200,6 +197,7 @@ shadow_link_service_impl::fail_over(
200197
*redirect_node)
201198
.fail_over(ctx, std::move(req));
202199
}
200+
vlog(sllog.info, "fail_over_request: {}", req);
203201
auto link_name = cluster_link::model::name_t{req.get_name()};
204202
auto current_link = handle_error(
205203
_service->local().get_cluster_link(link_name));

0 commit comments

Comments
 (0)