Skip to content

Conversation

@2uasimojo
Copy link
Member

@2uasimojo 2uasimojo commented Dec 10, 2025

This was a copy/paste error from HIVE-2302 / #2729.

@openshift-ci-robot openshift-ci-robot added the jira/valid-reference Indicates that this PR references a valid Jira ticket of any type. label Dec 10, 2025
@openshift-ci-robot
Copy link

openshift-ci-robot commented Dec 10, 2025

@2uasimojo: This pull request references HIVE-3023 which is a valid jira issue.

In response to this:

This was a copy/paste error from HIVE-2302 / #2729.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the openshift-eng/jira-lifecycle-plugin repository.

1 similar comment
@openshift-ci-robot
Copy link

openshift-ci-robot commented Dec 10, 2025

@2uasimojo: This pull request references HIVE-3023 which is a valid jira issue.

In response to this:

This was a copy/paste error from HIVE-2302 / #2729.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the openshift-eng/jira-lifecycle-plugin repository.

@openshift-ci openshift-ci bot requested review from dlom and jstuever December 10, 2025 16:50
@openshift-ci openshift-ci bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Dec 10, 2025
@2uasimojo
Copy link
Member Author

/assign @huangmingxia
/cc @jianping-shu this may help you validate #2805 :)

@huangmingxia
Copy link
Contributor

/lgtm

Local test passed.

 % cat pkg/clusterresource/builder.go | grep metadata-json
	return fmt.Sprintf("%s-adopted-metadata-json", o.Name)

 %  ./bin/hiveutil create-cluster mihuangadpot --cloud=aws --adopt --adopt-admin-kubeconfig=./kubeconfig3  --adopt-metadata-json=./metadata.json --adopt-cluster-id=xxx --adopt-infra-id=xxx -o yaml > 3025.yaml

secret has been correctly generated.
 % cat 3025.yaml
...
- apiVersion: v1
  data:
    metadata.json: xxx
  kind: Secret
  metadata:
    creationTimestamp: null
    name: mihuangadpot-adopted-metadata-json
...

 % echo "xxx" | base64 -d | jq .
{
  "clusterName": "xxx",
  "clusterID": "xxx",
  "infraID": "xxx",
  "aws": {
    "region": "ap-northeast-1",
    "identifier": [
      {
        "kubernetes.io/cluster/xxx": "owned"
      },
      {
        "openshiftClusterID": "xxx"
      },
      {
        "sigs.k8s.io/cluster-api-provider-aws/cluster/xxx": "owned"
      }
    ],
    "clusterDomain": "cluster.base.domain.com"
  },
  "featureSet": "",
  "customFeatureSet": null
}

@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label Dec 10, 2025
@huangmingxia
Copy link
Contributor

huangmingxia commented Dec 10, 2025

/retitle HIVE-3025: fix adopted metadata.json secret name

@openshift-ci openshift-ci bot changed the title HIVE-3023: fix adopted metadata.json secret name HIVE-3025: fix adopted metadata.json secret name Dec 10, 2025
@openshift-ci-robot
Copy link

openshift-ci-robot commented Dec 10, 2025

@2uasimojo: This pull request references HIVE-3025 which is a valid jira issue.

In response to this:

This was a copy/paste error from HIVE-2302 / #2729.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the openshift-eng/jira-lifecycle-plugin repository.

@codecov
Copy link

codecov bot commented Dec 10, 2025

Codecov Report

❌ Patch coverage is 0% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 50.39%. Comparing base (e597cf4) to head (0ac3a5f).
⚠️ Report is 2 commits behind head on master.

Files with missing lines Patch % Lines
pkg/clusterresource/builder.go 0.00% 1 Missing ⚠️
Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #2812   +/-   ##
=======================================
  Coverage   50.39%   50.39%           
=======================================
  Files         279      279           
  Lines       34190    34190           
=======================================
  Hits        17231    17231           
  Misses      15595    15595           
  Partials     1364     1364           
Files with missing lines Coverage Δ
pkg/clusterresource/builder.go 63.02% <0.00%> (ø)
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@2uasimojo
Copy link
Member Author

/retest hive-mce-210-on-pull-request

@openshift openshift deleted a comment from openshift-ci bot Dec 10, 2025
This was a copy/paste error from HIVE-2302 / openshift#2729.
@2uasimojo
Copy link
Member Author

The bare "Red Hat Konflux" job is bogus. I would override it, but I need to re-push anyway cause I made a typo in the commit message. Since I'm not changing content, I'll force-merge at that point.

@2uasimojo 2uasimojo force-pushed the HIVE-3025/adopt-metadata branch from 438e9eb to 0ac3a5f Compare December 10, 2025 20:16
@openshift-ci openshift-ci bot removed the lgtm Indicates that a PR is ready to be merged. label Dec 10, 2025
@huangmingxia
Copy link
Contributor

/lgtm

@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label Dec 10, 2025
@openshift-ci
Copy link
Contributor

openshift-ci bot commented Dec 10, 2025

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: 2uasimojo, huangmingxia

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Dec 10, 2025

@2uasimojo: all tests passed!

Full PR test history. Your PR dashboard.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.

@openshift-merge-bot openshift-merge-bot bot merged commit b4f8e48 into openshift:master Dec 10, 2025
22 checks passed
@2uasimojo 2uasimojo deleted the HIVE-3025/adopt-metadata branch December 10, 2025 22:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. jira/valid-reference Indicates that this PR references a valid Jira ticket of any type. lgtm Indicates that a PR is ready to be merged.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants