Skip to content

templates: Move more files into common/#783

Merged
openshift-merge-robot merged 1 commit intoopenshift:masterfrom
cgwalters:more-template-common
May 21, 2019
Merged

templates: Move more files into common/#783
openshift-merge-robot merged 1 commit intoopenshift:masterfrom
cgwalters:more-template-common

Conversation

@cgwalters
Copy link
Copy Markdown
Member

Follow up to #627
which only moved the pull secret to avoid conflicting with other PRs,
but since that went well let's do the rest of the _base/files dir.

Via this script:

set -euo pipefail
master=templates/master/00-master
worker=templates/worker/00-worker
(cd ${master} && find _base/files -type f) | while read f; do
  if [ -f "${worker}/${f}" ] && cmp -s ${master}/${f} ${worker}/${f}; then
    git mv ${master}/$f templates/common
    git rm ${worker}/$f
  fi
done

@openshift-ci-robot openshift-ci-robot added approved Indicates a PR has been approved by an approver from all required OWNERS files. size/S Denotes a PR that changes 10-29 lines, ignoring generated files. labels May 20, 2019
Follow up to openshift#627
which only moved the pull secret to avoid conflicting with other PRs,
but since that went well let's do the rest of the `_base/files` dir.

Via this script:
```
set -euo pipefail
master=templates/master/00-master
worker=templates/worker/00-worker
(cd ${master} && find _base/files -type f) | while read f; do
  if [ -f "${worker}/${f}" ] && cmp -s ${master}/${f} ${worker}/${f}; then
    git mv ${master}/$f templates/common/_base/files
    git rm ${worker}/$f
  fi
done
```
@cgwalters cgwalters force-pushed the more-template-common branch from c74f11a to faa5a9c Compare May 21, 2019 01:10
@kikisdeliveryservice
Copy link
Copy Markdown
Contributor

/lgtm

@openshift-ci-robot openshift-ci-robot added the lgtm Indicates that a PR is ready to be merged. label May 21, 2019
@openshift-ci-robot
Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: cgwalters, kikisdeliveryservice

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:
  • OWNERS [cgwalters,kikisdeliveryservice]

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

@openshift-merge-robot openshift-merge-robot merged commit 96f0d4a into openshift:master May 21, 2019
@runcom
Copy link
Copy Markdown
Member

runcom commented May 21, 2019

/appove
/lgtm

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. lgtm Indicates that a PR is ready to be merged. size/S Denotes a PR that changes 10-29 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants