-
Notifications
You must be signed in to change notification settings - Fork 702
(POC) Add devfile v2 outerloop flow to console #6321
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
Closed
Closed
Changes from all commits
Commits
Show all changes
53 commits
Select commit
Hold shift + click to select a range
9d760b8
added isDevfilePresent function and associated tests
reginaelyse 104badb
added devfile import form and tile, incorporated isDevfilePresent to …
reginaelyse 0a2224e
removed devfile props from lazy loader
reginaelyse 9626a0a
Removed validation for devfiles with dockerfileValidationSchema
reginaelyse cfd0be7
Removed error from previous commit
reginaelyse 8013d74
Added beginnings of communication with backend
reginaelyse 761935d
Added functions to detect if devfile is present & to get devfile content
reginaelyse a08a584
Added parser to get devfile version and logic to use version
reginaelyse bc01c23
start of new function to use devfile for building
reginaelyse eb127b1
rough code for createOrUpdateBuildResource
jaideepr97 b4ebb85
more guideline code
jaideepr97 260fd93
pending test of mock data
jaideepr97 b5eae9c
Merge pull request #10 from reginapizza/jd-addDevfile
reginaelyse 9bbfb1f
removed promise return from devfileCreate
jaideepr97 f0478f9
fixed errors with devfileFlow
reginaelyse a1e5b33
Merge branch 'devfileBackendCommunication' of github.com:reginapizza/…
jaideepr97 4064c3d
solved merge conflicts from regina-devfile
jaideepr97 5f70552
mock build complete - need to revisit
jaideepr97 991337b
Merge pull request #12 from reginapizza/jd-addDevfile
reginaelyse a9a421b
adding SemVar function to compare versions
reginaelyse 578c7cf
Merge pull request #13 from reginapizza/regina-devfile
reginaelyse eac06dd
code restructured, mock data done - pending bug fixes
jaideepr97 b1482d1
frontend works with mock data
jaideepr97 22373da
Merge pull request #14 from reginapizza/jd-addDevfile
reginaelyse 2a81778
rebasing from upstream master
reginaelyse ce7114d
commented out metadata check that was removed in upstream
reginaelyse e96188a
Added new logo for devfile
reginaelyse 63bf40e
Added dev preview badge on devfile import form
reginaelyse 4623151
Fixed flow for edit application with devfile and devfile icon
reginaelyse 377208c
removed comment
reginaelyse 7294293
made builderImages a required field again
reginaelyse b457b48
Merge pull request #15 from reginapizza/devfileEditApp
reginaelyse 097278d
added annotation for isDevfilePresent to use for edit application
reginaelyse 0999bc2
fixing merge conflicts
reginaelyse 72db3d1
WIP- successful push to registry but still can't edit
reginaelyse 2434247
Vendor updates for api.
bigkevmcd e4f1f6c
fixed dependency issues, able to import openshift/api and build
jaideepr97 1241d21
attmepting pull from devfileBackendCommunication
jaideepr97 b5a783c
Merge pull request #16 from reginapizza/EditApplication
reginaelyse a38e8b8
import from devfile working with data coming from bridge
jaideepr97 54e8f25
Merge branch 'devfileBackendCommunication' of github.com:reginapizza/…
jaideepr97 55b426b
updated to support edit app redirect to import from devfile
jaideepr97 80dd462
figuring out service/route issue
jaideepr97 428c35f
fixed save button issue with mockdata frontend
jaideepr97 b986801
Merge pull request #17 from reginapizza/jd-odc-add-devfile
reginaelyse 3dc2606
added comments and code cleanup
jaideepr97 413ae83
fixed service/route not showing up issue
jaideepr97 00ed4a3
merged from upstream
reginaelyse 424d354
api addition and commenting out mocked data
reginaelyse f371d19
readding in jaideep's changes
reginaelyse 930af0b
Merge branch 'master' of https://github.com/openshift/console into de…
reginaelyse feb1f20
removed mocked data from frontend- using backend mocked only
reginaelyse 7054371
small code cleanup
reginaelyse File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
39 changes: 39 additions & 0 deletions
39
frontend/packages/dev-console/src/components/import/DevfileImportForm.tsx
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,39 @@ | ||
| import * as React from 'react'; | ||
| import * as _ from 'lodash'; | ||
| import { Form } from '@patternfly/react-core'; | ||
| import { FormikProps, FormikValues } from 'formik'; | ||
| import { FormFooter } from '@console/shared/src/components/form-utils'; | ||
| import { DevfileImportFormProps } from './import-types'; | ||
| import GitSection from './git/GitSection'; | ||
| import AppSection from './app/AppSection'; | ||
|
|
||
| const DevfileImportForm: React.FC<FormikProps<FormikValues> & DevfileImportFormProps> = ({ | ||
| values, | ||
| errors, | ||
| handleSubmit, | ||
| handleReset, | ||
| status, | ||
| builderImages, | ||
| isSubmitting, | ||
| dirty, | ||
| projects, | ||
| }) => ( | ||
| <Form onSubmit={handleSubmit} data-test-id="import-devfile-form"> | ||
| <GitSection buildStrategy= "Devfile" builderImages={builderImages}/> | ||
| <AppSection | ||
| project={values.project} | ||
| noProjectsAvailable={projects.loaded && _.isEmpty(projects.data)} | ||
| /> | ||
| <FormFooter | ||
| handleReset={handleReset} | ||
| errorMessage={status && status.submitError} | ||
| isSubmitting={isSubmitting} | ||
| submitLabel="Create" | ||
| sticky | ||
| disableSubmit={!dirty || !_.isEmpty(errors)} | ||
| resetLabel="Cancel" | ||
| /> | ||
| </Form> | ||
| ); | ||
|
|
||
| export default DevfileImportForm; |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Both these if conditions check for
warning. Looks like the condition for not reachable should be anerrorlooking at the logic above in this fileThere was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Interesting, I'll have to validate the UI code once I move it over. I'm trying to close off the ticket I'm working on but clusters seem to be having some issues today. Good catch though.