Skip to content
Snippets Groups Projects
Commit 9b190967 authored by Rhys Arkins's avatar Rhys Arkins
Browse files

fix: filter out null docker digests

Closes #1739
parent 6feaeb7e
No related branches found
No related tags found
No related merge requests found
...@@ -27,7 +27,11 @@ async function getPackageUpdates(config) { ...@@ -27,7 +27,11 @@ async function getPackageUpdates(config) {
currentTag currentTag
); );
if (!newDigest) { if (!newDigest) {
logger.debug({ content: config.content }, 'Dockerfile no digest'); logger.warn(
{ dockerRegistry, depName, currentTag },
'Dockerfile no digest'
);
return [];
} }
if (newDigest && config.currentDigest !== newDigest) { if (newDigest && config.currentDigest !== newDigest) {
const upgrade = {}; const upgrade = {};
...@@ -144,6 +148,12 @@ async function getPackageUpdates(config) { ...@@ -144,6 +148,12 @@ async function getPackageUpdates(config) {
config.depName, config.depName,
upgrade.newTag upgrade.newTag
); );
if (!upgrade.newDigest) {
logger.warn(
{ dockerRegistry, depName, tag: upgrade.newTag },
'Dockerfile no digest'
);
}
upgrade.newFrom += `@${upgrade.newDigest}`; upgrade.newFrom += `@${upgrade.newDigest}`;
} }
if (newVersionMajor > currentMajor) { if (newVersionMajor > currentMajor) {
...@@ -160,7 +170,7 @@ async function getPackageUpdates(config) { ...@@ -160,7 +170,7 @@ async function getPackageUpdates(config) {
); );
} }
} }
return upgrades; return upgrades.filter(u => u.newDigest !== null);
} }
function isStable(tag, unstablePattern) { function isStable(tag, unstablePattern) {
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment