feat: added app to fill a pvc in 1GB increments #24

wjro · 2025-04-29T19:58:12Z

wjro commented

2025-04-29 19:58:12 +00:00

WIP: This takes roughly 35 seconds to add 1GB to the db, which I am not sure is fast enough. I tried a bunch of other methods to speed it up, for example:

larger data chunks in 8mb and 16mb -> took about double the time,
create a file 1Gb file to pass to the container and have the app add that to the db -> then crashed because of the 16mb limit of mysql
a couple other things which either which crashed with a php error or crashed the app

WIP: This takes roughly 35 seconds to add 1GB to the db, which I am not sure is fast enough. I tried a bunch of other methods to speed it up, for example: - larger data chunks in 8mb and 16mb -> took about double the time, - create a file 1Gb file to pass to the container and have the app add that to the db -> then crashed because of the 16mb limit of mysql - a couple other things which either which crashed with a php error or crashed the app

wjro added 1 commit 2025-04-29 19:58:13 +00:00

feat: added app to fill a pvc in 1GB increments d7c17ceef9

johnride requested changes 2025-04-29 20:08:31 +00:00

examples/lamp/php/index.php Outdated

						
				@@ -3,0 +33,4 @@

				// Disk usage (PVC mount path)

				$mountPath = '/var/lib/mysql'; // Change this if PVC is mounted elsewhere

				$freeBytes = disk_free_space($mountPath);

johnride commented

2025-04-29 20:07:10 +00:00

Mariadb / mysql is not in the same container as the PHP process. You'll have to query mariadb stats or something else.

wjro added 5 commits 2025-05-01 20:24:05 +00:00

feat(HelmScore): Add values yaml option to helm chart score (#23 ) f8fb6b27d5

Co-authored-by: tahahawa <tahahawa@gmail.com>
Reviewed-on: #23

feat: add mariadb helm deployment to lamp interpreter 317db2342c

- Adds a `deploy_database` function to the `LAMPInterpret` struct to deploy a MariaDB database using Helm.
- Integrates `HelmCommand` trait requirement to the `LAMPInterpret` struct.
- Introduces `HelmChartScore` to manage MariaDB deployment.
- Adds namespace configuration for helm deployments.
- Updates trait bounds for `LAMPInterpret` to include `HelmCommand`.
- Implements `get_namespace` function to retrieve the namespace.

feat: push docker image to registry and deploy with full tag 6b47d4140f

- Added functionality to tag and push the built Docker image to a specified registry.
- Modified deployment score to use the full image tag (including registry and project).
- Included error handling and logging for the `docker tag` and `docker push` commands.
- Updated the `K8sDeploymentScore` struct to include a namespace field and environment variables for database credentials.
- Added kebab-case conversion for deployment name and namespace.
- Implemented a check_output function for better error reporting.

feat: Add comments explaining a bit of what harmony does in the lamp demo 6653cebb87

WIP: got the env variables to set in the php pod, need to make them dynamically be added rather than coded directly into lamp.rs ac24eb6206

wjro added 1 commit 2025-05-02 14:13:05 +00:00

fix: modified env vars to use values from helm deployment df3c8ee2a6

wjro added 1 commit 2025-05-02 14:50:53 +00:00

fix: added database size to lamp score to be able to specify database pvc size from lampScore b13766f960

wjro added 1 commit 2025-05-02 14:59:54 +00:00

bug: typo 73e15e12ce

johnride requested changes 2025-05-02 15:01:31 +00:00

johnride left a comment

I think the p-r should be recreated to make sure to have a clean diff.

For now there is only 1 comment for sure : the db_size must move and have a sensible default.

I think the p-r should be recreated to make sure to have a clean diff. For now there is only 1 comment for sure : the db_size must move and have a sensible default.

examples/lamp/src/main.rs Outdated

						
				@@ -17,3 +26,4 @@

				            project_root: "./php".into(),

				            ..Default::default()

				        },

				        db_size: "2Gi".to_string(),

johnride commented

2025-05-02 14:59:08 +00:00

This belongs in the LAMPConfig struct and should use the default option. I think 2G is a sensible default for now.

harmony/src/modules/lamp.rs

						
				@@ -40,3 +53,4 @@

				    fn create_interpret(&self) -> Box<dyn Interpret<T>> {

				        Box::new(LAMPInterpret {

				            score: self.clone(),

				            namespace: "harmony-lamp".to_string(),

johnride commented

2025-05-02 15:00:16 +00:00

use the score namespace.

But there seems to be something wrong with this p-r. There should not be so many changes.

use the score namespace. But there seems to be something wrong with this p-r. There should not be so many changes.

wjro closed this pull request

2025-05-02 15:43:13 +00:00

Pull request closed

This pull request cannot be reopened because the branch was deleted.

Sign in to join this conversation.

2 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: NationTech/harmony#24