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.

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

[Task]: Make the latest image adaptive #36478

Closed
abhvsn opened this issue Sep 23, 2024 · 0 comments · Fixed by #36569
Closed

[Task]: Make the latest image adaptive #36478

abhvsn opened this issue Sep 23, 2024 · 0 comments · Fixed by #36569
Assignees
@abhvsn
Labels
DB Infrastructure Pod Pod to handle database infrastructure Move to Postgres Issues required to be solved for the move to Postgres as repository layer Task A simple Todo

Comments

@abhvsn
Copy link
Contributor

abhvsn commented Sep 23, 2024 •
edited
Loading

SubTasks

Workflow for generating the latest and versioned tags is https://github.com/appsmithorg/appsmith/blob/release/.github/workflows/github-release.yml
@abhvsn abhvsn added Task A simple Todo Move to Postgres Issues required to be solved for the move to Postgres as repository layer labels Sep 23, 2024
@abhvsn abhvsn self-assigned this Sep 23, 2024
@github-actions github-actions bot added the DB Infrastructure Pod Pod to handle database infrastructure label Sep 23, 2024
@abhvsn abhvsn mentioned this issue Sep 26, 2024
Merged
2 tasks
abhvsn added a commit that referenced this issue Sep 30, 2024
@abhvsn
chore: Exit the script if server artifact placer is not available (#3…
73d3960 
…6569)

## Description
PR to fail fast at the build step only if the script to place the server
artifacts for pg and mongo is not available. This PR also replaces pg
tag with nightly tag for placing the server artifacts for pg to improve
the stability of the image shipped to customers.

Fixes: #36478

/test Sanity

### 🔍 Cypress test results
<!-- This is an auto-generated comment: Cypress test results  -->
> [!TIP]
> 🟢 🟢 🟢 All cypress tests have passed! 🎉 🎉 🎉
> Workflow run:
<https://github.com/appsmithorg/appsmith/actions/runs/11050797504>
> Commit: 40c5621
> <a
href="https://internal.appsmith.com/app/cypress-dashboard/rundetails-65890b3c81d7400d08fa9ee5?branch=master&workflowId=11050797504&attempt=1"
target="_blank">Cypress dashboard</a>.
> Tags: `@tag.Sanity`
> Spec:
> <hr>Thu, 26 Sep 2024 11:23:28 UTC
<!-- end of auto-generated comment: Cypress test results  -->


## Communication
Should the DevRel and Marketing teams inform users about this change?
- [ ] Yes
- [x] No


<!-- This is an auto-generated comment: release notes by coderabbit.ai
-->

## Summary by CodeRabbit

- **Bug Fixes**
- Improved error handling for missing scripts in the build and release
workflows, ensuring clearer error messages and preventing process
failures.

- **Chores**
- Introduced a new environment variable for using the nightly PostgreSQL
Docker image in the release process.

<!-- end of auto-generated comment: release notes by coderabbit.ai -->
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@abhvsn abhvsn

Labels
DB Infrastructure Pod Pod to handle database infrastructure Move to Postgres Issues required to be solved for the move to Postgres as repository layer Task A simple Todo
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

chore: Exit the script if server artifact placer is not available appsmithorg/appsmith
1 participant
@abhvsn