fix: reuse current env hash registry tags
This commit is contained in:
+59
-14
@@ -130,11 +130,10 @@ export async function createCiPlan(options = {}) {
|
||||
const codeInputHash = envReuse ? await hashGitPaths(repoRoot, targetRef, codeInputPaths, {
|
||||
skipPath: (filePath) => isTestOnlyPath(filePath) || isDocsOnlyPath(filePath)
|
||||
}) : null;
|
||||
const environmentImage = envReuse ? environmentImageFromCatalog(model.serviceId, catalogRecord) : null;
|
||||
const environmentDigest = envReuse ? environmentDigestFromCatalog(catalogRecord) : null;
|
||||
const environmentReady = envReuse && /^sha256:[a-f0-9]{64}$/u.test(environmentDigest ?? "") && Boolean(environmentImage);
|
||||
const environmentImage = envReuse ? envReuseImageRef(registryPrefix, model.serviceId, environmentInputHash) : null;
|
||||
let environmentDigest = envReuse ? environmentDigestFromCatalogForHash(catalogRecord, environmentInputHash) : null;
|
||||
const environmentInputChanged = envReuse ? catalogRecord?.environmentInputHash !== environmentInputHash : null;
|
||||
const reuseRegistry = envReuse && reuseRegistryProbe && environmentReady
|
||||
const reuseRegistry = envReuse && reuseRegistryProbe && environmentImage
|
||||
? await reuseRegistryProbe({
|
||||
serviceId: model.serviceId,
|
||||
image: environmentImage,
|
||||
@@ -143,8 +142,13 @@ export async function createCiPlan(options = {}) {
|
||||
timeoutMs: Number.isFinite(registryProbeTimeoutMs) ? registryProbeTimeoutMs : 3000
|
||||
})
|
||||
: null;
|
||||
if (reuseRegistry?.status === "present" && /^sha256:[a-f0-9]{64}$/u.test(reuseRegistry.digest ?? "")) {
|
||||
environmentDigest = reuseRegistry.digest;
|
||||
}
|
||||
const reuseRegistryUnavailable = Boolean(reuseRegistry && reuseRegistry.status !== "present");
|
||||
const envChanged = envReuse ? Boolean(relevantEnvMatches.length > 0 || !environmentReady || environmentInputChanged || reuseRegistryUnavailable) : null;
|
||||
const digestReady = /^sha256:[a-f0-9]{64}$/u.test(environmentDigest ?? "");
|
||||
const environmentReady = envReuse && Boolean(environmentImage) && (reuseRegistryProbe ? reuseRegistry?.status === "present" && digestReady : !environmentInputChanged && digestReady);
|
||||
const envChanged = envReuse ? Boolean(relevantEnvMatches.length > 0 || !environmentReady || reuseRegistryUnavailable) : null;
|
||||
const codeChanged = envReuse ? relevantCodeMatches.length > 0 || catalogRecord?.codeInputHash !== codeInputHash : null;
|
||||
const componentInputPaths = uniqueSorted([
|
||||
...model.componentPaths,
|
||||
@@ -188,7 +192,7 @@ export async function createCiPlan(options = {}) {
|
||||
} : catalogRecord;
|
||||
const componentInputChanged = !envReuse && catalogComponentProvenance?.safeToReuse !== true;
|
||||
const catalogReuse = envReuse
|
||||
? envReuseCandidate(catalogRecord, envChanged)
|
||||
? envReuseCandidate({ catalogRecord, envChanged, environmentImage, environmentDigest, environmentInputHash })
|
||||
: reuseCandidate(catalogRecordForReuse, imageRelevantChangedPaths.length > 0, componentInputChanged);
|
||||
const affected = envReuse
|
||||
? Boolean(envChanged || codeChanged || runtimeConfigChanged)
|
||||
@@ -494,7 +498,7 @@ function effectiveDownloadStackConfig(value, env = process.env) {
|
||||
if (!configured || !usesK8sNativeDownloadProxy(env)) return configured;
|
||||
const httpProxy = normalizeDeclarationString(env.HTTP_PROXY) || normalizeDeclarationString(env.http_proxy) || configured.httpProxy;
|
||||
const httpsProxy = normalizeDeclarationString(env.HTTPS_PROXY) || normalizeDeclarationString(env.https_proxy) || httpProxy || configured.httpsProxy;
|
||||
const noProxy = noProxyFromEnv(env) || configured.noProxy;
|
||||
const noProxy = sanitizeK8sDownloadNoProxy(noProxyFromEnv(env) || configured.noProxy);
|
||||
return {
|
||||
...configured,
|
||||
httpProxy,
|
||||
@@ -513,6 +517,22 @@ function noProxyFromEnv(env = process.env) {
|
||||
return text.split(",").map((item) => item.trim()).filter(Boolean);
|
||||
}
|
||||
|
||||
function sanitizeK8sDownloadNoProxy(values) {
|
||||
return uniquePreserveOrder(normalizeStringList(values, []).filter(isK8sDownloadNoProxyEntry));
|
||||
}
|
||||
|
||||
function isK8sDownloadNoProxyEntry(value) {
|
||||
const item = String(value ?? "").trim().toLowerCase();
|
||||
if (!item || item === "*") return false;
|
||||
if (["localhost", "::1", "[::1]", "hyueapi.com", ".hyueapi.com"].includes(item)) return true;
|
||||
if (item.endsWith(".hyueapi.com")) return true;
|
||||
if (item === ".svc" || item === ".svc.cluster.local" || item === ".cluster.local") return true;
|
||||
if (item.endsWith(".svc") || item.endsWith(".svc.cluster.local") || item.endsWith(".cluster.local")) return true;
|
||||
if (/^(127|10)\./u.test(item) || /^192\.168\./u.test(item) || /^172\.(1[6-9]|2\d|3[01])\./u.test(item)) return true;
|
||||
if (/^(127|10)\.[0-9./:-]+$/u.test(item) || /^192\.168\.[0-9./:-]+$/u.test(item) || /^172\.(1[6-9]|2\d|3[01])\.[0-9./:-]+$/u.test(item)) return true;
|
||||
return false;
|
||||
}
|
||||
|
||||
function normalizeOptionalHttpUrl(value, label) {
|
||||
const text = normalizeDeclarationString(value);
|
||||
if (!text) return null;
|
||||
@@ -896,8 +916,16 @@ function text(value) {
|
||||
return String(value ?? "").trim();
|
||||
}
|
||||
|
||||
function envReuseCandidate(catalogRecord, envChanged) {
|
||||
function envReuseCandidate({ catalogRecord, envChanged, environmentImage, environmentDigest, environmentInputHash }) {
|
||||
if (envChanged) return { status: "not-reused", reason: "environment-affected" };
|
||||
if (environmentImage && /^sha256:[a-f0-9]{64}$/u.test(environmentDigest ?? "")) {
|
||||
return {
|
||||
status: "ready",
|
||||
image: environmentImage,
|
||||
digest: environmentDigest,
|
||||
reusedFrom: environmentInputHash ?? catalogRecord?.environmentInputHash ?? catalogRecord?.commitId ?? catalogRecord?.imageTag ?? null
|
||||
};
|
||||
}
|
||||
if (!catalogRecord) return { status: "candidate-no-catalog", reason: "no-previous-artifact-record" };
|
||||
const image = environmentImageFromCatalog(catalogRecord.serviceId, catalogRecord);
|
||||
const digest = environmentDigestFromCatalog(catalogRecord);
|
||||
@@ -923,6 +951,15 @@ function environmentDigestFromCatalog(catalogRecord) {
|
||||
return catalogRecord?.environmentDigest ?? (catalogRecord?.environmentImage ? catalogRecord?.digest : null) ?? null;
|
||||
}
|
||||
|
||||
function environmentDigestFromCatalogForHash(catalogRecord, environmentInputHash) {
|
||||
return catalogRecord?.environmentInputHash === environmentInputHash ? environmentDigestFromCatalog(catalogRecord) : null;
|
||||
}
|
||||
|
||||
function envReuseImageRef(registryPrefix, serviceId, environmentInputHash) {
|
||||
const tag = String(environmentInputHash || "unknown").slice(0, 12);
|
||||
return `${registryPrefix}/${serviceId}-env:env-${tag}`;
|
||||
}
|
||||
|
||||
function digestFromImageReference(image) {
|
||||
const match = String(image ?? "").match(/@(sha256:[a-f0-9]{64})$/u);
|
||||
return match ? match[1] : null;
|
||||
@@ -936,21 +973,22 @@ async function probeRegistryManifest({ image, digest, timeoutMs = 3000 }) {
|
||||
const request = registryManifestRequest(image, digest);
|
||||
if (!request) return { status: "invalid-reference", image, digest, reason: "registry-reference-unparseable" };
|
||||
const result = await registryManifestHttpProbe(request, "HEAD", timeoutMs);
|
||||
if (result.status === "present" || result.status === "missing") return { ...result, image, digest };
|
||||
if (result.status === "present" || result.status === "missing") return { ...result, image, digest: result.digest ?? digest ?? null };
|
||||
const fallback = await registryManifestHttpProbe(request, "GET", timeoutMs);
|
||||
return { ...fallback, image, digest };
|
||||
return { ...fallback, image, digest: fallback.digest ?? digest ?? null };
|
||||
}
|
||||
|
||||
function registryManifestRequest(image, digest) {
|
||||
const parsed = parseTaggedImage(image);
|
||||
if (!parsed || !/^sha256:[a-f0-9]{64}$/u.test(digest ?? "")) return null;
|
||||
const reference = /^sha256:[a-f0-9]{64}$/u.test(digest ?? "") ? digest : parsed?.tag;
|
||||
if (!parsed || !reference) return null;
|
||||
const protocol = parsed.host === "127.0.0.1" || parsed.host.startsWith("127.") || parsed.host === "localhost" || parsed.host.includes(":5000")
|
||||
? "http:"
|
||||
: "https:";
|
||||
return {
|
||||
protocol,
|
||||
host: parsed.host,
|
||||
path: `/v2/${parsed.repository}/manifests/${digest}`
|
||||
path: `/v2/${parsed.repository}/manifests/${reference}`
|
||||
};
|
||||
}
|
||||
|
||||
@@ -964,7 +1002,8 @@ function parseTaggedImage(image) {
|
||||
if (!host || tagSeparator <= 0 || rest.includes("@")) return null;
|
||||
return {
|
||||
host,
|
||||
repository: rest.slice(0, tagSeparator)
|
||||
repository: rest.slice(0, tagSeparator),
|
||||
tag: rest.slice(tagSeparator + 1)
|
||||
};
|
||||
}
|
||||
|
||||
@@ -990,7 +1029,13 @@ function registryManifestHttpProbe(request, method, timeoutMs) {
|
||||
res.resume();
|
||||
res.on("end", () => {
|
||||
if (res.statusCode && res.statusCode >= 200 && res.statusCode < 300) {
|
||||
resolve({ status: "present", method, statusCode: res.statusCode });
|
||||
const digest = String(res.headers["docker-content-digest"] ?? "").trim();
|
||||
resolve({
|
||||
status: "present",
|
||||
method,
|
||||
statusCode: res.statusCode,
|
||||
digest: /^sha256:[a-f0-9]{64}$/u.test(digest) ? digest : null
|
||||
});
|
||||
} else if (res.statusCode === 404) {
|
||||
resolve({ status: "missing", method, statusCode: res.statusCode });
|
||||
} else {
|
||||
|
||||
Reference in New Issue
Block a user