Commit Graph

403 Commits

Author SHA1 Message Date
Arpit Mohan
b2ff88a9e1 ci: Removing the redundant permissions in fat-container-test step 2022-09-12 11:11:30 +05:30
Arpit Mohan
5db7aa0fa1
ci: Reverting the permission check on ok-to-test
This needs to be tested properly on a test organization.
2022-09-11 20:50:59 +05:30
Arpit Mohan
4c55b0edd7 ci: Making ok-to-test command executable only to users with write permission on the appsmithorg organization 2022-09-11 17:57:40 +05:30
Arpit Mohan
978e68820e
ci: Fixing the notification & Depot Docker build step (#16671)
Fixing the notification & Depot Docker build step for integration test & test-build-docker workflows
2022-09-09 18:48:16 +05:30
Arpit Mohan
5642274694 ci: Removing depot docker build from fat-container-test in integration-tests.yml 2022-09-09 15:36:33 +05:30
Arpit Mohan
655a5a668d ci: Fixing the condition check for server running in ui-test and perf-test steps 2022-09-09 14:59:32 +05:30
Arpit Mohan
408bcaf1a9 ci: Moving ok-to-test permission to none till further debugging 2022-09-09 14:06:34 +05:30
Arpit Mohan
d5a6dacb2a ci: Fixing the yml syntax for test-build-docker image workflow 2022-09-09 11:53:21 +05:30
Arpit Mohan
c0e1a6202f
ci: Fixing the package step for test-build-docker workflow (#16661)
## Description

The depot docker step wasn't initialized. This was causing Docker builds to fail.

## Type of change

- Bug fix (non-breaking change which fixes an issue)

## How Has This Been Tested?

- Relying on CI

## Checklist:

- [ ] My code follows the style guidelines of this project
- [ ] I have performed a self-review of my own code
- [ ] I have commented my code, particularly in hard-to-understand areas
- [ ] I have made corresponding changes to the documentation
- [ ] My changes generate no new warnings
- [ ] I have added tests that prove my fix is effective or that my feature works
- [ ] New and existing unit tests pass locally with my changes
2022-09-09 11:47:34 +05:30
Arpit Mohan
121b1eb8e4
ci: Swap docker for depot (#16658)
Refer to #16020 for details. This PR has been opened since we could not modify the previous PR since it's a contribution by an external contributor.

Co-authored-by: kylegalbraith <kyle.galbraith459@gmail.com>
2022-09-09 01:43:03 +05:30
Arpit Mohan
697bdecaa5
ci: Modularizing the Github actions workflow for maintainability (#16505) 2022-09-09 01:08:55 +05:30
Arpit Mohan
7664f65e43 ci: Fixing the chrome stable version format 2022-09-07 18:50:54 +05:30
Arpit Mohan
52dfdd8eb4 ci: Moving to stable chrome version for Cypress 2022-09-07 17:49:21 +05:30
Arpit Mohan
05c109d220 ci: Fixing the yaml syntax for browser version 2022-09-07 15:47:53 +05:30
Aishwarya-U-R
9499f3188d
test: Running CI on latest Chrome browser (#16613)
## Description

- This PR makes changes to run CI on latest stable Chrome always

## Type of change

- Browser update

## Checklist:

- [X] My code follows the style guidelines of this project
- [X] I have performed a self-review of my own code
- [X] I have commented my code, particularly in hard-to-understand areas
- [ ] I have made corresponding changes to the documentation
- [X] My changes generate no new warnings
- [X] I have added tests that prove my fix is effective or that my feature works
- [X] New and existing unit tests pass locally with my changes
2022-09-07 15:44:14 +05:30
Arpit Mohan
577ac298da Moving the client build & package steps to Buildjet 2022-09-07 14:00:15 +05:30
Aishwarya-U-R
8a7ec8560e
test: Moving from Chrome to Electron browser for CI runs (#16598)
## Description

- This PR makes changes for moving from Chrome to Electron browser for CI runs & fixes the git hub runners failure issue incases of Chrome version updates

## Type of change

- Browser name update

## Checklist:

- [X] My code follows the style guidelines of this project
- [X] I have performed a self-review of my own code
- [X] I have commented my code, particularly in hard-to-understand areas
- [ ] I have made corresponding changes to the documentation
- [X] My changes generate no new warnings
- [X] I have added tests that prove my fix is effective or that my feature works
- [X] New and existing unit tests pass locally with my changes
2022-09-07 10:54:55 +05:30
Jaroslav Lobačevski
bf98cb992f
ci: Update ok-to-test.yml to run only on pull request comments (#16566)
The expression was treated as a string and always evaluated as true.
2022-09-06 22:05:21 +05:30
Ashok Kumar M
0fc7f4c7f6
fix: removing "Exit if Server hasn't started" coz its not needed. (#16589) 2022-09-06 19:03:32 +05:30
Ashok Kumar M
d4eee444f2
fix: Updating the server and client artifacts path. (#16585) 2022-09-06 17:14:13 +05:30
Ashok Kumar M
3594b81a9b
fix: fixing perf test workflow during merge. (#16574) 2022-09-06 15:02:34 +05:30
Ashok Kumar M
cefd0b6469
fix: adding perf test runs to workflows post merge to release. (#16559) 2022-09-06 13:09:14 +05:30
Ashok Kumar M
4c90f3ecd4
fix: adding git token for perf infra. (#16546)
* fix: adding git token for perf infra.

* changing name PERF_GITHUB_PAT
2022-09-05 18:27:13 +05:30
Ashok Kumar M
d0491f7a75
chore: replacing supabase secrets with performance db secrets. (#16423)
task: replacing supabase secrets with performance db secrets.
2022-09-01 15:02:46 +05:30
Arpit Mohan
3e00356e71 Increasing the memory for self-hosted runners for server build in CI 2022-08-27 16:45:47 +05:30
Arpit Mohan
0420161861
ci: Moving server builds to Buildjet to confirm if it's faster and more reliable (#16346)
Moving server builds to Buildjet to confirm if it's faster and more reliable

Also, cleaning up some CI workflows to remove references to release-frozen branch.
2022-08-27 16:15:59 +05:30
Arpit Mohan
037efacbf3
fix: Display community edition along with version (#16341)
## Description

This PR adds the string `Community` along with the Appsmith version. This will help users identify the type of instance that they are running.

## Type of change

- New feature (non-breaking change which adds functionality)

## How Has This Been Tested?

Manually

## Checklist:

- [ ] My code follows the style guidelines of this project
- [ ] I have performed a self-review of my own code
- [ ] I have commented my code, particularly in hard-to-understand areas
- [ ] I have made corresponding changes to the documentation
- [ ] My changes generate no new warnings
- [ ] I have added tests that prove my fix is effective or that my feature works
- [ ] New and existing unit tests pass locally with my changes
2022-08-27 15:10:03 +05:30
Tanvi Bhakta
17fd4f1512
fix: a typo (conatiner -> container) (#16200) 2022-08-23 10:44:54 +05:30
arslanhaiderbuttar
62b97c8203
fix: changed the download paths for s3 in CI workflows (#16194) 2022-08-22 22:50:31 +05:30
Arpit Mohan
943cfa9cd3
ci: Removing unused CI workflow files (#16142) 2022-08-19 05:28:39 +02:00
arslanhaiderbuttar
e9bc7063b6
Fix TED run command updated (#16089) 2022-08-18 10:09:12 +05:30
Satish Gandham
92a00e590e
ci: Remove directory listing command from perf tests (#16117)
* Clean up the perf folder and update integration commnds to use the new repo

* Use a different token for github auth

* Update the token

* - Remove directory listing command

* Move perf checkout command to the back

* resolve conflicts

* Add directory listing code

* Fix an issue with directory listing

* - Remove directory listing code

Co-authored-by: Satish Gandham <satish@appsmith.com>
2022-08-18 09:31:54 +05:30
Satish Gandham
5c7d2d0568
ci: Change directory listing code (#16109)
* Clean up the perf folder and update integration commnds to use the new repo

* Use a different token for github auth

* Update the token

* - Remove directory listing command

* Move perf checkout command to the back

* resolve conflicts

* Add directory listing code

* Fix an issue with directory listing

Co-authored-by: Satish Gandham <satish@appsmith.com>
2022-08-17 17:42:32 +05:30
Satish Gandham
47e5040c86
ci: Move performance-infra checkout to the back in ci script (#16107)
* Clean up the perf folder and update integration commnds to use the new repo

* Use a different token for github auth

* Update the token

* - Remove directory listing command

* Move perf checkout command to the back

* resolve conflicts

* Add directory listing code

Co-authored-by: Satish Gandham <satish@appsmith.com>
2022-08-17 16:27:38 +05:30
Satish Gandham
d232823edb
ci: Remove directory listing command from perf tests (#16084)
* Clean up the perf folder and update integration commnds to use the new repo

* Use a different token for github auth

* Update the token

* - Remove directory listing command

Co-authored-by: Satish Gandham <satish@appsmith.com>
2022-08-17 12:30:48 +05:30
Satish Gandham
50014eef78
ci: Update the access token in ci script (#16058)
* Clean up the perf folder and update integration commnds to use the new repo

* Use a different token for github auth

* Update the token

Co-authored-by: Satish Gandham <satish@appsmith.com>
2022-08-16 16:44:44 +05:30
Satish Gandham
3990162ca6
ci: Update the access token in ci script. (#16032)
* Clean up the perf folder and update integration commnds to use the new repo

* Use a different token for github auth

Co-authored-by: Satish Gandham <satish@appsmith.com>
2022-08-16 14:33:55 +05:30
Satish Gandham
a76fd48ddc
chore: Update integration script to use performance testing code from private repo (#15991)
Clean up the perf folder and update integration commnds to use the new repo

Co-authored-by: Satish Gandham <satish@appsmith.com>
2022-08-16 11:54:13 +05:30
arslanhaiderbuttar
058d90cea0
test: Update workflows for upgrade scenarios (#15841)
Update workflows for upgrade scenarios
2022-08-15 14:08:30 +00:00
Arpit Mohan
ec29bbfac4
ci: Running the "Test plan approved" check only on pull requests to release branch (#15850) 2022-08-08 22:24:00 +05:30
Satish Gandham
e2db190ed2
ci: Stabilise the host before running perf tests (#15130)
* Run perf tests only on buildjet machine

* - Set chrome process priority to maximum to stabilise the run data

* Update index.js

* Update perf.js

* Update index.js

* - Move around args

* - Add --no-sandbox option to initial setup

* -We don't need sudo on the main script, we can use sudo in node scripts to set process priority

* - Some tweaks to the perf scripts.

* - Remove no-sandbox option in puppeteer launch

* - Remove sudo at unnecessary places

* - Add code to force grabage collection in chrome

* Update logins and some cleanup

* Remove unsed import

Co-authored-by: Satish Gandham <satish@appsmith.com>
2022-07-11 21:22:25 +05:30
albinAppsmith
28b2561d32
ci: Design system label name fix (#14996)
Design system label name fix
2022-07-07 22:00:06 +05:30
albinAppsmith
8c1a4af4b9
ci: Added workflow for adding reviewers to PR if ads folder changed (#14880)
Added workflow for adding reviwers to PR if ads folder chnaged
2022-07-01 01:48:16 +05:30
yatinappsmith
703b0efda6
Add matrix for fat container tests (#14850) 2022-06-30 11:32:40 +05:30
Arpit Mohan
1d76fe3ceb
ci: Revert "ci: Use buildjet runners for Cypress tests, and fewer of them"
Reverts #14636

Reverting this change for now because there is an issue when re-running the failed specs. The action is unable to find the correct directory.
2022-06-22 09:57:40 +05:30
Hetu Nandu
09bd4e35bd
ci: Update Test plan checker (#14633) 2022-06-20 15:10:36 +05:30
Shrikant Sharat Kandula
054b6942b8
Use buildjet runners for Cypress tests, and fewer of them (#14636) 2022-06-20 11:52:17 +05:30
Satish Gandham
4f62ad799b
Remove sudo from perf tests command (#14649)
We don't need sudo on the main script, we can use sudo in node scripts to set process priority
2022-06-17 18:10:04 +05:30
Arpit Mohan
abffbaae1e
Adding the configuration parameter to ensure that the target branch is master for all releases are created (#14615) 2022-06-17 11:31:08 +05:30
Satish Gandham
88a2fc228e
Run UI performance tests as root (#14536)
We want to run chrome at higher priority on OS level, to set the process priority we need to run the script as root.
2022-06-14 19:10:10 +05:30