Update the layout and content within the root scripts directory#121
Update the layout and content within the root scripts directory#121timflannagan wants to merge 3 commits intoopenshift:masterfrom
Conversation
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: timflannagan The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:
Approvers can indicate their approval by writing |
openshift-ci
Bot
added
the
approved
timflannagan
changed the title
|
cc @kevinrizza - IIRC, we had talked about removing the pull/push remote scripts that were causing confusion during the last team retro and updating the README.md to simply point to the google doc that's currently housing the downstream guide. |
|
/retest |
1 similar comment
|
/retest |
timflannagan
force-pushed
the
update-scripts-directory
branch
from
4a9f2e1 to
a613c2b
Compare
|
/test unit-olm |
Upstream-repository: api Upstream-commit: 41d37db9141f27da70d0066f2acf2dd070f8e092
openshift-ci
Bot
added
the
needs-rebase
Remove any unused scripts: - Remove any of the scripts that were previous attempts at automating the downstream port process. - Remove the nested olm script directory.
timflannagan
force-pushed
the
update-scripts-directory
branch
from
a613c2b to
952911a
Compare
openshift-ci
Bot
removed
the
needs-rebase
|
@timflannagan: The following test failed, say
Full PR test history. Your PR dashboard. DetailsInstructions 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/test-infra repository. I understand the commands that are listed here. |
|
Issues go stale after 90d of inactivity. Mark the issue as fresh by commenting If this issue is safe to close now please do so with /lifecycle stale |
openshift-ci
Bot
added
lifecycle/stale
|
@timflannagan: PR needs rebase. DetailsInstructions 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/test-infra repository. |
2 participants
Update the README.md and remove the outdated downstream port guide command(s). Instead, reference the google doc that outlines the current process as we iterate on a more concrete process that will eventually live downstream. This is mainly a stop gap to avoid the OCP release cadence where we can't update something like the scripts/README.md during periods inside of feature/code freeze without a valid bug.
Remove the unused bash scripts. This mainly includes the helper scripts that attempted to automate the deliverance of upstream commits to the staging repositories as the presence of these scripts were generating confusion on the process. Remove the nested scripts/olm directory as those also appear to be unused.
Update the scripts/*.patch.yaml YQ patch files that are used to update the rendered staging OLM helm charts to make the requisite downstream-specific modifications to their own directory. Those patch files now live in scripts/patches and the manifest generation script has been updated to point to that new location.