Skip to content

Commit

Permalink
Remove logLevel from server config
Browse files Browse the repository at this point in the history
Where we need it, take it from the environment.
  • Loading branch information
Andrew Farries authored and roboquat committed Apr 29, 2022
1 parent 5d99db0 commit 5509d9e
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 3 deletions.
4 changes: 3 additions & 1 deletion components/server/ee/src/prebuilds/github-app.ts
Original file line number Diff line number Diff line change
Expand Up @@ -67,12 +67,14 @@ export class GithubApp {
@inject(PrebuildStatusMaintainer) protected readonly statusMaintainer: PrebuildStatusMaintainer,
) {
if (config.githubApp?.enabled) {
const logLevel = LogrusLogLevel.getFromEnv() ?? "info";

this.server = new Server({
Probot: Probot.defaults({
appId: config.githubApp.appId,
privateKey: GithubApp.loadPrivateKey(config.githubApp.certPath),
secret: config.githubApp.webhookSecret,
logLevel: GithubApp.mapToGitHubLogLevel(config.logLevel),
logLevel: GithubApp.mapToGitHubLogLevel(logLevel),
baseUrl: config.githubApp.baseUrl,
}),
});
Expand Down
3 changes: 1 addition & 2 deletions components/server/src/config.ts
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@ import { CodeSyncConfig } from "./code-sync/code-sync-service";
import { ChargebeeProviderOptions, readOptionsFromFile } from "@gitpod/gitpod-payment-endpoint/lib/chargebee";
import * as fs from "fs";
import * as yaml from "js-yaml";
import { log, LogrusLogLevel } from "@gitpod/gitpod-protocol/lib/util/logging";
import { log } from "@gitpod/gitpod-protocol/lib/util/logging";
import { filePathTelepresenceAware } from "@gitpod/gitpod-protocol/lib/env";

export const Config = Symbol("Config");
Expand Down Expand Up @@ -52,7 +52,6 @@ export interface ConfigSerialized {
installationShortname: string;
devBranch?: string;
insecureNoDomain: boolean;
logLevel: LogrusLogLevel;

// Use one or other - licenseFile reads from a file and populates license
license?: string;
Expand Down

0 comments on commit 5509d9e

Please sign in to comment.