Skip to content

Poll backup status from the repo-host rather than the instance pod. #10

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 31 commits into
base: flyio-2.6.0
Choose a base branch
from
Open

Poll backup status from the repo-host rather than the instance pod. #10

wants to merge 31 commits into from

Conversation

@gargakshit
Copy link
Member

This should prevent unnecessary memory pressure on the instance pods.

gargakshit and others added 21 commits May 28, 2025 11:48
@gargakshit
Poll backup status from the repo-host rather than the instance pod.
2173ea8 
This should prevent unnecessary memory pressure on the instance pods.

Signed-off-by: Akshit Garg <[email protected]>
@gargakshit
Build on all branches
0299116 
Signed-off-by: Akshit Garg <[email protected]>
@jphenow
more hints
5f44ed0
@jphenow
tired of this misspelling error
b4a5604
@jphenow
fix linting
b99aab0
@jphenow
some possible test fixes
258ef88
@jphenow
bump jwt for vuln
ac250f3
@jphenow
fix manifests check
02982ed
@jphenow
fixes manifests... again
39d0516
@jphenow
see if claude can fix it
e373794
@jphenow
claude is bad
d915202
@jphenow
fixes generated files that get kustomized
6d4f257
@jphenow
see if this fixes our segf
23df737
@jphenow
fix pgbackrest dance
b45b7f0
@jphenow
fix reconciler test
a64d31b
@jphenow
committing an abusive fix from claude so I can revert some
4c8e46b
@jphenow
fix another more tests and maybe some pgbackrest handling
4bcf159
@jphenow
local testing simpler
7cd1e4f
@jphenow
fix manifests again? wtf?
ce1163a
@jphenow
maybe fix this last set of failures
65ffe4e
@gargakshit
Merge remote-tracking branch 'origin/flyio-2.6.0-sidecars-upstreamabl…
773416c 
…e' into akshit/flyio-2.6.1/poll-repo-host-instead-of-instance
@gargakshit gargakshit changed the base branch from flyio-2.6.0 to flyio-2.6.0-sidecars-upstreamable May 29, 2025 09:06
gargakshit added 7 commits May 29, 2025 16:01
@gargakshit
Exec repo commands in backrest container
4f8d025 
Signed-off-by: Akshit Garg <[email protected]>
@gargakshit
CI hates typos
52c28d4 
Signed-off-by: Akshit Garg <[email protected]>
@gargakshit
Log execs
c2ef00d 
Signed-off-by: Akshit Garg <[email protected]>
@gargakshit
Also log the command
e65c525 
Signed-off-by: Akshit Garg <[email protected]>
@gargakshit
Move one more to repo host
0c81287 
Signed-off-by: Akshit Garg <[email protected]>
@gargakshit
Replace another instance with repo-host calls
56b8393 
Signed-off-by: Akshit Garg <[email protected]>
@gargakshit
Thanks supermaven
ff936c7 
Signed-off-by: Akshit Garg <[email protected]>
@gargakshit gargakshit marked this pull request as ready for review June 4, 2025 05:42
@jphenow jphenow changed the base branch from flyio-2.6.0-sidecars-upstreamable to flyio-2.6.0 June 6, 2025 14:58
@jphenow
Copy link
Member

jphenow commented Jun 6, 2025

Damn. I squash merged which messed with this. I'll try to fix in a sec

@jphenow jphenow mentioned this pull request Jun 6, 2025
Merged
@gargakshit
Copy link
Member Author

Okay wow this branch is messy :>
Need to bisect something but its going to be hard now :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@soulware soulware Awaiting requested review from soulware

@jphenow jphenow Awaiting requested review from jphenow

@kylemclaren kylemclaren Awaiting requested review from kylemclaren

@senyosimpson senyosimpson Awaiting requested review from senyosimpson

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@gargakshit @jphenow