Skip to content
This repository was archived by the owner on Jul 15, 2025. It is now read-only.

CI build & release workflows for Ubuntu, MacOS & Windows - #119

Merged
miteshashar merged 53 commits into
mainfrom
ci/macos-builds
Mar 28, 2025
Merged

CI build & release workflows for Ubuntu, MacOS & Windows#119
miteshashar merged 53 commits into
mainfrom
ci/macos-builds

Conversation

@miteshashar

@miteshashar miteshashar commented Mar 23, 2025 •
edited
Loading

Copy link
Copy Markdown
Contributor

@miteshashar
Add paths filter for main_admin workflow
4fcdd8f
@miteshashar
Remove main_gliner github workflow
e918e55
@miteshashar
Run test build for MacOS
9306e75
@miteshashar
Test CI build on MacOS with brew, llvm & rocksdb installed
daac840
@miteshashar
Test CI build with corrected OS check for MacOS
ba9785e
@miteshashar
Test CI build with lld installed and more env variables set
1e32fc1
@miteshashar
Test MacOS CI build with corrected paths llvm, rocksdb in env variables
3a22cb9
@miteshashar
Test MacOS CI build with empty DYLD_LIBRARY_PATH not resulting in tra…
b0e197f 
…iling `:`
@miteshashar
Test CI Build without any additional setup on MacOS 15
8ff7fd4
@miteshashar
Test CI Build on MacOS 15 with DYLD_FALLBACK_LIBRARY_PATH set
1560610 
Ref: https://crates.io/crates/opencv/0.54.0#troubleshooting
@miteshashar
Test CI Build on MacOS 15 with DYLD_FALLBACK_LIBRARY_PATH set
c5f966c 
Ref: https://crates.io/crates/opencv/0.54.0#troubleshooting
@miteshashar
Test CI Build on MacOS 15 with DYLD_FALLBACK_LIBRARY_PATH set
2d56d2a 
Ref: https://crates.io/crates/opencv/0.54.0#troubleshooting
@miteshashar
Test MacOS CI build with DYLD_LIBRARY_PATH & DYLD_FALLBACK_LIBRARY_PA…
e4efabe 
…TH set to llvm's lib path
@miteshashar
Run MacOS CI Build with an all-in config
9d041da
@miteshashar
Test MacOS CI build with libclang.dylib soft-linked from brew to loca…
190fef7 
…l lib
@miteshashar
Test MacOS CI build with libclang.dylib soft-linked from brew to loca…
f8ccb76 
…l lib
@miteshashar
Test MacOS CI build with libclang.dylib soft-linked from brew to loca…
ad73ae0 
…l lib
@miteshashar
Test MacOS CI build with libclang.dylib soft-linked from brew to loca…
932a186 
…l lib
@miteshashar
Combine branch & tag workflows, add check for API types & release con…
d15c510 
…ditionally
@miteshashar
Remove cwd definition for Test pixlie_ai in GHA
127e354
@miteshashar
Reintroduce old API type definition for testing CI
d67a7df
@miteshashar
Run CI build to check API types failure
bc9adbb
@miteshashar
Fix installation of prettier on CI
82962d2
@miteshashar
Add cacahe for homebrew and rust
58929e4
@miteshashar
Update latest backend changes of API types on the frontend
790c19b
@miteshashar
Add frontend API types to paths for main Pixlie AI GHA workflow
cdf5eb9
@miteshashar
Correct check for uncommitted API type changes in CI workflow
cdfa80b
@miteshashar
Re-enable ubuntu for backend CI builds, Combine release into single a…
77643f8 
…dmin CI workflow
@miteshashar
Disable Ubuntu builds in CI
c017801
@miteshashar
Fix CI errors for release files not being found
ae9fdba
@miteshashar
miteshashar requested a review from brainless March 23, 2025 11:37
@miteshashar miteshashar mentioned this pull request Mar 25, 2025
Closed
@miteshashar
Run test Windows build on GHA
829d322
@miteshashar
Finalize build & release GHA workflows
5a70ff3 
- Re-enable MacOS & Ubuntu for pixlie_ai os matrix
- Move commands to set environment variables for MacOS into a shell script
- Run archiving for admin build only for release tags
- Introduce concurrency to ensure old worklows do not override new ones, specially for releases
@miteshashar
Correct shell script path in pixlie_ai workflow
8f13785
@miteshashar miteshashar changed the title Fix CI build & release workflows for Ubuntu & MacOS CI build & release workflows for Ubuntu, MacOS & Windows Mar 25, 2025
@miteshashar

miteshashar commented Mar 26, 2025

Copy link
Copy Markdown
Contributor Author

We can merge this after the Windows binary is tested once.

@brainless

brainless commented Mar 27, 2025

Copy link
Copy Markdown
Member

@miteshashar let me know when the Windows build works, I will close this when you want.

@miteshashar

miteshashar commented Mar 28, 2025

Copy link
Copy Markdown
Contributor Author

Issues identified until now running the production build on MacOS:

  1. My existing dev rocksdb database couldnt be read. It throws the error Error from rocksdb: Not implemented: Unsupported compression method for this build: Snappy. 123
    • For now, I just backed up my current pixlie database and let the production build create it's own, which worked.
  2. The :58236 frontend delivery persisted to use a cached version. Initially, I was facing issues which were resolved after I merged main into this branch.
    • The home URL kept delivering the old homepage with a 304, which would keep redirecting to /settings/status. Hard reloads weren't working.
    • I found resolution only when I manually set "Disable cache" on the browser console.
    • So, for now I think we should deliver index.html with a Cache-Control: no-cache header. If that doesn't resolve the issue, we can deliver it with Cache-Control: no-store.
  3. After bypassing the above issues, it's now throwing this error.
    image

Footnotes

  1. https://github.com/solana-labs/solana/issues/32402 ↩

  2. https://github.com/solana-labs/solana/pull/32426 ↩

  3. https://github.com/rust-rocksdb/rust-rocksdb/issues/799 ↩

@miteshashar

miteshashar commented Mar 28, 2025

Copy link
Copy Markdown
Contributor Author

@brainless I have left the last release published on https://github.com/pixlie/PixlieAI/releases/tag/v0.2.0 intact. Can you test the Linux build?

@miteshashar

miteshashar commented Mar 28, 2025

Copy link
Copy Markdown
Contributor Author

@brainless Given that this PR is not atomic and has the following apart from fixing the CI/CD pipelines:

… I think we should handle #119 (comment) in a separate branch.

On merge, we can release a version v0.2.0-beta to ensure our developer experience does not break, else download_admin_site() will start breaking.

@miteshashar
miteshashar merged commit 51fabb0 into main Mar 28, 2025
@miteshashar
miteshashar deleted the ci/macos-builds branch March 28, 2025 14:02
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

@brainless brainless Awaiting requested review from brainless

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Admin site download does not work first time

2 participants

@miteshashar @brainless