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

chore(build): refactor buildx script and add ci builds #449

Merged
merged 6 commits into from
Jul 7, 2020

Conversation

akhilerm
Copy link
Contributor

@akhilerm akhilerm commented Jul 6, 2020

Why is this PR required? What issue does it fix?:
Currently, multiarch images are built only after merging the changes to master. This PR adds a workflow to build the multiarch image for pull requests.

What this PR does?:

  • Adds option to specify operation for mulitarch image result (push, load, cache)
    1. Push: the mulitarch image will be pushed to the image registry
    2. Load: the image will be loaded to local docker (image will be built only for the platform of build machine)
    3. Cache: Image will be available only in the build cache
  • refactor platform names

Does this PR require any upgrade changes?:
No

If the changes in this PR are manually verified, list down the scenarios covered::
Yes, built and loaded single platform image using buildx

Any additional information for your reviewer? :
Mention if this PR is part of any design or a continuation of previous PRs
Continuation of #428

Checklist:

  • Fixes #
  • PR Title follows the convention of <type>(<scope>): <subject>
  • Has the change log section been updated?
  • Commit has unit tests
  • Commit has integration tests
  • (Optional) Are upgrade changes included in this PR? If not, mention the issue/PR to track:
  • (Optional) If documentation changes are required, which issue on https://github.com/openebs/openebs-docs is used to track them:

@akhilerm
Copy link
Contributor Author

akhilerm commented Jul 6, 2020

@xunholy PTAL. This PR is for enabling mulitarch builds on pull requests.

@akhilerm akhilerm requested a review from kmova July 6, 2020 08:12
akhilerm added 2 commits July 6, 2020 13:44
refactor buildx scripts to disable push and instead load the image
to local docker

Signed-off-by: Akhil Mohan <[email protected]>
- refactor buildx scripts to specify image result to load, push or keep in cache
- added workflow to build image on pull requests

Signed-off-by: Akhil Mohan <[email protected]>
Copy link
Contributor

@xunholy xunholy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A few small suggestions, overall nice work @akhilerm!

.github/workflows/pull_request.yml Outdated Show resolved Hide resolved
.github/workflows/pull_request.yml Outdated Show resolved Hide resolved
.github/workflows/pull_request.yml Outdated Show resolved Hide resolved
Makefile.buildx.mk Outdated Show resolved Hide resolved
@codecov-commenter
Copy link

codecov-commenter commented Jul 6, 2020

Codecov Report

Merging #449 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #449   +/-   ##
=======================================
  Coverage   43.49%   43.49%           
=======================================
  Files          61       61           
  Lines        2968     2968           
=======================================
  Hits         1291     1291           
  Misses       1575     1575           
  Partials      102      102           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update da3267f...b6e4c32. Read the comment docs.

@akhilerm akhilerm force-pushed the architecture-fix branch from 5de101e to 4258e35 Compare July 6, 2020 09:43
xunholy
xunholy previously approved these changes Jul 6, 2020
Copy link
Contributor

@xunholy xunholy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 🚢 it

@akhilerm akhilerm added this to the 0.7 milestone Jul 6, 2020
@akhilerm
Copy link
Contributor Author

akhilerm commented Jul 7, 2020

@kmova Made the requested changes.

@kmova kmova merged commit c250388 into openebs-archive:master Jul 7, 2020
@akhilerm akhilerm deleted the architecture-fix branch July 13, 2020 14:28
kmova added a commit to kmova/openebs that referenced this pull request Jul 15, 2020
Refer: [GOVERNANCE.md](https://github.com/openebs/openebs/blob/master/GOVERNANCE.md).

The following community members have been helping with enhancing and testing
of several areas of the OpenEBS project that broadly fall in the purview of the
control plane.

Based on their past contributions and their interest/commitment to contributing
to OpenEBS, we take pride in adding them as reviewers to the OpenEBS project.

The list of community members in alphabetical order are:

- Mehran Kholdi for the contributions to the Local PV and control plane in general.
  ```
  - openebs#2975
  - 40+ commits in https://github.com/openebs/rawfile-localpv
  ```

- Michael Fornaro for the contributions to the ARM, Multi-arch builds and helm charts that fall under control plane.
  ```
  - openebs-archive/e2e-tests#405
  - openebs#3038
  - openebs#3023
  - openebs#3037
  - openebs#1295
  - openebs/linux-utils#5
  - openebs-archive/node-disk-manager#446
  - openebs-archive/node-disk-manager#449
  - openebs-archive/node-disk-manager#428
  ```

- Peeyush Gupta for the contributions to the Power builds and control plane in general.
  ```
  - openebs/charts#127
  - openebs-archive/node-disk-manager#448
  - openebs-archive/maya#1632
  - openebs-archive/jiva#279
  - openebs-archive/maya#667
  - openebs-archive/istgt#131
  - openebs-archive/maya#561
  - openebs-archive/maya#750
  ```

Signed-off-by: kmova <[email protected]>
kmova added a commit to openebs/openebs that referenced this pull request Jul 15, 2020
Refer: [GOVERNANCE.md](https://github.com/openebs/openebs/blob/master/GOVERNANCE.md).

The following community members have been helping with enhancing and testing
of several areas of the OpenEBS project that broadly fall in the purview of the
control plane.

Based on their past contributions and their interest/commitment to contributing
to OpenEBS, we take pride in adding them as reviewers to the OpenEBS project.

The list of community members in alphabetical order are:

- Mehran Kholdi for the contributions to the Local PV and control plane in general.
  ```
  - #2975
  - 40+ commits in https://github.com/openebs/rawfile-localpv
  ```

- Michael Fornaro for the contributions to the ARM, Multi-arch builds and helm charts that fall under control plane.
  ```
  - openebs-archive/e2e-tests#405
  - #3038
  - #3023
  - #3037
  - #1295
  - openebs/linux-utils#5
  - openebs-archive/node-disk-manager#446
  - openebs-archive/node-disk-manager#449
  - openebs-archive/node-disk-manager#428
  ```

- Peeyush Gupta for the contributions to the Power builds and control plane in general.
  ```
  - openebs/charts#127
  - openebs-archive/node-disk-manager#448
  - openebs-archive/maya#1632
  - openebs-archive/jiva#279
  - openebs-archive/maya#667
  - openebs-archive/istgt#131
  - openebs-archive/maya#561
  - openebs-archive/maya#750
  ```

Signed-off-by: kmova <[email protected]>
Akshay-Nagle pushed a commit to Akshay-Nagle/openebs that referenced this pull request Oct 7, 2020
Refer: [GOVERNANCE.md](https://github.com/openebs/openebs/blob/master/GOVERNANCE.md).

The following community members have been helping with enhancing and testing
of several areas of the OpenEBS project that broadly fall in the purview of the
control plane.

Based on their past contributions and their interest/commitment to contributing
to OpenEBS, we take pride in adding them as reviewers to the OpenEBS project.

The list of community members in alphabetical order are:

- Mehran Kholdi for the contributions to the Local PV and control plane in general.
  ```
  - openebs#2975
  - 40+ commits in https://github.com/openebs/rawfile-localpv
  ```

- Michael Fornaro for the contributions to the ARM, Multi-arch builds and helm charts that fall under control plane.
  ```
  - openebs-archive/e2e-tests#405
  - openebs#3038
  - openebs#3023
  - openebs#3037
  - openebs#1295
  - openebs/linux-utils#5
  - openebs-archive/node-disk-manager#446
  - openebs-archive/node-disk-manager#449
  - openebs-archive/node-disk-manager#428
  ```

- Peeyush Gupta for the contributions to the Power builds and control plane in general.
  ```
  - openebs/charts#127
  - openebs-archive/node-disk-manager#448
  - openebs-archive/maya#1632
  - openebs-archive/jiva#279
  - openebs-archive/maya#667
  - openebs-archive/istgt#131
  - openebs-archive/maya#561
  - openebs-archive/maya#750
  ```

Signed-off-by: kmova <[email protected]>
Signed-off-by: Kung Fu Panda <[email protected]>
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.

4 participants