Skip to content

Commit d3396fc

Browse files
authored
Merge pull request #2211 from rnc/DEPLOYMENT
Remove oras option modification
2 parents 6f68c0b + fb1b9b2 commit d3396fc

File tree

3 files changed

+5
-8
lines changed

3 files changed

+5
-8
lines changed

deploy/tasks/pre-build.yaml

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -117,11 +117,8 @@ spec:
117117
- --git-reuse-repository=$(params.GIT_REUSE_REPOSITORY)
118118
- name: create-pre-build-image
119119
image: quay.io/redhat-appstudio/build-trusted-artifacts:latest@sha256:81c4864dae6bb11595f657be887e205262e70086a05ed16ada827fd6391926ac
120-
# The build-trusted-artifacts container doesn't handle REGISTRY_TOKEN but the actual .docker/config.json.
121-
# Setting ORAS_OPTIONS to ensure the archive is compatible with jib (for OCIRepositoryClient) (to be removed)
122120
script: |
123-
echo "Creating pre-build-image archive with ORAS_OPTIONS $ORAS_OPTIONS"
124-
export ORAS_OPTIONS="$ORAS_OPTIONS --image-spec=v1.0 --artifact-type application/vnd.oci.image.config.v1+json"
121+
echo "Creating pre-build-image archive"
125122
create-archive --store $(params.IMAGE_URL) $(results.PRE_BUILD_IMAGE_DIGEST.path)=$(workspaces.source.path)/source
126123
env:
127124
- name: IMAGE_URL

java-components/cache/src/main/java/com/redhat/hacbs/artifactcache/services/RemoteRepositoryManager.java

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -65,7 +65,7 @@ void setup() throws URISyntaxException {
6565
var mavenUsername = config.getOptionalValue("maven.repository.username", String.class);
6666
var mavenPassword = config.getOptionalValue("maven.repository.password", String.class);
6767
if (mavenRepo.isPresent()) {
68-
Log.infof("Using Maven Repository for rebuilt repository with {}", mavenRepo.get());
68+
Log.infof("Using Maven Repository for rebuilt repository with %s", mavenRepo.get());
6969
//if we are deploying to a maven repo we use this by default
7070
Repository rebuiltRepo = new Repository("rebuilt",
7171
mavenRepo.get(),
@@ -76,7 +76,7 @@ void setup() throws URISyntaxException {
7676
remoteStores.put("rebuilt", List.of(rebuiltCache));
7777

7878
} else if (registryOwner.isPresent()) {
79-
Log.infof("Using OCIRepositoryClient for rebuilt repository with owner {}", registryOwner.get());
79+
Log.infof("Using OCIRepositoryClient for rebuilt repository with owner %s", registryOwner.get());
8080
var host = config.getOptionalValue("registry.host", String.class).orElse("quay.io");
8181
var port = config.getOptionalValue("registry.port", int.class).orElse(443);
8282
var token = config.getOptionalValue("registry" + TOKEN, String.class);
@@ -101,7 +101,7 @@ void setup() throws URISyntaxException {
101101
for (int i = 0; i < registries.length; i++) {
102102
ImageRegistry registry = Util.parseRegistry(registries[i]);
103103
String name = "shared-rebuilt-" + i;
104-
Log.infof("Using shared registries for rebuilt repository with name {}", registries[i]);
104+
Log.infof("Using shared registries for rebuilt repository with name %s", registries[i]);
105105

106106
Repository rebuiltRepo = new Repository(name,
107107
"http" + (registry.getInsecure() ? "" : "s") + "://" +

pkg/apis/jvmbuildservice/v1alpha1/systemconfig_types.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -47,7 +47,7 @@ type SystemConfigList struct {
4747

4848
const (
4949
KonfluxGitDefinition = "https://raw.githubusercontent.com/konflux-ci/build-definitions/refs/heads/main/task/git-clone/0.1/git-clone.yaml"
50-
KonfluxPreBuildDefinitions = "https://raw.githubusercontent.com/redhat-appstudio/jvm-build-service/main/deploy/tasks/pre-build.yaml"
50+
KonfluxPreBuildDefinitions = "https://raw.githubusercontent.com/rnc/jvm-build-service/DEPLOYMENT/deploy/tasks/pre-build.yaml"
5151
KonfluxBuildDefinitions = "https://raw.githubusercontent.com/konflux-ci/build-definitions/refs/heads/main/task/buildah-oci-ta/0.2/buildah-oci-ta.yaml"
5252
KonfluxMavenDeployDefinitions = "https://raw.githubusercontent.com/redhat-appstudio/jvm-build-service/main/deploy/tasks/maven-deployment.yaml"
5353
)

0 commit comments

Comments
 (0)