Skip to content

feat: new build page #2026

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.

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

Merged
merged 23 commits into from
Apr 11, 2025
Merged

feat: new build page #2026

merged 23 commits into from
Apr 11, 2025

Conversation

ferruhcihan
Copy link
Collaborator

Implements

https://jira.linode.com/browse/APL-535
https://docs.google.com/document/d/1xAtM88sVC7VXAskz413Ec5AYVR7RDJoCcSZAaHYXZV0/edit?tab=t.0
https://www.figma.com/design/YuBiIolysFdtQgFEfAIJd1/APL-Master-Design?node-id=0-1&p=f&t=HFl0TjPGZV14lcGW-0

Description

This PR;

  • updates the values schema and team-ns templates for the new build page feature,
  • added separate migration function for the build image name field migration,
  • updates migrate tests for the new file structure,
  • adjust the tests/fixtures files according to the repo.ts kinds/types,
  • and also updates the some related files.

Is paired with: https://github.com/linode/apl-console/pull/536 and https://github.com/linode/apl-api/pull/678

Checklist

  • Architecture Design Records have been added as adr/*.md and appended to list in adr/_index.md, if applicable.
  • The values-schema.yaml file and test/** fixtures have been updated to reflect code changes, if applicable.
  • The OpenApi Schema from redkubes/otomi-api project is compatible with definitions from values-schema.yaml file, if applicable.
  • Helm releases are meeting otomi's baseline security policies, if applicable.
  • Helm chart and helmfile changes are tested against upgrade scenario, if applicable.

Copy link

github-actions bot commented Apr 1, 2025

Coverage report

St.
Category Percentage Covered / Total
🔴 Statements
48.28% (+0.72% 🔼)
997/2065
🔴 Branches
31.8% (+1.45% 🔼)
193/607
🔴 Functions
38.28% (+1.69% 🔼)
111/290
🔴 Lines
47.54% (+0.78% 🔼)
878/1847

Test suite run success

87 tests passing in 9 suites.

Report generated by 🧪jest coverage report action from cacf60a

@dennisvankekem dennisvankekem self-assigned this Apr 2, 2025
@ferruhcihan ferruhcihan requested a review from j-zimnowoda April 10, 2025 12:21
@j-zimnowoda j-zimnowoda self-assigned this Apr 10, 2025
@j-zimnowoda j-zimnowoda self-requested a review April 11, 2025 07:50
Copy link
Contributor

@dennisvankekem dennisvankekem left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

reverted versions back to main, LGTM

@dennisvankekem dennisvankekem merged commit 7bee88b into main Apr 11, 2025
7 checks passed
@dennisvankekem dennisvankekem deleted the APL-535 branch April 11, 2025 16:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants