Allow adding addons to the clusters

[maciaszczykm]

What this PR does / why we need it: This adds initial addons functionality.

Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged): Fixes #1469. Follow up in the #1690.

Special notes for your reviewer:

Release note:

Allowed adding addons to the clusters.

[kubermatic-bot]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: maciaszczykm

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

• OWNERS [maciaszczykm]

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

[maciaszczykm]

I have created the pull request quicker so we can discuss a few things. First of all, as I have stated in #1469 at the moment we have no images for addons and we will no be able to fully reflect the designs:

@cschieder Do you have any idea how we can style the addon names for now?

[codecov]

Codecov Report

Merging #1683 into master will decrease coverage by 0.15%.
The diff coverage is 60.57%.

@@            Coverage Diff             @@
##           master    #1683      +/-   ##
==========================================
- Coverage   71.51%   71.35%   -0.16%     
==========================================
  Files         222      224       +2     
  Lines        7164     7266     +102     
  Branches      712      722      +10     
==========================================
+ Hits         5123     5185      +62     
- Misses       1764     1801      +37     
- Partials      277      280       +3

Impacted Files	Coverage Δ
...nfirmation-dialog/confirmation-dialog.component.ts	73.68% <ø> (ø)	⬆️
...hared/utils/health-status/cluster-health-status.ts	90.47% <100%> (ø)	⬆️
src/app/testing/services/cluster-mock-service.ts	84.48% <100%> (+0.55%)	⬆️
src/app/shared/shared.module.ts	100% <100%> (ø)	⬆️
src/app/testing/services/api-mock.service.ts	73.78% <100%> (+0.51%)	⬆️
src/app/core/services/api/api.service.ts	37.06% <33.33%> (-0.08%)	⬇️
src/app/core/services/cluster/cluster.service.ts	36.73% <33.33%> (-0.35%)	⬇️
...ist/add-addon-dialog/add-addon-dialog.component.ts	57.14% <57.14%> (ø)
...ared/components/addon-list/addon-list.component.ts	60.34% <60.34%> (ø)
...uster/cluster-details/cluster-details.component.ts	68.66% <63.63%> (-0.62%)	⬇️
... and 4 more

---

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 783e236...f3ea86d. Read the comment docs.

[cschieder]

@maciaszczykm honestly I would leave as is except using title case/whatever capitalisation the logo uses, if possible. Do we have any estimate on when we want to add the images?

[maciaszczykm]

@cschieder Unfortunately all the addons are retrieved from the API, so we cannot change the names as we don't know the original ones.

I'd like to add the images sooner rather than later but there are a few things related to addons that are more important at the moment. The first of them is how we can implement the Variables edition (this is basically an object without any structure and normal form will not help us here).

[maciaszczykm]

It's ready for a review. #1690 is a follow-up issue for this.

@kgroschoff PTAL

[kgroschoff]

Nice! 👍
/lgtm

[kubermatic-bot]

LGTM label has been added.

Git tree hash: 721e575adbf2f5a8738eb0432bab845e963ababd

[maciaszczykm]

/retest

[kubermatic-triage-bot]

/retest
This bot automatically retries jobs that failed/flaked on approved PRs

Review the full test history

Silence the bot with an /lgtm cancel or /hold comment for consistent failures.

Also, here is a cat.
/meow

[kubermatic-bot]

@kubermatic-triage-bot:

In response to this:

/retest
This bot automatically retries jobs that failed/flaked on approved PRs

Review the full test history

Silence the bot with an /lgtm cancel or /hold comment for consistent failures.

Also, here is a cat.
/meow

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

[kubermatic-triage-bot]

/retest
This bot automatically retries jobs that failed/flaked on approved PRs

Review the full test history

Silence the bot with an /lgtm cancel or /hold comment for consistent failures.

Also, here is a cat.
/meow

[kubermatic-bot]

@kubermatic-triage-bot:

In response to this:

/retest
This bot automatically retries jobs that failed/flaked on approved PRs

Review the full test history

Silence the bot with an /lgtm cancel or /hold comment for consistent failures.

Also, here is a cat.
/meow

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

[kubermatic-bot]

New changes are detected. LGTM label has been removed.

[kubermatic-triage-bot]

/retest
This bot automatically retries jobs that failed/flaked on approved PRs

Review the full test history

Silence the bot with an /lgtm cancel or /hold comment for consistent failures.

Also, here is a cat.
/meow

[kubermatic-bot]

@kubermatic-triage-bot:

In response to this:

/retest
This bot automatically retries jobs that failed/flaked on approved PRs

Review the full test history

Silence the bot with an /lgtm cancel or /hold comment for consistent failures.

Also, here is a cat.
/meow

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

[kubermatic-triage-bot]

/retest
This bot automatically retries jobs that failed/flaked on approved PRs

Review the full test history

Silence the bot with an /lgtm cancel or /hold comment for consistent failures.

Also, here is a cat.
/meow

[kubermatic-bot]

@kubermatic-triage-bot:

In response to this:

/retest
This bot automatically retries jobs that failed/flaked on approved PRs

Review the full test history

Silence the bot with an /lgtm cancel or /hold comment for consistent failures.

Also, here is a cat.
/meow

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

[kubermatic-triage-bot]

/retest
This bot automatically retries jobs that failed/flaked on approved PRs

Review the full test history

Silence the bot with an /lgtm cancel or /hold comment for consistent failures.

Also, here is a cat.
/meow

[kubermatic-bot]

@kubermatic-triage-bot:

In response to this:

/retest
This bot automatically retries jobs that failed/flaked on approved PRs

Review the full test history

Silence the bot with an /lgtm cancel or /hold comment for consistent failures.

Also, here is a cat.
/meow

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

[kubermatic-triage-bot]

/retest
This bot automatically retries jobs that failed/flaked on approved PRs

Review the full test history

Silence the bot with an /lgtm cancel or /hold comment for consistent failures.

Also, here is a cat.
/meow

[kubermatic-bot]

@kubermatic-triage-bot:

In response to this:

/retest
This bot automatically retries jobs that failed/flaked on approved PRs

Review the full test history

Silence the bot with an /lgtm cancel or /hold comment for consistent failures.

Also, here is a cat.
/meow

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.