Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Disable broken smoke #37

Merged
merged 3 commits into from
Jul 28, 2020
Merged

Conversation

cevich
Copy link
Member

@cevich cevich commented Jul 28, 2020

No description provided.

In order to better re-use common Github Action-specific helper script
libraries, move them into a more promintnt location, and add a README.md

Signed-off-by: Chris Evich <cevich@redhat.com>
Some users of the common library are not operating under a github
actions environment.  To support installation under a github actions
environment while retaining that characteristic, new installer behavior
is required to allow subdirectory installers to modify the environment
file before "installation finished" (version file) is put in place.

Add a github action common lib installer which adds it's installation
directory into the system-wide environment file.

Update cirrus-ci_retrospective to utilize the new github action common
lib for some operations.

Update unit-tests.

Fix bug installing system-wide environment in Ubuntu vs Fedora

Signed-off-by: Chris Evich <cevich@redhat.com>
Building container images from the PR code, and testing them is too
onerous of a task to encode directly into the workflow yaml.  Replace
with a TODO for now, and rely on a future commit to implement this step
as a proper github-action or a simple script.

Signed-off-by: Chris Evich <cevich@redhat.com>
@github-actions
Copy link

github-actions bot commented Jul 28, 2020

Failure running Cirrus-CI Retrospective Github Action failed against this PR's 0d6938e60daa6ba28fd4d0e05df8294ab62b1ced

@cevich cevich merged commit af956ca into containers:master Jul 28, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant