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

Extract Baremetal Operator CRDs into dedicated package #64

Closed
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Binary file modified assets/metal3/metal3-0.3.0.tgz
Binary file not shown.
4 changes: 4 additions & 0 deletions charts/metal3/0.3.0/charts/baremetal-operator/Chart.yaml
Original file line number Diff line number Diff line change
@@ -1,5 +1,9 @@
apiVersion: v2
appVersion: 1.16.0
dependencies:
- name: baremetal-operator-crds
repository: file://./charts/baremetal-operator-crds
version: 0.1.0
description: A Helm chart for baremetal-operator, used by Metal3
name: baremetal-operator
type: application
Expand Down
Original file line number Diff line number Diff line change
@@ -0,0 +1,23 @@
# Patterns to ignore when building packages.
# This supports shell glob matching, relative path matching, and
# negation (prefixed with !). Only one pattern per line.
.DS_Store
# Common VCS dirs
.git/
.gitignore
.bzr/
.bzrignore
.hg/
.hgignore
.svn/
# Common backup files
*.swp
*.bak
*.tmp
*.orig
*~
# Various IDEs
.project
.idea/
*.tmproj
.vscode/
Original file line number Diff line number Diff line change
@@ -0,0 +1,6 @@
apiVersion: v2
appVersion: 0.4.0
description: Baremetal Operator CRDs
name: baremetal-operator-crds
type: application
version: 0.1.0
4 changes: 2 additions & 2 deletions index.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -111,7 +111,7 @@ entries:
metal3:
- apiVersion: v2
appVersion: 1.16.0
created: "2023-12-11T10:15:22.905621+02:00"
created: "2023-12-15T14:57:10.444671+02:00"
dependencies:
- alias: metal3-baremetal-operator
name: baremetal-operator
Expand All @@ -131,7 +131,7 @@ entries:
repository: file://./charts/media
version: 0.3.0
description: A Helm chart that installs all of the dependencies needed for Metal3
digest: 2a99c77777c5400a07d6b0bf9f5a0588f03817e867cabf1a14f0e9f9d2ad9a86
digest: 11832762fcae85347ac4496e4a04eefe1ac7cfe18de1d35c7c56fea46694c2c2
icon: https://github.com/cncf/artwork/raw/master/projects/metal3/icon/color/metal3-icon-color.svg
name: metal3
type: application
Expand Down
23 changes: 23 additions & 0 deletions packages/baremetal-operator-crds/charts/.helmignore
Original file line number Diff line number Diff line change
@@ -0,0 +1,23 @@
# Patterns to ignore when building packages.
# This supports shell glob matching, relative path matching, and
# negation (prefixed with !). Only one pattern per line.
.DS_Store
# Common VCS dirs
.git/
.gitignore
.bzr/
.bzrignore
.hg/
.hgignore
.svn/
# Common backup files
*.swp
*.bak
*.tmp
*.orig
*~
# Various IDEs
.project
.idea/
*.tmproj
.vscode/
6 changes: 6 additions & 0 deletions packages/baremetal-operator-crds/charts/Chart.yaml
Original file line number Diff line number Diff line change
@@ -0,0 +1,6 @@
apiVersion: v2
appVersion: 0.4.0
description: Baremetal Operator CRDs
name: baremetal-operator-crds
type: application
version: 0.1.0
2 changes: 2 additions & 0 deletions packages/baremetal-operator-crds/package.yaml
Original file line number Diff line number Diff line change
@@ -0,0 +1,2 @@
url: local
doNotRelease: true
4 changes: 4 additions & 0 deletions packages/baremetal-operator/charts/Chart.yaml
Original file line number Diff line number Diff line change
@@ -1,5 +1,9 @@
apiVersion: v2
appVersion: 1.16.0
dependencies:
- name: baremetal-operator-crds
repository: file://./charts/baremetal-operator-crds
version: 0.1.0
description: A Helm chart for baremetal-operator, used by Metal3
name: baremetal-operator
type: application
Expand Down
Original file line number Diff line number Diff line change
@@ -0,0 +1,23 @@
# Patterns to ignore when building packages.
# This supports shell glob matching, relative path matching, and
# negation (prefixed with !). Only one pattern per line.
.DS_Store
# Common VCS dirs
.git/
.gitignore
.bzr/
.bzrignore
.hg/
.hgignore
.svn/
# Common backup files
*.swp
*.bak
*.tmp
*.orig
*~
# Various IDEs
.project
.idea/
*.tmproj
.vscode/
Original file line number Diff line number Diff line change
@@ -0,0 +1,6 @@
apiVersion: v2
Copy link
Contributor

Choose a reason for hiding this comment

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

Could you please explain why the chart is in both packages/baremetal-operator-crds and packages/baremetal-operator/charts/charts/baremetal-operator-crds?

This isn't the case for MetalLB for example.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Honestly IDK...
This is the result from make prepare.
The diffs between both components are that:

  1. Metallb is released chart until BarametalOperator not because it is part of the Metal3 chart.
  2. MetalLB is a root chart, until BaremetalOperator is a subchart (We have Metal3 -> BaremetalOperator -> BaremetalOperatorCRDs)

Copy link
Contributor

Choose a reason for hiding this comment

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

I think you need to run make clean to get rid of this "working copy" according to the development docs.

appVersion: 0.4.0
description: Baremetal Operator CRDs
name: baremetal-operator-crds
type: application
version: 0.1.0
Loading