Skip to content
This repository has been archived by the owner on Mar 25, 2021. It is now read-only.

Implement a continuous integration pipeline #35

Closed
JQrdan opened this issue Oct 7, 2020 · 0 comments · Fixed by #45
Closed

Implement a continuous integration pipeline #35

JQrdan opened this issue Oct 7, 2020 · 0 comments · Fixed by #45
Assignees
@JQrdan
Projects
UI Development

Comments

@JQrdan
Copy link

JQrdan commented Oct 7, 2020

@JQrdan JQrdan created this issue from a note in UI Development (Backlog) Oct 7, 2020
@JQrdan JQrdan mentioned this issue Oct 7, 2020
Merged
@JQrdan JQrdan self-assigned this Oct 8, 2020
@matthew-chirgwin matthew-chirgwin moved this from Backlog to In progress in UI Development Oct 13, 2020
@JQrdan JQrdan mentioned this issue Oct 21, 2020
Merged
@matthew-chirgwin matthew-chirgwin moved this from In progress to Review in progress in UI Development Oct 22, 2020
UI Development automation moved this from Review in progress to Done Oct 27, 2020
matthew-chirgwin pushed a commit to matthew-chirgwin/strimzi-ui that referenced this issue Oct 27, 2020
feat: add multi report processing to action
44cf74e 
- updates bundle action/workflows to support multiple report files

Contributes to: strimzi#35

Signed-off-by: Matthew Chirgwin <chirmatt@uk.ibm.com>
matthew-chirgwin pushed a commit to matthew-chirgwin/strimzi-ui that referenced this issue Oct 27, 2020
refactor: add multi report processing to action, rebase updates
88a3e5d 
- updates bundle action/workflows to support multiple report files
- adds simple e2e test
- rebase updates

Contributes to: strimzi#35

Signed-off-by: Matthew Chirgwin <chirmatt@uk.ibm.com>
matthew-chirgwin pushed a commit to matthew-chirgwin/strimzi-ui that referenced this issue Oct 27, 2020
refactor: add multi report processing to action, rebase updates
5b13fca 
- updates bundle action/workflows to support multiple report files
- adds simple e2e test
- rebase updates

Contributes to: strimzi#35

