Skip to content

HIVE-2550,HIVE-2551: AWS deprov job compliant with cred proc#2325

Merged
openshift-merge-bot[bot] merged 2 commits intoopenshift:masterfrom
celebdor:HIVE-2551
Jun 25, 2024
Merged

HIVE-2550,HIVE-2551: AWS deprov job compliant with cred proc#2325
openshift-merge-bot[bot] merged 2 commits intoopenshift:masterfrom
celebdor:HIVE-2551

Conversation

@celebdor
Copy link
Contributor

In #2322, in order to fix AWS provisioining in STS mode we pointed the credential_process configuration to /output/hiveutil. The problem is that the same configuration is used for both provisioning and deprovisioning. Thus, when trying to deprovision in STS mode, when it came time for delegating operation to hiveutil, we'd get an:

/output/hiveutil not found

This commit fixes it by replicating the /output/hiveutil existance for deprovisioning in AWS (though it will only get used in STS mode).

In openshift#2322, in order to fix AWS provisioining in STS mode we pointed the
credential_process configuration to /output/hiveutil. The problem is
that the same configuration is used for both provisioning and
deprovisioning. Thus, when trying to deprovision in STS mode, when it
came time for delegating operation to hiveutil, we'd get an:

    /output/hiveutil not found

This commit fixes it by replicating the /output/hiveutil existance for
deprovisioning in AWS (though it will only get used in STS mode).

Signed-off-by: Antoni Segura Puimedon <antoni@redhat.com>
@openshift-ci-robot openshift-ci-robot added the jira/valid-reference Indicates that this PR references a valid Jira ticket of any type. label Jun 24, 2024
@openshift-ci-robot
Copy link

openshift-ci-robot commented Jun 24, 2024

@celebdor: This pull request references HIVE-2551 which is a valid jira issue.

Details

In response to this:

In #2322, in order to fix AWS provisioining in STS mode we pointed the credential_process configuration to /output/hiveutil. The problem is that the same configuration is used for both provisioning and deprovisioning. Thus, when trying to deprovision in STS mode, when it came time for delegating operation to hiveutil, we'd get an:

/output/hiveutil not found

This commit fixes it by replicating the /output/hiveutil existance for deprovisioning in AWS (though it will only get used in STS mode).

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 jstuever and suhanime June 24, 2024 16:05
@codecov
Copy link

codecov bot commented Jun 24, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 58.57%. Comparing base (73c38b4) to head (03ed534).

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #2325      +/-   ##
==========================================
+ Coverage   58.54%   58.57%   +0.03%     
==========================================
  Files         182      182              
  Lines       25843    25863      +20     
==========================================
+ Hits        15130    15150      +20     
  Misses       9437     9437              
  Partials     1276     1276              
Files Coverage Δ
pkg/install/generate.go 49.47% <100.00%> (+1.84%) ⬆️

There's a few conditions that make `ln` fail to create a hard link which
may be reproduced in a special volume like an emptyDir. The requirements
for a symlink are way lower.

Signed-off-by: Antoni Segura Puimedon <antoni@redhat.com>
@celebdor celebdor changed the title HIVE-2551: AWS deprov job compliant with cred proc HIVE-2550,HIVE-2551: AWS deprov job compliant with cred proc Jun 24, 2024
@openshift-ci-robot
Copy link

openshift-ci-robot commented Jun 24, 2024

@celebdor: An error was encountered searching for bug HIVE-2551 on the Jira server at https://issues.redhat.com/. No known errors were detected, please see the full error message for details.

Full error message. You do not have the permission to see the specified issue.: request failed. Please analyze the request body for more details. Status code: 401:

Please contact an administrator to resolve this issue, then request a bug refresh with /jira refresh.

Details

In response to this:

In #2322, in order to fix AWS provisioining in STS mode we pointed the credential_process configuration to /output/hiveutil. The problem is that the same configuration is used for both provisioning and deprovisioning. Thus, when trying to deprovision in STS mode, when it came time for delegating operation to hiveutil, we'd get an:

/output/hiveutil not found

This commit fixes it by replicating the /output/hiveutil existance for deprovisioning in AWS (though it will only get used in STS mode).

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-robot
Copy link

openshift-ci-robot commented Jun 24, 2024

@celebdor: This pull request references HIVE-2550 which is a valid jira issue.

Details

In response to this:

In #2322, in order to fix AWS provisioining in STS mode we pointed the credential_process configuration to /output/hiveutil. The problem is that the same configuration is used for both provisioning and deprovisioning. Thus, when trying to deprovision in STS mode, when it came time for delegating operation to hiveutil, we'd get an:

/output/hiveutil not found

This commit fixes it by replicating the /output/hiveutil existance for deprovisioning in AWS (though it will only get used in STS mode).

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.

@lleshchi
Copy link
Contributor

/lgtm

@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label Jun 25, 2024
@openshift-ci
Copy link
Contributor

openshift-ci bot commented Jun 25, 2024

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: celebdor, lleshchi

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

The pull request process is described here

Details 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 openshift-ci bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Jun 25, 2024
@openshift-ci-robot
Copy link

/retest-required

Remaining retests: 0 against base HEAD 73c38b4 and 2 for PR HEAD 03ed534 in total

@lleshchi
Copy link
Contributor

/override "Red Hat Konflux / hive-on-pull-request"

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Jun 25, 2024

@lleshchi: Overrode contexts on behalf of lleshchi: Red Hat Konflux / hive-on-pull-request

Details

In response to this:

/override "Red Hat Konflux / hive-on-pull-request"

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.

@lleshchi
Copy link
Contributor

/override "ci/prow/periodic-images"

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Jun 25, 2024

@lleshchi: Overrode contexts on behalf of lleshchi: ci/prow/periodic-images

Details

In response to this:

/override "ci/prow/periodic-images"

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.

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Jun 25, 2024

@celebdor: all tests passed!

Full PR test history. Your PR dashboard.

Details

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 e01288d into openshift:master Jun 25, 2024
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