Skip to content
Merged
Show file tree
Hide file tree
Changes from 1 commit
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
fmt fix
  • Loading branch information
k-anshul committed Apr 24, 2026
commit 3893d8bb904028f8fa3fde8cc23597762ceefe51
2 changes: 1 addition & 1 deletion proto/rill/runtime/v1/api.proto
Original file line number Diff line number Diff line change
Expand Up @@ -546,7 +546,7 @@ message ReloadConfigRequest {

// Response message for RuntimeService.ReloadConfig
message ReloadConfigResponse {
// Number of variables pulled from admin service
// Number of variables pulled from admin service
int32 variables_count = 1;
// Whether the .env file(s) were modified
bool modified = 2;
Expand Down
2 changes: 1 addition & 1 deletion runtime/server/instances.go
Original file line number Diff line number Diff line change
Expand Up @@ -311,7 +311,7 @@ func (s *Server) ReloadConfig(ctx context.Context, req *runtimev1.ReloadConfigRe
}
// Ideally pullEnv should be called inside ReloadConfig only since it is just a simple version of ReloadConfig on local
// The issue is that `adminOverride` is available in server and not in runtime
// TODO: revisit this when relooking adminOverride
// TODO: revisit this when relooking adminOverride
count, modified, err := s.pullEnv(ctx, req.InstanceId)
Comment thread
k-anshul marked this conversation as resolved.
Outdated
if err != nil {
return nil, err
Expand Down
Loading