Signed-off-by: Matthew Chirgwin <chirmatt@uk.ibm.com>
matthew-chirgwin pushed a commit to matthew-chirgwin/strimzi-ui that referenced this issue Oct 27, 2020
refactor: add multi report processing to action, rebase updates
027077f 
- updates bundle action/workflows to support multiple report files
- adds simple e2e test
- rebase updates
- change all dcokerfiles to use lts base images
(https://stackoverflow.com/q/64494474)

Contributes to: strimzi#35

Signed-off-by: Matthew Chirgwin <chirmatt@uk.ibm.com>
matthew-chirgwin pushed a commit to matthew-chirgwin/strimzi-ui that referenced this issue Oct 27, 2020
refactor: add multi report processing to action, rebase updates
dd6fc0a 
- updates bundle action/workflows to support multiple report files/no
files
- adds simple e2e test
- rebase updates
- change all dcokerfiles to use lts base images
(https://stackoverflow.com/q/64494474)

Contributes to: strimzi#35

Signed-off-by: Matthew Chirgwin <chirmatt@uk.ibm.com>
matthew-chirgwin pushed a commit to matthew-chirgwin/strimzi-ui that referenced this issue Oct 27, 2020
refactor: add multi report processing to action, rebase updates
bac58be 
- updates bundle action/workflows to support multiple report files/no
files
- updates commenter action to expect multiple bundles
- makes jest exit in node workflow (in case handles are left jest would
run forever)
- updates main server steps to work on node 14
- adds simple e2e test
- rebase updates
- change all dcokerfiles to use lts base images
(https://stackoverflow.com/q/64494474)

Contributes to: strimzi#35

Signed-off-by: Matthew Chirgwin <chirmatt@uk.ibm.com>
matthew-chirgwin pushed a commit to matthew-chirgwin/strimzi-ui that referenced this issue Oct 28, 2020
refactor: add multi report processing to action, rebase updates
20a763b 
- updates bundle action/workflows to support multiple report files/no
file cases
- updates commenter action to expect multiple bundles
- makes jest exit in node workflow (in case handles are left jest would
run forever)
- removes 'main' server tests. These do not run reliably in CI and
require reimplementation (to follow in strimzi#13)
- adds simple e2e test for client
- rebase updates
- change all dcokerfiles to use lts base images
(https://stackoverflow.com/q/64494474), and update of lock file

Contributes to: strimzi#35

Signed-off-by: Matthew Chirgwin <chirmatt@uk.ibm.com>
matthew-chirgwin added a commit that referenced this issue Oct 29, 2020
@matthew-chirgwin
feat: add ui server implementation (#40)
ae571ee 
* feat: add ui server implementation

- adds TS config (derived from #29)
- implements server build
- adds placeholder dockerfile to use the build
- implements server + adds helmet
- implements api module
- adds other modules boilerplate
- updates linting config for ts
- updates doc around build/commands to run
- update doc to reference ts files
- changes alias approach to use tsconfig paths
- dependencies added to use this

Contributes to: #36

Signed-off-by: Matthew Chirgwin <chirmatt@uk.ibm.com>

* feat: complete server implementation

- allow config files to be js or json
- generate certs for development w/ helpers
- add http/https server support

Contributes to: #36

Signed-off-by: Matthew Chirgwin <chirmatt@uk.ibm.com>

* refactor: require config logic

- change require logic to account for webpack
- fixes issues around e2e tests not running
- changes test script names to reflect what the are running (jest wont
be just unit tests)

Contributes to: #36

Signed-off-by: Matthew Chirgwin <chirmatt@uk.ibm.com>

* feat: add tests for server

- adds tests for server
- files at the root level of the server not tested - ie the server
entry code and filesystem pieces. Looking into options
- turns off coverage collection for root server directory, and
placeholder index.ts in client bootstrap
- fixes few test config issues found

Contributes to: #36

Signed-off-by: Matthew Chirgwin <chirmatt@uk.ibm.com>

* fix: add stylelint config removed in rebase

- reverts stylelint rebase configuration removal

Contributes to: #36

Signed-off-by: Matthew Chirgwin <chirmatt@uk.ibm.com>

* feat: add additional tests

- adds tests to cover the bootstrap of the UI server

Contributes to: #36

Signed-off-by: Matthew Chirgwin <chirmatt@uk.ibm.com>

* refactor: address review comments

- restore deleted file from rebase
- addresses review comments
- fixes e2e config issue

Contributes to: #36

Signed-off-by: Matthew Chirgwin <chirmatt@uk.ibm.com>

* fix: make sure context root preserved on proxy

- makes sure context root can be preserved on proxy via new
configuration option
- test to cover

Contributes to: #36

Signed-off-by: Matthew Chirgwin <chirmatt@uk.ibm.com>

* refactor: add multi report processing to action, rebase updates

- updates bundle action/workflows to support multiple report files/no
file cases
- updates commenter action to expect multiple bundles
- makes jest exit in node workflow (in case handles are left jest would
run forever)
- removes 'main' server tests. These do not run reliably in CI and
require reimplementation (to follow in #13)
- adds simple e2e test for client
- rebase updates
- change all dcokerfiles to use lts base images
(https://stackoverflow.com/q/64494474), and update of lock file

Contributes to: #35

Signed-off-by: Matthew Chirgwin <chirmatt@uk.ibm.com>

Co-authored-by: Matthew Chirgwin <CHIRMATT@uk.ibm.com>
@matthew-chirgwin matthew-chirgwin mentioned this issue Oct 29, 2020
Closed
suyash-naithani pushed a commit to suyash-naithani/strimzi-ui that referenced this issue Feb 1, 2021
@dlabaj
Merge pull request strimzi#35 from Redhat-AppServices/openapi-rest
6315fc2 
Added REST api. Signed-off-by: Donald Labaj dlabaj@redhat.com
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@JQrdan JQrdan

Labels
None yet
Projects
No open projects
UI Development
  
Done
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

feat: add a continuous integration pipeline JQrdan/strimzi-ui
1 participant
@JQrdan