diff --git a/lib/platform/github/index.js b/lib/platform/github/index.js index efc9dcd7c67069c084ed71337da2899c479624e1..8f5deb0fd67450d4ed1bbf8438407acfcab502f8 100644 --- a/lib/platform/github/index.js +++ b/lib/platform/github/index.js @@ -779,6 +779,7 @@ async function createPr( `repos/${config.parentRepo || config.repository}/pulls`, options )).body; + logger.info({ branchName, pr: pr.number }, 'PR created'); pr.displayNumber = `Pull Request #${pr.number}`; pr.branchName = branchName; await addLabels(pr.number, labels); @@ -912,6 +913,7 @@ async function updatePr(prNo, title, body) { `repos/${config.parentRepo || config.repository}/pulls/${prNo}`, options ); + logger.info({ pr: prNo }, 'PR updated'); } async function mergePr(prNo, branchName) { @@ -997,13 +999,13 @@ async function mergePr(prNo, branchName) { { err: err3 }, `Failed to ${options.body.merge_method} PR` ); - logger.info({ pr: prNo }, 'All merge attempts failed'); + logger.debug({ pr: prNo }, 'All merge attempts failed'); return false; } } } } - logger.info('Automerging succeeded'); + logger.info({ pr: prNo }, 'PR merged'); // Update base branch SHA config.baseCommitSHA = null; // Delete branch diff --git a/lib/platform/github/storage.js b/lib/platform/github/storage.js index a9bd11550a3687d788dee0d70476230748339d82..a6bb0cdf923e2a6b1fe30aed07e1c0228f015b1e 100644 --- a/lib/platform/github/storage.js +++ b/lib/platform/github/storage.js @@ -181,6 +181,7 @@ class Storage { }; try { await get.patch(url, options); + logger.info({ branchName }, 'Branch merged'); } catch (err) { logger.info( expandError(err), @@ -297,8 +298,10 @@ class Storage { try { if (isBranchExisting) { await updateBranch(branchName, commit); + logger.info({ branchName }, 'Branch updated'); } else { await createBranch(branchName, commit); + logger.info({ branchName }, 'Branch created'); } } catch (err) /* istanbul ignore next */ { logger.debug({ diff --git a/lib/workers/branch/automerge.js b/lib/workers/branch/automerge.js index 9981fef3d1deebf79a15781c187755c052759e56..eae321c1fc52d9964573d3e48e0105a60a634a7f 100644 --- a/lib/workers/branch/automerge.js +++ b/lib/workers/branch/automerge.js @@ -16,7 +16,7 @@ async function tryBranchAutomerge(config) { config.requiredStatusChecks ); if (branchStatus === 'success') { - logger.info(`Automerging branch`); + logger.debug(`Automerging branch`); try { await platform.mergeBranch(config.branchName); return 'automerged'; // Branch no longer exists diff --git a/lib/workers/pr/index.js b/lib/workers/pr/index.js index cb8ab04d58651672572f55a51a5256a4ec0f423b..18f2fc53ac9519a35a4e4d9abc01af3499493283 100644 --- a/lib/workers/pr/index.js +++ b/lib/workers/pr/index.js @@ -240,7 +240,7 @@ async function ensurePr(prConfig) { logger.info({ pr: existingPr.displayNumber }, `Updated PR`); return existingPr; } - logger.info({ branchName, prTitle }, `Creating PR`); + logger.debug({ branchName, prTitle }, `Creating PR`); // istanbul ignore if if (config.updateType === 'rollback') { logger.warn('Creating Rollback PR'); @@ -387,7 +387,7 @@ async function checkAutoMerge(pr, config) { return platform.ensureComment(pr.number, null, automergeComment); } // Let's merge this - logger.info(`Automerging #${pr.number}`); + logger.debug(`Automerging #${pr.number}`); return platform.mergePr(pr.number, branchName); } logger.debug('No automerge');