Skip to content
This repository has been archived by the owner on Mar 4, 2024. It is now read-only.

EVEREST-488 Merged tags into main branch #162

Closed
wants to merge 7 commits into from
Closed

Conversation

gen1us2k
Copy link
Contributor

@gen1us2k gen1us2k commented Oct 4, 2023

CHANGE DESCRIPTION

Problem:
EVEREST-488

Short explanation of the problem.
The output of the

git describe --always --tags
v0.3.0-22-g7a403cf

was empty because we do not merge tags into the main branch
Related pull requests

  • [link]

Cause:
Short explanation of the root cause of the issue if applicable.

Solution:
Short explanation of the solution we are providing with this PR.

CHECKLIST

Jira

  • Is the Jira ticket created and referenced properly?

Tests

  • Is an Integration test/test case added for the new feature/change?
  • Are unit tests added where appropriate?

gen1us2k and others added 7 commits September 21, 2023 20:27
…rs (#151)

* EVEREST-203 Do not run registration for already registered k8s clusters

* EVEREST-203 Generated files

* EVEREST-203 Added integration test

* EVEREST-203 Fixed test

* EVEREST-203 fixed test

* EVEREST-203 Better deletion handling

* EVEREST-203 Shut up linter

* Update pkg/install/operators.go

Co-authored-by: Michal Kralik <[email protected]>

---------

Co-authored-by: Michal Kralik <[email protected]>
@gen1us2k gen1us2k marked this pull request as ready for review October 4, 2023 13:25
@gen1us2k gen1us2k requested a review from a user October 4, 2023 13:25
@@ -9,7 +9,7 @@ spec:
sourceType: grpc
grpcPodConfig:
securityContextConfig: restricted
image: docker.io/percona/everest-catalog:latest
image: docker.io/percona/everest-catalog:0.3.0
Copy link
Collaborator

Choose a reason for hiding this comment

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

Shouldn't the main branch use the latest catalog tag? Why are we changing it to 0.3.0?

Copy link
Contributor

Choose a reason for hiding this comment

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

I agree we should keep the latest tag in main

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I'm still playing around with tags.

@gen1us2k gen1us2k closed this Oct 5, 2023
@gen1us2k gen1us2k deleted the EVEREST-107 branch October 5, 2023 10:46
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants