Skip to content

Commit

Permalink
Merge current master into devmain (#606)
Browse files Browse the repository at this point in the history
* chore: move src to a new location

Signed-off-by: Nastya Rusina <[email protected]>

* ci: allow branch to be built again

Signed-off-by: Nastya Rusina <[email protected]>

* chore: replace link to github repository to a proper one

Signed-off-by: Nastya Rusina <[email protected]>

* ci: basic test and test-coverage setup

Signed-off-by: Nastya Rusina <[email protected]>

* chore: add first plugin package

Signed-off-by: Nastya Rusina <[email protected]>

* chore: allow to start test from main directory

Signed-off-by: Nastya Rusina <[email protected]>

* ci: update lowest mkdirp version (#423)

Signed-off-by: Nastya Rusina <[email protected]>

* chore: add build scripts for console app (#429)

Signed-off-by: Carina Ursu <[email protected]>

Co-authored-by: Carina Ursu <[email protected]>

* fix: hide the legend (#435)

Signed-off-by: eugenejahn <[email protected]>

* ci: move storybook related packages to top level (#434)

* ci: move storybook related packages to top level
* ci: ensure that lint can be run in PR checks
* ci: allow to run tests linux setup

Signed-off-by: Nastya Rusina <[email protected]>

* fix: fix issue 386 (#437)

Signed-off-by: eugenejahn <[email protected]>

* ci: ensure unified tsconfig and remove mocks, test, stories from build (#436)

* ci: ensure unified tsconfig and remove mocks,test,stories from build
* ci: move certificates into certificate folder inside script

Signed-off-by: Nastya Rusina <[email protected]>

* Make whole row clickable to open TaskExecutionDetails panel (#444)

* fix: issue 398
* fix: prevent parent onclick event trigger

Signed-off-by: eugenejahn <[email protected]>

* ci: allow to start jest config for all project at once (#447)

* minor: unable to view all the workflow versions (#446)

* fix: unable to view all the workflow versions

Signed-off-by: Carina Ursu <[email protected]>

* chore: cleanup

Signed-off-by: Carina Ursu <[email protected]>

Co-authored-by: Carina Ursu <[email protected]>

* chore(tsc): exclude test/stories/mock files only from build (#451)

* chore(tsc): exclude test/stories/mock files only from build

Signed-off-by: Nastya Rusina <[email protected]>

* ci: ensure that webpack too doesn't include test/mock/specs files (#452)

Signed-off-by: Nastya Rusina <[email protected]>

* minor: add support for StructuredDataSet Input/Output type (#445)

* chore: support for StructuredDataSet Input/Output type

Signed-off-by: Carina Ursu <[email protected]>

* chore: add tests

Signed-off-by: Carina Ursu <[email protected]>

* chore: add package

Signed-off-by: Carina Ursu <[email protected]>

* chore: resolving local package issue

Signed-off-by: Carina Ursu <[email protected]>

* chore: yarn lock

Signed-off-by: Carina Ursu <[email protected]>

* chore: stories

Signed-off-by: Carina Ursu <[email protected]>

* chore: fix associative arrays in copy functionality

Signed-off-by: Carina Ursu <[email protected]>

* chore: fix scalar stories

Signed-off-by: Carina Ursu <[email protected]>

* chore: oops

Signed-off-by: Carina Ursu <[email protected]>

* chore: cleanup

Signed-off-by: Carina Ursu <[email protected]>

* chore: revert old viewer to original

Signed-off-by: Carina Ursu <[email protected]>

* chore: fix

Signed-off-by: Carina Ursu <[email protected]>

* chore: remove non-null assertion

Signed-off-by: Carina Ursu <[email protected]>

* chore: fix types

Signed-off-by: Carina Ursu <[email protected]>

* chore: cleanup

Signed-off-by: Carina Ursu <[email protected]>

* chore: fix tests

Signed-off-by: Carina Ursu <[email protected]>

* chore: fix test types

Signed-off-by: Carina Ursu <[email protected]>

* chore: cleanup

Signed-off-by: Carina Ursu <[email protected]>

* chore: cleanup

Signed-off-by: Carina Ursu <[email protected]>

* chore: cleanup

Signed-off-by: Carina Ursu <[email protected]>

* chore: cleanup

Signed-off-by: Carina Ursu <[email protected]>

Co-authored-by: Carina Ursu <[email protected]>

* chore: Provide basic repo structure and ensure that stories, tests, vscode setup properly (#458)

* chore: separate AppInfo
* chore: vscode properly understands @flyteconsole
* chore: add basics/locale entry
* test: add storybooks and tests
* test: fix istanbul test-coverage issue by binding babel versions

Signed-off-by: Nastya Rusina <[email protected]>

* ci: minimize babel presence (#457)

* ci: minimize babel presence

Signed-off-by: Nastya Rusina <[email protected]>

* fix: update node executions to display map tasks (#455)

* fix: update node executions to display map tasks
* fix: update map task logs styles
* test: add/update unit tests
* fix: fix flickering and unnecessary re-renders

Signed-off-by: Olga Nad <[email protected]>

* ci: resolve docker build issues (#462)

* ci: resolve docker build issues
* chore: remove unneeded changes

Signed-off-by: Nastya Rusina <[email protected]>

* Chore: Rebase devmain on top of master (#464)

* fix: ensure that admin version is fully shown (#465)

* chore: fix for console not respecting admin url (#468)

* chore: fix

Signed-off-by: Carina Ursu <[email protected]>

* chore: remove console

Signed-off-by: Carina Ursu <[email protected]>

* chore: add cherio o=to package

Signed-off-by: Carina Ursu <[email protected]>

* chore: add cherio o=to package

Signed-off-by: Carina Ursu <[email protected]>

* chore: fix start prod

Signed-off-by: Carina Ursu <[email protected]>

* chore: fix start prod

Signed-off-by: Carina Ursu <[email protected]>

* chore: cleanup

Signed-off-by: Carina Ursu <[email protected]>

* chore: cleanup

Signed-off-by: Carina Ursu <[email protected]>

Co-authored-by: Carina Ursu <[email protected]>

* chore: rebase on top of master

Signed-off-by: Nastya <[email protected]>

* fix: trigger release #patch (#477)

Signed-off-by: Nastya <[email protected]>

* feat: add task version info (#485)

made the entities component become more generic that can support Workflow, Task, and others
added the inputs and outputs to task details page
added the versions table in task details page
added the version details page
added task details link in node side panel

Signed-off-by: eugenejahn <[email protected]>

Co-authored-by: Eugene Jahn <[email protected]>

* chore: update Readme with python note + package version update (#486)

Signed-off-by: Nastya Rusina <[email protected]>

Co-authored-by: Nastya Rusina <[email protected]>

* hotfix ci pipeline (#487)

* hotfix ci pipeline

Signed-off-by: Yuvraj <[email protected]>

* more changes

Signed-off-by: Yuvraj <[email protected]>

Co-authored-by: Yuvraj <[email protected]>

* [Mapping][TaskInfo] V.2 - Update Task details to allow check information for child task execution (#467)

* fix: add eventVersion check for map tasks (#484)

* fix: resolve minimist package to v1.2.6 (#492)

Signed-off-by: Nastya <[email protected]>

* ci: add flyte-api plugin package (#490)

Signed-off-by: Nastya <[email protected]>

* fix: update/add graph related stories (#493)

* fix: separate api auth to plugin folder (#495)

* fix: separate api auth to plugin folder
* chore: for local admin URL is undefined
* chore: update README.md
* test: fix tests + add new ones
* v0.0.2 - release for flyte-api plugin

Signed-off-by: Nastya <[email protected]>

* feat: rerun task action in execution page  (#488)

* feat: task rerun done

Signed-off-by: Eugene Jahn <[email protected]>

* feat: fix initialParameters

Signed-off-by: Eugene Jahn <[email protected]>

* fix: remove a file

Signed-off-by: Eugene Jahn <[email protected]>

* feat: rerun task done

Signed-off-by: Eugene Jahn <[email protected]>

* feat: rerun fix literal type

Signed-off-by: Eugene Jahn <[email protected]>

* feat: small typo fixed

Signed-off-by: Eugene Jahn <[email protected]>

* feat: small typo fixed

Signed-off-by: Eugene Jahn <[email protected]>

* feat: fix test coverage

Signed-off-by: Eugene Jahn <[email protected]>

* chore: update release version (#501)

Signed-off-by: Nastya <[email protected]>

* chore(deps-dev): bump protobufjs from 6.8.9 to 6.11.3 (#502)

Bumps [protobufjs](https://github.com/protobufjs/protobuf.js) from 6.8.9 to 6.11.3.
- [Release notes](https://github.com/protobufjs/protobuf.js/releases)
- [Changelog](https://github.com/protobufjs/protobuf.js/blob/v6.11.3/CHANGELOG.md)
- [Commits](https://github.com/protobufjs/protobuf.js/commits/v6.11.3)

---
updated-dependencies:
- dependency-name: protobufjs
  dependency-type: direct:development
...

Signed-off-by: dependabot[bot] <[email protected]>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* chore: add package generator for basics/composites/plugins (#503)

* chore: add package generator for basics/composites/plugins

Signed-off-by: Nastya <[email protected]>

* Update CONTRIBUTING.md  with package generate info

* fix(bug-508): executions can not be filtered by start time (#509)

Signed-off-by: Nastya <[email protected]>

* chore(deps-dev): bump semantic-release from 17.2.3 to 19.0.3 (#510)

Bumps [semantic-release](https://github.com/semantic-release/semantic-release) from 17.2.3 to 19.0.3.
- [Release notes](https://github.com/semantic-release/semantic-release/releases)
- [Commits](semantic-release/semantic-release@v17.2.3...v19.0.3)

---
updated-dependencies:
- dependency-name: semantic-release
  dependency-type: direct:development
...

Signed-off-by: dependabot[bot] <[email protected]>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* feat: support flyte decks (#504)

* feat: support flyte deck #none

Signed-off-by: James <[email protected]>

* fix: rebase with master #none

Signed-off-by: James <[email protected]>

* fix: fix iframe height and api response type; #none

Signed-off-by: James <[email protected]>

* fix: modal style; #none;

Signed-off-by: James <[email protected]>

* fix: use env variable for server ssl config #none;

Signed-off-by: James <[email protected]>

* fix: use h2 instead of h3 for future migration #none

Signed-off-by: James <[email protected]>

* feat: navbar navigation dropdown (#511)

* feat: navbar navigstion dropdown
* chore: storybook update
* chore: fix test + add base readme

Signed-off-by: Nastya <[email protected]>

* chore: support internal/external navigsation better (#513)

Signed-off-by: Nastya <[email protected]>

* Update Contributing.md (#515)

Add line how to add nodejs plugin to asdf, to simplify setup

* chore: update navigationDropdown usage (#517)

Signed-off-by: Work <[email protected]>

Co-authored-by: Work <[email protected]>

* chore(deps): bump jsdom from 16.4.0 to 16.7.0 (#516)

Bumps [jsdom](https://github.com/jsdom/jsdom) from 16.4.0 to 16.7.0.
- [Release notes](https://github.com/jsdom/jsdom/releases)
- [Changelog](https://github.com/jsdom/jsdom/blob/master/Changelog.md)
- [Commits](jsdom/jsdom@16.4.0...16.7.0)

---
updated-dependencies:
- dependency-name: jsdom
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <[email protected]>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* feat: launch plans list & detail page #none; (#507)

* feat: launch plans list #none;
* fix: launch plan detail page the top navigation header
* fix: added expected inputs and fixed inputs; #none
* fix: remove radio buttons for view all launch plan versions; #none

Signed-off-by: James <[email protected]>

* chore: release 1.1.3 (#520)

Signed-off-by: Nastya Rusina <[email protected]>

* fix: cache icon fro map task (#519)

* fix: cache icon fro map task
* fix: icon and text center

Signed-off-by: eugenejahn <[email protected]>

* fix: Relaunch form does not persist security context values when changed (#527)

fix: use execution security context in relaunch; #none

Signed-off-by: James <[email protected]>

* fix: release process (#529)

* fix release process

Signed-off-by: Yuvraj <[email protected]>

* more changes

Signed-off-by: Yuvraj <[email protected]>

* more changes

Signed-off-by: Yuvraj <[email protected]>

* fix workflow

Signed-off-by: Yuvraj <[email protected]>

Co-authored-by: Yuvraj <[email protected]>

* fix: fix semantic-release config (#532)

Signed-off-by: Yuvraj <[email protected]>

* test: fix time sensitive test (#533)

chore: fix test

Signed-off-by: Nastya Rusina <[email protected]>

* Rename upgrade idl workflow (#534)

Signed-off-by: Yuvraj <[email protected]>

Co-authored-by: Yuvraj <[email protected]>

* fix: support mapped tasks (#494)

* fix: support mapped tasks #none

Signed-off-by: James <[email protected]>

* fix: fix comments #none

Signed-off-by: James <[email protected]>

* fix: fix unit test #none

Signed-off-by: James <[email protected]>

* fix: add string constants #none

Signed-off-by: James <[email protected]>

* fix: added test for mapInputHelper #none

Signed-off-by: James <[email protected]>

* fix: fix test for utils.test.ts #none

Signed-off-by: James <[email protected]>

* chore: trigger snyk re-run

* fix: multiple keys for mapped types; #none

Signed-off-by: James <[email protected]>

* chore: storybook item (#530)

Signed-off-by: Nastya Rusina <[email protected]>

* fix: fix validation for duplicate and fix focus issue

Signed-off-by: James <[email protected]>

* chore: provide previous run values on relaunch

Signed-off-by: Nastya Rusina <[email protected]>

Co-authored-by: Nastya <[email protected]>
Co-authored-by: Nastya Rusina <[email protected]>

* fix(491): remove favicon package + use favicon.svg by default (#537)

* fix(491): remove favicon package + use favicon.svg by default

Signed-off-by: Nastya Rusina <[email protected]>

* chore: resolve yarn.lock merge conflict

Signed-off-by: Nastya Rusina <[email protected]>

* Fixed undefined task input types access in NodeExecutionActions (#538)

* Fixed undefined task input types access in NodeExecutionActions
Issue occurred due to race condition while loading data, indentical fix applies as for #506

Signed-off-by: Nick Müller <[email protected]>

* fix: cannot read properties of undefined (#506)

fix: nullref

Signed-off-by: Carina Ursu <[email protected]>

Co-authored-by: Carina Ursu <[email protected]>
(cherry picked from commit d38b98b)

Signed-off-by: Nick Müller <[email protected]>

Co-authored-by: Carina Ursu <[email protected]>

* feat: support union type for launch plan (#540)

* feat: support union type for launch plan

Signed-off-by: eugenejahn <[email protected]>

* fix: format

Signed-off-by: eugenejahn <[email protected]>

* fix: update type label

Signed-off-by: eugenejahn <[email protected]>

* fix: update the format

Signed-off-by: eugenejahn <[email protected]>

* fix: graph has realtime updates as execution progresses (#543)

Signed-off-by: Olga Nad <[email protected]>

* fix: make sure groups used in graph aren't undefined (#545)

Signed-off-by: Olga Nad <[email protected]>

* Hotfix docker push GHWF (#547)

* Rename upgrade idl workflow

Signed-off-by: Yuvraj <[email protected]>

* fix docker push wf

Signed-off-by: Yuvraj <[email protected]>

Co-authored-by: Yuvraj <[email protected]>

* fix: Graph Center on initial render (#541)

* fix: upgrade react-flow-renderer version

Signed-off-by: James <[email protected]>

* fix: use setTimeout for queue on the next render

Signed-off-by: James <[email protected]>

* fix: use one state variable and fitView after nodesChange

Signed-off-by: James <[email protected]>

* introduce needFitView

* fix: graph edge overlaps nodes (#542)

* fix: upgrade react-flow-renderer version

Signed-off-by: James <[email protected]>

* fix: use setTimeout for queue on the next render

Signed-off-by: James <[email protected]>

* fix: use one state variable and fitView after nodesChange

Signed-off-by: James <[email protected]>

* fix: graph edge overlaps nodes issue

Signed-off-by: James <[email protected]>

* introduce needFitView

* edge overlap

* fix: flyteconsole tag in ci pipeline  (#550)

* fix tag issue in ci

Signed-off-by: Yuvraj <[email protected]>

* remove welcome bot from boilerplate config

Signed-off-by: Yuvraj <[email protected]>

Co-authored-by: Yuvraj <[email protected]>

* enable docker push for console (#552)

* chore(deps): bump terser from 4.8.0 to 4.8.1 (#548)

Bumps [terser](https://github.com/terser/terser) from 4.8.0 to 4.8.1.
- [Release notes](https://github.com/terser/terser/releases)
- [Changelog](https://github.com/terser/terser/blob/master/CHANGELOG.md)
- [Commits](https://github.com/terser/terser/commits)

---
updated-dependencies:
- dependency-name: terser
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <[email protected]>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Update Flyteidl version (#558)

Signed-off-by: Flyte-Bot <[email protected]>

Signed-off-by: Flyte-Bot <[email protected]>
Co-authored-by: flyte-bot <[email protected]>

* fix: fix searchbar X button (#564)

chore: fix searchbar X button

Signed-off-by: Carina Ursu <[email protected]>

Signed-off-by: Carina Ursu <[email protected]>

* fix: update timeline view to show dynamic wf internals on first render (#562)

* fix: update timeline view to show dynamic wf internals on first render

Signed-off-by: Olga Nad <[email protected]>

* fix: update tests and clean up code

Signed-off-by: Olga Nad <[email protected]>

* fix: test

Signed-off-by: Olga Nad <[email protected]>

Signed-off-by: Olga Nad <[email protected]>

* fix: webmanifest missing crossorigin attribute (#566)

Signed-off-by: Carina Ursu <[email protected]>

Signed-off-by: Carina Ursu <[email protected]>

* fix: console showing subworkflows as unknown (#570)

* fix: console showing subworkflows as unknown

Signed-off-by: Olga Nad <[email protected]>

* fix: replace comparison with lodash isEqual

Signed-off-by: Olga Nad <[email protected]>

* fix: remove checkIfObjectsAreSame

Signed-off-by: Olga Nad <[email protected]>

Signed-off-by: Olga Nad <[email protected]>

* fix: Dict value loses 1 trailing character on UI Launch. (#561)

fix: dict lose trailing issue

Signed-off-by: James <[email protected]>

Signed-off-by: James <[email protected]>

* fix: launchform validation (#557)

* fix: disable launch button and show validation messaages

Signed-off-by: James <[email protected]>

* fix: code clean

Signed-off-by: James <[email protected]>

Signed-off-by: James <[email protected]>

* fix: integrate timeline and graph tabs wrappers under one component (#572)

* fix: integrate timeline and graph tabs wrappers under one component

Signed-off-by: Olga Nad <[email protected]>

* fix: details tab in graph view and clean-up

Signed-off-by: Olga Nad <[email protected]>

* fix: old unit tests

Signed-off-by: Olga Nad <[email protected]>

* fix: temp filename change

Signed-off-by: Olga Nad <[email protected]>

* fix: final filename change

Signed-off-by: Olga Nad <[email protected]>

* fix: tsconfig

Signed-off-by: Olga Nad <[email protected]>

Signed-off-by: Olga Nad <[email protected]>

* chore(deps-dev): bump moment from 2.29.3 to 2.29.4 (#549)

Bumps [moment](https://github.com/moment/moment) from 2.29.3 to 2.29.4.
- [Release notes](https://github.com/moment/moment/releases)
- [Changelog](https://github.com/moment/moment/blob/develop/CHANGELOG.md)
- [Commits](moment/moment@2.29.3...2.29.4)

---
updated-dependencies:
- dependency-name: moment
  dependency-type: direct:development
...

Signed-off-by: dependabot[bot] <[email protected]>

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* added none type in union type (#577)

* added none type in union type

Signed-off-by: eugenejahn <[email protected]>

* udpated the test case

Signed-off-by: eugenejahn <[email protected]>

* fix: fixed eslint

Signed-off-by: eugenejahn <[email protected]>

Signed-off-by: eugenejahn <[email protected]>

* fix: correctly coerce `defaultValue` when rendering LaunchPlan details (#578)

fix: correctly coerce defaultValue when rendering LaunchPlan details

Signed-off-by: Rahul Mehta <[email protected]>

Signed-off-by: Rahul Mehta <[email protected]>

* #minor: inputHelpers InputProps (#579)

fix: inputHelpers InputProps

Signed-off-by: Carina Ursu <[email protected]>

Signed-off-by: Carina Ursu <[email protected]>

* fix: fix test of launchform (#581)

* fix: fix test of launchform

Signed-off-by: James <[email protected]>

* fix: inputHelpers tests regression

Signed-off-by: Olga Nad <[email protected]>

* fix: update tests description

Signed-off-by: Olga Nad <[email protected]>

* fix: spelling typo

Signed-off-by: Olga Nad <[email protected]>

Signed-off-by: James <[email protected]>
Signed-off-by: Olga Nad <[email protected]>
Co-authored-by: Olga Nad <[email protected]>

* Pruning some unused packages (#583)

Signed-off-by: Jason Porter <[email protected]>

Signed-off-by: Jason Porter <[email protected]>

* feat: minor change (#584)

Minor change

Signed-off-by: Jason Porter <[email protected]>

Signed-off-by: Jason Porter <[email protected]>

* fix: floor seconds to int in the edge case moment returns it as float (#582)

Signed-off-by: Olga Nad <[email protected]>

Signed-off-by: Olga Nad <[email protected]>

* Update Flyteidl version (#575)

Signed-off-by: Flyte-Bot <[email protected]>

Signed-off-by: Flyte-Bot <[email protected]>
Co-authored-by: flyte-bot <[email protected]>

* fix: add BASE_URL to dev startup, open deeply nested urls (#589)

* fix: add BASE_URL to dev startup, open deeply nested urls

Signed-off-by: Carina Ursu <[email protected]>

* fix: remove debug code

Signed-off-by: Carina Ursu <[email protected]>

* chore: empty BASE_URL handling

Signed-off-by: Carina Ursu <[email protected]>

Signed-off-by: Carina Ursu <[email protected]>

* fix: add default disabled state for only mine filter (#585)

* fix: add default disabled state for only mine filter

Signed-off-by: Olga Nad <[email protected]>

* fix: tests

Signed-off-by: Olga Nad <[email protected]>

* fix: use api context to default filter state

Signed-off-by: Olga Nad <[email protected]>

* fix: revert test updates

Signed-off-by: Olga Nad <[email protected]>

Signed-off-by: Olga Nad <[email protected]>

* Update Flyteidl version (#590)

Signed-off-by: Flyte-Bot <[email protected]>

Signed-off-by: Flyte-Bot <[email protected]>
Co-authored-by: flyte-bot <[email protected]>

* Launch plan ref v2 (#601)

* progress

* Fixed

* Removed debug code

Signed-off-by: Jason Porter <[email protected]>

Signed-off-by: Jason Porter <[email protected]>

* fix: enable deeplinks in development (#602)

chore: enable deeplinks in development

Signed-off-by: Carina Ursu <[email protected]>

Signed-off-by: Carina Ursu <[email protected]>

* chore: release 1.3.5 (#605)

Signed-off-by: Jason Porter <[email protected]>

Signed-off-by: Jason Porter <[email protected]>

Signed-off-by: Nastya Rusina <[email protected]>
Signed-off-by: eugenejahn <[email protected]>
Signed-off-by: Olga Nad <[email protected]>
Signed-off-by: Nastya <[email protected]>
Signed-off-by: Eugene Jahn <[email protected]>
Signed-off-by: James <[email protected]>
Signed-off-by: Yuvraj <[email protected]>
Signed-off-by: Flyte-Bot <[email protected]>
Signed-off-by: Carina Ursu <[email protected]>
Signed-off-by: dependabot[bot] <[email protected]>
Signed-off-by: Rahul Mehta <[email protected]>
Signed-off-by: Jason Porter <[email protected]>
Co-authored-by: Nastya Rusina <[email protected]>
Co-authored-by: Nastya <[email protected]>
Co-authored-by: Carina Ursu <[email protected]>
Co-authored-by: Carina Ursu <[email protected]>
Co-authored-by: Eugene Jahn <[email protected]>
Co-authored-by: apTalya <[email protected]>
Co-authored-by: Yuvraj <[email protected]>
Co-authored-by: Yuvraj <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: james-union <[email protected]>
Co-authored-by: Work <[email protected]>
Co-authored-by: Nick Müller <[email protected]>
Co-authored-by: Flyte Bot <[email protected]>
Co-authored-by: flyte-bot <[email protected]>
Co-authored-by: Rahul Mehta <[email protected]>
Co-authored-by: Jason Porter <[email protected]>
  • Loading branch information
17 people authored Sep 30, 2022
1 parent d38b98b commit f8cb277
Show file tree
Hide file tree
Showing 164 changed files with 6,677 additions and 2,306 deletions.
43 changes: 29 additions & 14 deletions .github/workflows/checks.yml
Original file line number Diff line number Diff line change
Expand Up @@ -12,14 +12,14 @@ on:
- devmain

jobs:
extraxt_branch:
extract_branch:
runs-on: ubuntu-latest
outputs:
currentTag: ${{ steps.extract_branch.outputs.extract_branch }}
branch: ${{ steps.extract_branch.outputs.branch }}
steps:
- name: Extract branch name
shell: bash
run: echo "##[set-output name=branch;]$(echo ${GITHUB_REF#refs/heads/})"
run: echo "::set-output name=branch::$(echo ${GITHUB_HEAD_REF:-${GITHUB_REF#refs/heads/}})"
id: extract_branch

unit_tests_with_coverage:
Expand Down Expand Up @@ -66,7 +66,7 @@ jobs:
release:
name: Generate Release
if: ${{ (github.event_name != 'pull_request') && (needs.extract_branch.outputs.branch == 'master') }}
needs: [unit_tests_with_coverage, lint_project, build_docker_image, extraxt_branch]
needs: [unit_tests_with_coverage, lint_project, build_docker_image, extract_branch]
runs-on: ubuntu-latest
steps:
- name: Checkout
Expand Down Expand Up @@ -107,17 +107,32 @@ jobs:
# a tag, update our target
run: |
CURRENT_TAG=$(git tag --sort=-refname --points-at ${{ github.ref }} | head -n 1)
echo $CURRENT_TAG
echo "::set-output name=currentTag::$CURRENT_TAG"
push_docker_image:
name: Build & Push Flyteconsole Image
name: Push to Github Registry
needs: [check_for_tag]
uses: flyteorg/flytetools/.github/workflows/publish.yml@master
with:
version: ${{ needs.check_for_tag.outputs.currentTag }}
dockerfile: Dockerfile
push: true
repository: ${{ github.repository }}
secrets:
FLYTE_BOT_PAT: ${{ secrets.FLYTE_BOT_PAT }}
FLYTE_BOT_USERNAME: ${{ secrets.FLYTE_BOT_USERNAME }}
runs-on: ubuntu-latest
if: ${{ needs.check_for_tag.outputs.currentTag != '' }}
steps:
- uses: actions/checkout@v2
with:
fetch-depth: '0'
- name: Autobump version
env:
TAG: ${{ needs.check_for_tag.outputs.currentTag }}
run: |
VERSION=${TAG:1} make update_npmversion
- name: Push Docker Image to Github Registry
uses: whoan/docker-build-with-cache-action@v5
with:
username: "${{ secrets.FLYTE_BOT_USERNAME }}"
password: "${{ secrets.FLYTE_BOT_PAT }}"
image_name: flyteorg/flyteconsole
image_tag: latest,${{ github.sha }},${{ needs.check_for_tag.outputs.currentTag }}
push_git_tag: true
push_image_and_stages: true
dockerfile: Dockerfile
registry: ghcr.io
build_extra_args: "--compress=true"
Original file line number Diff line number Diff line change
Expand Up @@ -10,10 +10,11 @@ on:
options:
- boilerplate
- flyteidl
- flyteconsole
jobs:
trigger-upgrade:
name: ${{ github.event.inputs.component }} Upgrade
if: ${{ github.event.inputs.component == "boilerplate" }}
if: ${{ github.event.inputs.component == 'boilerplate' }}
uses: flyteorg/flytetools/.github/workflows/flyte_automation.yml@master
with:
component: ${{ github.event.inputs.component }}
Expand All @@ -23,7 +24,7 @@ jobs:
upgrade_flyteidl:
name: Upgrade Flyteidl
runs-on: ubuntu-latest
if: ${{ github.event.inputs.component == "boilerplate" }}
if: ${{ github.event.inputs.component == 'flyteidl' }}
steps:
- uses: actions/checkout@v2
with:
Expand All @@ -47,7 +48,7 @@ jobs:
signoff: true
branch: flyte-bot-update-flyteidl
delete-branch: true
title: 'Update Flyteidl version'
title: "Update Flyteidl version"
body: |
Update Flyteidl version
- Auto-generated by [flyte-bot]
Expand All @@ -57,3 +58,4 @@ jobs:
owners
maintainers
draft: false

9 changes: 1 addition & 8 deletions .releaserc.json
Original file line number Diff line number Diff line change
Expand Up @@ -9,13 +9,6 @@
}
],
["@semantic-release/npm", { "npmPublish": false }],
"@semantic-release/github",
[
"@semantic-release/git",
{
"assets": ["package.json", "CHANGELOG.md"],
"message": "chore(release): Release ${nextRelease.version} [skip ci]\n\n${nextRelease.notes}"
}
]
"@semantic-release/github"
]
}
16 changes: 16 additions & 0 deletions .vscode/launch.json
Original file line number Diff line number Diff line change
@@ -0,0 +1,16 @@
{
// Use IntelliSense to learn about possible attributes.
// Hover to view descriptions of existing attributes.
// For more information, visit: https://go.microsoft.com/fwlink/?linkid=830387
"version": "0.2.0",
"configurations": [
{
"name": "launch generator",
"type": "node",
"request": "launch",
"skipFiles": ["<node_internals>/**"],
"program": "${workspaceFolder}/script/generator/src/index.js",
"console": "integratedTerminal"
}
]
}
27 changes: 26 additions & 1 deletion CONTRIBUTING.md
Original file line number Diff line number Diff line change
Expand Up @@ -22,6 +22,30 @@ a single module, you can specify that one specifically
(ex. `localStorage.debug = 'flyte:adminEntity'` to only see decoded Flyte
Admin API requests).

## Generate new package

To add a new package use a script

```bash
yarn generate:package
```

After new package is generated, you will need to update some values to be able to use it with other packages.
For example in case if package plan to be used in `console` app

Ensure to add proper webpack alias path resolutions into:
* ./storybook/main.js - as `'@flyteconsole/flyte-api': path.resolve(__dirname, '../packages/plugins/flyte-api/src’),`
* packages/zapp/console/webpack.common.config.ts to alias section - as `'@flyteconsole/flyte-api': path.resolve(__dirname, '../packages/plugins/flyte-api/src’),`

To add child package usage to other package, in parent package ->
* Add `{ "path": “../../${type}/${package-name}" }` to tsconfig.json
* Add `{ "path": “../../${type}/${package-name}/tsconfig.build.json" }` to tsconfig.build.json (if exists)
- Then you can import your changes as `import { getLoginUrl } from '@flyteconsole/flyte-api’;`

> If you see `yarn lint` package not defined issues update `.\eslintrc.js` by adding your package to
'import/core-modules': ['@clients/locale', '@clients/primitives', '@clients/theme'],


## Storybook

This project has support for [Storybook](https://storybook.js.org/).
Expand Down Expand Up @@ -115,9 +139,10 @@ brew install asdf
brew install vips
```

- Add Yarn plugin to asdf, to manage yarn versions
- Add Yarn and NodeJs plugins to asdf, to manage versions

```bash
asdf plugin add nodejs https://github.com/asdf-vm/asdf-nodejs.git
asdf plugin-add yarn https://github.com/twuni/asdf-yarn.git
brew install gpg
```
Expand Down
7 changes: 7 additions & 0 deletions Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -42,3 +42,10 @@ test_unit_codecov:
.PHONY: generate_ssl
generate_ssl:
./script/generate_ssl.sh

PLACEHOLDER_NPM := \"version\": \"0.0.0-develop\"

.PHONY: update_npmversion
update_npmversion:
grep "$(PLACEHOLDER_NPM)" "packages/zapp/console/package.json"
sed -i "s/$(PLACEHOLDER_NPM)/\"version\": \"${VERSION}\"/g" "packages/zapp/console/package.json"
30 changes: 28 additions & 2 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -54,10 +54,28 @@ For help with installing dependencies look into

3. Start the server (uses localhost:3000)

`yarn start`
`bash yarn start `

4. Explore your local copy at `http://localhost:3000`

### Note: Python errors with OSX

Recently OSX (12.3) removed python 2.7 from default installation and this can cause build errors for some users depending on it's setup. In this repository you can experience `env: python: No such file or directory` error from gyp ([node-gyp](https://github.com/nodejs/node-gyp)).
The easiest way to fix it:
- Install the XCode Command Line Tools standalone by running `xcode-select --install` in the terminal
OR
```bash
brew install python # install python with brew
which python # check if python path is properly defined
# if path not defined
where python3
# Take the version and location from above and run this command (replacing `/usr/bin/python3` with the location of your python instalation). This will symlink python to python3
ln -s /usr/bin/python3 /usr/local/bin/python
```
### Environment Variables
- `ADMIN_API_URL` (default: [window.location.origin](https://developer.mozilla.org/en-US/docs/Web/API/Window/location>))
Expand All @@ -80,11 +98,15 @@ For help with installing dependencies look into
- `BASE_URL` (default: `undefined`)
This allows running the console at a prefix on the target host. This is
This setting allows running the console at a prefix on the target host. This is
necessary when hosting the API and console on the same domain (with prefixes of
`/api/v1` and `/console` for example). For local development, this is
usually not needed, so the default behavior is to run without a prefix.
- `FLYTE_NAVIGATION` (default: `undefined`)
UI related. Allows you to change colors of the navigation bar and add links
to other internal pages or external sites. **[More info](packages/zapp/console/src/components/Navigation/Readme.md)**
### Running from docker image as localhost
To run flyteconsole directly from your docker image as localhost you must set a
Expand Down Expand Up @@ -116,6 +138,10 @@ and start the NodeJS server on the default port (3000). All requests to the Node
will be stalled until the bundles have finished. The application will be accessible
at http://localhost:3000 (if using the default port).
### 🎱 Using items in your own application
- Authorize your app to call flyte admin api. **[More info](packages/plugins/flyte-api/README.md)**
## 🛠 Development
For continious development we are using:
Expand Down
1 change: 0 additions & 1 deletion boilerplate/update.cfg
Original file line number Diff line number Diff line change
@@ -1,4 +1,3 @@
flyte/docker_build
flyte/pull_request_template
flyte/welcome_bot
flyte/code_of_conduct
14 changes: 9 additions & 5 deletions package.json
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
{
"name": "flyteconsole",
"version": "1.1.0-rc1",
"version": "1.3.5",
"private": true,
"description": "The web UI for the Flyte platform",
"repository": {
Expand All @@ -12,14 +12,16 @@
"packages/basics/**",
"packages/composites/**",
"packages/plugins/**",
"packages/zapp/**"
"packages/zapp/**",
"script/generator/**"
],
"scripts": {
"clean": "yarn workspace @flyteconsole/client-app clean",
"start": "yarn workspace @flyteconsole/client-app start",
"start:prod": "yarn workspace @flyteconsole/client-app start:prod",
"build:prod": "yarn workspace @flyteconsole/client-app build:prod",
"build:storybook": "build-storybook",
"generate:package": "yarn workspace @flyteconsole/generator start",
"lint": "eslint . --ext .js,.jsx,.ts,.tsx",
"storybook": "start-storybook -p 6006",
"test": "NODE_ENV=test jest",
Expand All @@ -45,7 +47,9 @@
"git add"
]
},
"dependencies": {},
"dependencies": {
"@flyteorg/flyteidl": "1.1.16"
},
"devDependencies": {
"@storybook/addon-actions": "^6.4.19",
"@storybook/addon-essentials": "^6.4.19",
Expand All @@ -58,9 +62,9 @@
"@testing-library/jest-dom": "^5.5.0",
"@testing-library/react": "^10.0.3",
"@testing-library/react-hooks": "^7.0.2",
"ts-jest": "^26.3.0",
"jest": "^26.0.0",
"react-hot-loader": "^4.1.2"
"react-hot-loader": "^4.1.2",
"ts-jest": "^26.3.0"
},
"resolutions": {
"@babel/cli": "~7.16.0",
Expand Down
24 changes: 24 additions & 0 deletions packages/composites/ui-atoms/src/Icons/MapCacheIcon/index.tsx
Original file line number Diff line number Diff line change
@@ -0,0 +1,24 @@
import * as React from 'react';
import SvgIcon, { SvgIconProps } from '@material-ui/core/SvgIcon';

export const MapCacheIcon = React.forwardRef<SVGSVGElement, SvgIconProps>((props, ref) => {
return (
<SvgIcon {...props} ref={ref} viewBox="0 0 17 17">
<g clipPath="url(#clip0_6712_89419)">
<path
d="M5.68615 2.99228L6.91566 7.15285L4.89438 6.0537C4.31136 7.12586 4.17812 8.3857 4.52399 9.55609C4.86986 10.7265 5.45103 11.4847 6.52318 12.0677C7.19695 12.4341 8.24054 12.5388 8.96464 12.5397L9.41341 14.0583C8.25879 14.1378 6.70623 14.0476 5.68964 13.4944C4.2601 12.717 3.51417 11.5513 3.05301 9.99079C2.59185 8.43027 2.76949 6.75048 3.54686 5.32094L1.52558 4.22179L5.68615 2.99228Z"
fill="#666666"
/>
<path
d="M9.90614 2.76737C11.152 3.02647 12.2756 3.69438 13.0985 4.66504C13.9214 5.6357 14.3964 6.85346 14.4481 8.12494C14.4998 9.39642 14.1252 10.6487 13.3839 11.683C12.6425 12.7173 11.5768 13.4742 10.3561 13.8336L8.74374 8.35689L9.90614 2.76737Z"
fill="#DDDDE5"
/>
</g>
<defs>
<clipPath id="clip0_6712_89419">
<rect width="16" height="16" fill="white" transform="translate(0.406982 0.356445)" />
</clipPath>
</defs>
</SvgIcon>
);
});
39 changes: 39 additions & 0 deletions packages/composites/ui-atoms/src/Icons/MuiLaunchPlanIcon/index.tsx
Original file line number Diff line number Diff line change
@@ -0,0 +1,39 @@
import { makeStyles } from '@material-ui/core/styles';
import { SvgIconProps, Theme } from '@material-ui/core';
import classnames from 'classnames';
import * as React from 'react';

const useStyles = makeStyles((theme: Theme) => ({
svg: {
marginTop: 0,
marginRight: theme.spacing(2),
display: 'inline-block',
fontSize: '1.5rem',
transition: 'fill 200ms cubic-bezier(0.4, 0, 0.2, 1) 0ms',
flexShrink: 0,
userSelect: 'none',
color: '#666666',
},
}));

export const MuiLaunchPlanIcon = (props: SvgIconProps): JSX.Element => {
const { fill, className, width = '1em', height = '1em', fontSize } = props;
const styles = useStyles();
return (
<svg
className={classnames(styles.svg, className)}
width={width}
height={height}
viewBox="0 0 16 16"
xmlns="http://www.w3.org/2000/svg"
fontSize={fontSize}
>
<path
fillRule="evenodd"
clipRule="evenodd"
d="M2 15V2C2 1.44772 2.44772 1 3 1H12.7391C13.2914 1 13.7391 1.44772 13.7391 2V11.4421H9.82593H9.32593V11.9421V16H3C2.44772 16 2 15.5523 2 15ZM10.3259 12.4421H13.384L10.3259 15.5002V12.4421ZM5.1307 5.93466H11.0003V4.93466H5.1307V5.93466ZM11.0004 8.83351H5.13079V7.83351H11.0004V8.83351ZM5.13079 11.732H8.02934V10.732H5.13079V11.732Z"
fill={fill || 'currentColor'}
/>
</svg>
);
};
2 changes: 2 additions & 0 deletions packages/composites/ui-atoms/src/Icons/index.tsx
Original file line number Diff line number Diff line change
@@ -1,3 +1,5 @@
export { FlyteLogo } from './FlyteLogo';
export { InfoIcon } from './InfoIcon';
export { RerunIcon } from './RerunIcon';
export { MapCacheIcon } from './MapCacheIcon';
export { MuiLaunchPlanIcon } from './MuiLaunchPlanIcon';
3 changes: 3 additions & 0 deletions packages/zapp/console/env.js
Original file line number Diff line number Diff line change
Expand Up @@ -39,6 +39,8 @@ const STATUS_URL = process.env.STATUS_URL;
const ENABLE_GA = process.env.ENABLE_GA || false;
const GA_TRACKING_ID = process.env.GA_TRACKING_ID || 'G-0QW4DJWJ20';

const FLYTE_NAVIGATION = process.env.FLYTE_NAVIGATION || '';

module.exports = {
ADMIN_API_URL,
ADMIN_API_USE_SSL,
Expand All @@ -58,5 +60,6 @@ module.exports = {
GA_TRACKING_ID,
NODE_ENV,
STATUS_URL,
FLYTE_NAVIGATION,
},
};
Loading

0 comments on commit f8cb277

Please sign in to comment.