Skip to content

cleanup docker compose based branch deployment#649

Open
boddumanohar wants to merge 1 commit intodevfrom
cleanup-compose-based-branch
Open

cleanup docker compose based branch deployment#649
boddumanohar wants to merge 1 commit intodevfrom
cleanup-compose-based-branch

Conversation

@boddumanohar
Copy link
Member

@boddumanohar boddumanohar commented Mar 2, 2026

We also don't support them as such VMs start is quite slow.

@boddumanohar boddumanohar force-pushed the cleanup-compose-based-branch branch from f76f4cd to bf485d5 Compare March 2, 2026 11:56
@boddumanohar boddumanohar marked this pull request as ready for review March 2, 2026 11:57
@boddumanohar boddumanohar changed the base branch from main to dev March 2, 2026 11:58
@boddumanohar boddumanohar requested review from mxsrc and noctarius and removed request for mxsrc March 2, 2026 11:58
@boddumanohar boddumanohar marked this pull request as draft March 2, 2026 12:02
@boddumanohar
Copy link
Member Author

just found there are 10 branches with image tag as 15.1.0.147'

# select count(*) from branch where database_image_tag='15.1.0.147';
 count 
-------
    10
(1 row)

Base automatically changed from dev to main March 4, 2026 12:01
@boddumanohar boddumanohar changed the base branch from main to dev March 6, 2026 17:16
@boddumanohar boddumanohar force-pushed the cleanup-compose-based-branch branch from bf485d5 to 6375b5c Compare March 6, 2026 17:17
@boddumanohar boddumanohar marked this pull request as ready for review March 6, 2026 17:17
@boddumanohar
Copy link
Member Author

I think its safe to having this. Because we have currently already blocked the new VMs from creating with 15.1.0.147 tag.

Changes that are touched as a part of this PR only affects the branch creation logic. Once the branches are already created, the VM lifecycle flows (Stop, Start, resize) never touch this parts. That's why even for existing VM this wouldn't break anything.

Copy link
Collaborator

@mxsrc mxsrc left a comment

Choose a reason for hiding this comment

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

Great effort!

- name: config
mountPath: "/vela/config"
configMap:
name: {{ include "vela.autoscaler.composeConfigName" . }}
Copy link
Collaborator

Choose a reason for hiding this comment

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

Is there a way to rename this as well?

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