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

Update the action to support astarte v1.2 #8

Conversation

matt-mazzucato
Copy link
Collaborator

At the moment astartectl v24.5 is not capable of deploying astarte v1.2. Therefore, rework the action to ensure that the new astarte version can be tested in our workflows.
In this context, a number of steps that astartectl was taking care of are explicitly handled (setting contexts, passing keys...).
In addition, tool versions are bumped.

The changes introduced in this PR are thought to be transparent from the perspective of the users of this action.

…rte 1.2.

In particular:
- use astartectl v24.5.0
- astarte default version is 1.2.0
- astarte operator default version is v24.5.0
In addition, bump auxiliary tool versions
- use cert-manager v1.14.5
- use kind 0.23.0 and kubernetes 1.30.0

Signed-off-by: Mattia Mazzucato <mattia.mazzucato@secomind.com>
Not a lint-test, rather a deploy test.

Signed-off-by: Mattia Mazzucato <mattia.mazzucato@secomind.com>
@matt-mazzucato matt-mazzucato marked this pull request as draft September 6, 2024 13:22
@matt-mazzucato matt-mazzucato force-pushed the update-action-to-support-astarte-1.2 branch from b155c92 to 6e6aac1 Compare September 9, 2024 13:02
The deployment of Astarte is done with a simple kubectl apply command instead of
relying on the "astartectl cluster instances deploy" command.
After this change, some of the hidden automations leveraged by astartectl are no
longer available. Therefore, adapt the procedures for ensuring a consistent
Astarte instance is deployed (e.g.: explicitly set astartectl clusters and
contexts, explicitly pass keys where needed).
This commit does not introduce user facing breaking changes.

Signed-off-by: Mattia Mazzucato <mattia.mazzucato@secomind.com>
@matt-mazzucato matt-mazzucato force-pushed the update-action-to-support-astarte-1.2 branch from 6e6aac1 to e3cca5d Compare September 9, 2024 13:27
@matt-mazzucato matt-mazzucato force-pushed the update-action-to-support-astarte-1.2 branch 2 times, most recently from 9408010 to e3cca5d Compare September 11, 2024 12:31
@matt-mazzucato matt-mazzucato marked this pull request as ready for review September 13, 2024 09:02
@Annopaolo Annopaolo merged commit f01d2d8 into astarte-platform:master Sep 13, 2024
3 checks passed
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.

2 participants