Skip to content

Remove the microshift-containerized subpackage#595

Merged
cooktheryan merged 1 commit intoopenshift:mainfrom
mangelajo:fix-microshift-service
Feb 2, 2022
Merged

Remove the microshift-containerized subpackage#595
cooktheryan merged 1 commit intoopenshift:mainfrom
mangelajo:fix-microshift-service

Conversation

@mangelajo
Copy link
Copy Markdown
Contributor

Our docs explain how to download the .service file [1] instead,
and it has proven problematic to package this.

Fix the microshift.service being overwritten by microshift-containerized,
even when the non-containerized package only is installed.

Resolves-Issue: #591

Signed-off-by: Miguel Angel Ajo majopela@redhat.com

@openshift-ci openshift-ci Bot requested review from copejon and oglok February 2, 2022 10:37
Our docs explain how to download the .service file [1] instead,
and it has proven problematic to package this.

Fix the microshift.service being overwritten by microshift-containerized,
even when the non-containerized package only is installed.

Resolves-Issue: openshift#591

Signed-off-by: Miguel Angel Ajo <majopela@redhat.com>
@mangelajo mangelajo force-pushed the fix-microshift-service branch from dd87e5e to 22f61dc Compare February 2, 2022 10:40
@mangelajo
Copy link
Copy Markdown
Contributor Author

@fzdarsky it compiled properly now:

+ /usr/bin/rm -rf /home/majopela/GolandProjects/microshift/packaging/rpm/_rpmbuild/BUILDROOT/microshift-4.8.0-2022_01_06_210147_20_g22f61dca.fc35.x86_64
+ RPM_EC=0
++ jobs -p
+ exit 0

@cooktheryan
Copy link
Copy Markdown
Contributor

/LGTM

@openshift-ci
Copy link
Copy Markdown
Contributor

openshift-ci Bot commented Feb 2, 2022

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: cooktheryan

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 lgtm Indicates that a PR is ready to be merged. approved Indicates a PR has been approved by an approver from all required OWNERS files. labels Feb 2, 2022
@cooktheryan cooktheryan merged commit c2bdffb into openshift:main Feb 2, 2022
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.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants