Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(gitlab): support GitLab CI/CD component references #26660

Merged
merged 24 commits into from Jan 23, 2024
Merged
Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
24 commits
Select commit Hold shift + click to select a range
7458246
feat(gitlab): support GitLab CI/CD component references
fgreinacher Jan 16, 2024
3384d43
Merge branch 'main' into feat/cicd-components
fgreinacher Jan 16, 2024
f2e6482
refactor: reject incomplete project paths
fgreinacher Jan 16, 2024
f784b6c
refactor: ignore external component references
fgreinacher Jan 16, 2024
36fbbc6
refactor: rename variable
fgreinacher Jan 16, 2024
d5107b0
Merge branch 'main' into feat/cicd-components
fgreinacher Jan 16, 2024
144978f
refactor: remove unneeded optional accessor
fgreinacher Jan 17, 2024
12a5a4f
refactor: replace snapshot by inline object
fgreinacher Jan 17, 2024
8e79c98
refactor: remove unneeded array handling
fgreinacher Jan 17, 2024
457c124
refactor: simplify regex matching
fgreinacher Jan 17, 2024
acb0b13
Merge branch 'main' into feat/cicd-components
fgreinacher Jan 17, 2024
80b0211
Merge branch 'main' into feat/cicd-components
fgreinacher Jan 17, 2024
c3a6318
refactor: set registry URL for external component reference
fgreinacher Jan 17, 2024
ef932dd
refactor: move new logic to gitlabci manager
fgreinacher Jan 17, 2024
14b9e30
test: adapt fixtures
fgreinacher Jan 17, 2024
08b3044
Merge remote-tracking branch 'upstream/main' into feat/cicd-components
fgreinacher Jan 19, 2024
5d82bc1
test: add missing test case
fgreinacher Jan 19, 2024
c11ba71
refactor: delete unneeded fixture
fgreinacher Jan 19, 2024
e61f617
fix: update supported datasources
fgreinacher Jan 19, 2024
dac0433
docs: remove wrong comment
fgreinacher Jan 19, 2024
2e37a6e
refactor: inline variable
fgreinacher Jan 20, 2024
3cbce5a
refactor: always set registryUrls
fgreinacher Jan 20, 2024
ec13858
Update extract.ts
fgreinacher Jan 20, 2024
91a4d20
Merge branch 'main' into feat/cicd-components
fgreinacher Jan 23, 2024
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
2 changes: 1 addition & 1 deletion lib/modules/manager/gitlabci-include/extract.spec.ts
Expand Up @@ -110,7 +110,7 @@ describe('modules/manager/gitlabci-include/extract', () => {
datasource: 'gitlab-tags',
depName: 'an-org/a-project',
depType: 'repository',
skipReason: 'invalid-value',
registryUrls: ['https://other-gitlab.example.com'],
},
]);
});
Expand Down
5 changes: 3 additions & 2 deletions lib/modules/manager/gitlabci-include/extract.ts
Expand Up @@ -80,11 +80,12 @@ function extractDepFromIncludeComponent(
}
const endpointUrl = parseUrl(endpoint);
if (endpointUrl && endpointUrl.hostname !== componentReference.fqdn) {
const registryUrl = `https://${componentReference.fqdn}`;
fgreinacher marked this conversation as resolved.
Show resolved Hide resolved
logger.debug(
{ componentReference: includeComponent.component },
'Ignoring external component reference',
'Setting registry URL for external component reference',
);
dep.skipReason = 'invalid-value';
dep.registryUrls = [registryUrl];
}
return dep;
}
Expand Down