Skip to content
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

CRUD web apps: i18n #5880

Merged
merged 21 commits into from Jun 21, 2021
Merged

Conversation

wg102
Copy link
Contributor

@wg102 wg102 commented May 3, 2021

This PR add i18n to jupyter+volumes using Angular ngx translate

Related to #5858
Closes StatCan#56

To test the feature:

  • Build the environment as usual
  • Modify your browser's language. The code detects the top-most language if multiple are set.
  • Languages supported are : English and French.
  • It defaults to English if neither of those are the top ones.

@aws-kf-ci-bot
Copy link

Hi @wg102. Thanks for your PR.

I'm waiting for a kubeflow member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.

Once the patch is verified, the new status will be reflected by the ok-to-test label.

I understand the commands that are listed here.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@google-cla
Copy link

google-cla bot commented May 3, 2021

Thanks for your pull request. It looks like this may be your first contribution to a Google open source project (if not, look below for help). Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

📝 Please visit https://cla.developers.google.com/ to sign.

Once you've signed (or fixed any issues), please reply here with @googlebot I signed it! and we'll verify it.


What to do if you already signed the CLA

Individual signers
Corporate signers

ℹ️ Googlers: Go here for more info.

@google-cla
Copy link

google-cla bot commented May 6, 2021

All (the pull request submitter and all commit authors) CLAs are signed, but one or more commits were authored or co-authored by someone other than the pull request submitter.

We need to confirm that all authors are ok with their commits being contributed to this project. Please have them confirm that by leaving a comment that contains only @googlebot I consent. in this pull request.

Note to project maintainer: There may be cases where the author cannot leave a comment, or the comment is not properly detected as consent. In those cases, you can manually confirm consent of the commit author(s), and set the cla label to yes (if enabled on your project).

ℹ️ Googlers: Go here for more info.

@saffaalvi
Copy link
Contributor

@googlebot I consent.

@google-cla google-cla bot added cla: yes and removed cla: no labels May 6, 2021
@wg102
Copy link
Contributor Author

wg102 commented May 12, 2021

Will need to update and rebase once angular version is upgraded (see #5805).

@wg102 wg102 marked this pull request as ready for review May 12, 2021 17:30
@wg102 wg102 changed the title CRUD web apps: i18n of jupyter CRUD web apps: i18n May 12, 2021
@thesuperzapper
Copy link
Member

@wg102 love this work!

If possible, to help future developers implement/update localisations, can you add some docs to the README for each of the apps:

The main things we need in those README's are:

  • Information about where to find the localisation JSON files (and how to update them)
  • Information (generally) about how localisation has been implemented

@wg102
Copy link
Contributor Author

wg102 commented Jun 3, 2021

@thesuperzapper I will add those.
I was actually wondering if the en.json (and any other languages) should be present in all of the apps, or once in the common, and get copied to the other ones. Thoughts?

@davidspek
Copy link
Contributor

@wg102 That's a good question. If there are a lot of common values between the web apps it does make sense to place then JSON files in common. However, it also might confuse or get forgotten by developers as they will need to change files in a different code base than the one they are working on.

@davidspek
Copy link
Contributor

/ok-to-test

@davidspek
Copy link
Contributor

davidspek commented Jun 4, 2021

Here is the log output of the failed tests:

Jupyter Web App (frontend):

[warn] src/app/pages/form/form-default/form-affinity-tolerations/form-affinity-tolerations.component.html
[warn] src/app/pages/form/form-default/form-configurations/form-configurations.component.html
[warn] src/app/pages/form/form-default/form-cpu-ram/form-cpu-ram.component.html
[warn] src/app/pages/form/form-default/form-data-volumes/form-data-volumes.component.html
[warn] src/app/pages/form/form-default/form-default.component.ts
[warn] src/app/pages/form/form-default/form-default.module.ts
[warn] src/app/pages/form/form-default/form-gpus/form-gpus.component.ts
[warn] src/app/pages/form/form-default/form-image/form-image.component.html
[warn] src/app/pages/form/form-default/form-workspace-volume/form-workspace-volume.component.ts
[warn] src/app/pages/form/form-rok/form-rok.component.ts
[warn] src/app/pages/index/index-default/index-default.component.ts
[warn] Code style issues found in the above file(s). Forgot to run Prettier?
-----------------------------------------------
Please run `npm run format:write` to format your code.
-----------------------------------------------
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR! frontend@0.0.0 format:check: `prettier --check 'src/**/*.{js,ts,html,scss,css}' || node scripts/check-format-error.js`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the frontend@0.0.0 format:check script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.
npm ERR! A complete log of this run can be found in:
npm ERR!     /root/.npm/_logs/2021-06-04T10_14_14_347Z-debug.log

Volumes Web App (backend):

W
Collecting flake8
  Downloading flake8-3.9.2-py2.py3-none-any.whl (73 kB)
Collecting importlib-metadata
  Downloading importlib_metadata-4.5.0-py3-none-any.whl (17 kB)
Collecting pycodestyle<2.8.0,>=2.7.0
  Downloading pycodestyle-2.7.0-py2.py3-none-any.whl (41 kB)
Collecting mccabe<0.7.0,>=0.6.0
  Downloading mccabe-0.6.1-py2.py3-none-any.whl (8.6 kB)
Collecting pyflakes<2.4.0,>=2.3.0
  Downloading pyflakes-2.3.1-py2.py3-none-any.whl (68 kB)
Collecting typing-extensions>=3.6.4
  Downloading typing_extensions-3.10.0.0-py3-none-any.whl (26 kB)
Collecting zipp>=0.5
  Downloading zipp-3.4.1-py3-none-any.whl (5.2 kB)
Installing collected packages: zipp, typing-extensions, pyflakes, pycodestyle, mccabe, importlib-metadata, flake8
Successfully installed flake8-3.9.2 importlib-metadata-4.5.0 mccabe-0.6.1 pycodestyle-2.7.0 pyflakes-2.3.1 typing-extensions-3.10.0.0 zipp-3.4
.1
WARNING: Running pip as root will break packages and permissions. You should install packages reliably by using venv: https://pip.pypa.io/warn
ings/venv
./apps/common/status.py:10:80: E501 line too long (134 > 79 characters)
./apps/common/status.py:14:80: E501 line too long (131 > 79 characters)
./apps/common/status.py:27:80: E501 line too long (138 > 79 characters)
./apps/common/status.py:48:80: E501 line too long (122 > 79 characters)

Common Frontend:

        error properties: Object({ ngTempTokenPath: null, ngTokenPath: [ 'TranslatePipe', Function ], ngDebugContext: DebugContext_({ view: Ob
ject({ def: Object({ factory: Function, nodeFlags: 914344979, rootNodeFlags: 1, nodeMatchedQueries: 30, flags: 0, nodes: [ Object({ nodeIndex:
 0, parent: null, renderParent: null, bindingIndex: 0, outputIndex: 0, checkIndex: 0, flags: 1, childFlags: 914344979, directChildFlags: 33554
433, childMatchedQueries: 30, matchedQueries: Object({  }), matchedQueryIds: 0, references: Object({  }), ngContentIndex: null, childCount: 26
, bindings: [  ], bindingFlags: 0, outputs: [  ], element: Object({ ns: '', name: 'div', attrs: [ Array ], template: null, componentProvider:
null, componentView: null, componentRendererType: null, publicProviders: null({  }), allProviders: null({  }), handleEvent: Function }), provi
der: null, text: null, query: null, ngContent: null }), Object({ nodeIndex: 1, parent: Object({ nodeIndex: 0, parent: null, renderParent: null
, bindingIndex: 0, outputIndex:  ...
            at <Jasmine>
            at NullInjector.get (http://localhost:9876/mnt/test-data-volume/kubeflow-kubeflow-presubmit-common-ui-5880-04ebc5e-4976-48a7/src/k
ubeflow/kubeflow/components/crud-web-apps/common/frontend/kubeflow-common-lib/node_modules/@angular/core/fesm2015/core.js:855:1)
            at resolveToken (http://localhost:9876/mnt/test-data-volume/kubeflow-kubeflow-presubmit-common-ui-5880-04ebc5e-4976-48a7/src/kubef
low/kubeflow/components/crud-web-apps/common/frontend/kubeflow-common-lib/node_modules/@angular/core/fesm2015/core.js:17514:1)
            at tryResolveToken (http://localhost:9876/mnt/test-data-volume/kubeflow-kubeflow-presubmit-common-ui-5880-04ebc5e-4976-48a7/src/ku
beflow/kubeflow/components/crud-web-apps/common/frontend/kubeflow-common-lib/node_modules/@angular/core/fesm2015/core.js:17440:1)
            at StaticInjector.get (http://localhost:9876/mnt/test-data-volume/kubeflow-kubeflow-presubmit-common-ui-5880-04ebc5e-4976-48a7/src
/kubeflow/kubeflow/components/crud-web-apps/common/frontend/kubeflow-common-lib/node_modules/@angular/core/fesm2015/core.js:17266:1)
            at resolveToken (http://localhost:9876/mnt/test-data-volume/kubeflow-kubeflow-presubmit-common-ui-5880-04ebc5e-4976-48a7/src/kubef
low/kubeflow/components/crud-web-apps/common/frontend/kubeflow-common-lib/node_modules/@angular/core/fesm2015/core.js:17514:1)
            at tryResolveToken (http://localhost:9876/mnt/test-data-volume/kubeflow-kubeflow-presubmit-common-ui-5880-04ebc5e-4976-48a7/src/ku
beflow/kubeflow/components/crud-web-apps/common/frontend/kubeflow-common-lib/node_modules/@angular/core/fesm2015/core.js:17440:1)
            at StaticInjector.get (http://localhost:9876/mnt/test-data-volume/kubeflow-kubeflow-presubmit-common-ui-5880-04ebc5e-4976-48a7/src
/kubeflow/kubeflow/components/crud-web-apps/common/frontend/kubeflow-common-lib/node_modules/@angular/core/fesm2015/core.js:17266:1)
            at resolveNgModuleDep (http://localhost:9876/mnt/test-data-volume/kubeflow-kubeflow-presubmit-common-ui-5880-04ebc5e-4976-48a7/src
/kubeflow/kubeflow/components/crud-web-apps/common/frontend/kubeflow-common-lib/node_modules/@angular/core/fesm2015/core.js:30393:1)
..........
HeadlessChrome 88.0.4324 (Linux 0.0.0): Executed 34 of 34 (11 FAILED) (3.524 secs / 3.388 secs)
TOTAL: 11 FAILED, 23 SUCCESS
TOTAL: 11 FAILED, 23 SUCCESS
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR! angular-frontend-kubeflow@0.0.0 test-ci: `ng test --no-watch --no-progress --browsers=ChromeHeadlessCI`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the angular-frontend-kubeflow@0.0.0 test-ci script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.
npm ERR! A complete log of this run can be found in:
npm ERR!     /home/blessuser/.npm/_logs/2021-06-04T10_13_48_146Z-debug.log


**When a language is added:**
- Copy the en.json file and rename is to the language you want to add. As it currently is, the culture should not be included.
- Change the values to the translated ones
Copy link
Contributor

Choose a reason for hiding this comment

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

It looks like this section is exactly the same as the one above. Was it meant to be different, or is it a copy/paste error?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Oh, yes copy/paste error... thanks for noticing!

@davidspek
Copy link
Contributor

/test kubeflow-kubeflow-presubmit

Copy link
Contributor

@davidspek davidspek left a comment

Choose a reason for hiding this comment

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

Awesome work everybody! I just have a few small questions, but everything looks good. I've tested the web apps and they all perform as expected and follow the language of the browser.

/lgtm

If one of the @kubeflow/wg-notebooks-leads can look through this we can hopefully get this approved soon so I can proceed with the Angular Update PR.


@Component({
selector: 'app-root',
templateUrl: './app.component.html',
styleUrls: ['./app.component.scss'],
})
export class AppComponent {
constructor(private translate: TranslateService) {
const currentLanguage = this.translate.getBrowserLang();
const lang = currentLanguage.match(/en|fr/) ? currentLanguage : 'en';
Copy link
Contributor

Choose a reason for hiding this comment

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

Will new languages need to be added to the currentLanguage.match(/en|fr/) section once the JSON files are added?

Copy link
Contributor

Choose a reason for hiding this comment

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

That is correct. Once a language's JSON file is added the corresponding app.component.ts will need to be updated as well

@davidspek
Copy link
Contributor

@Jose-Matsuda Thanks for the clarification.

/lgtm

@thesuperzapper
Copy link
Member

@wg102 after testing this locally, and reviewing the code, it looks good to go, we are incredibly grateful for your hard work on this!

I would say next steps are to consider a language dropdown box in the Central Dashboard (under some kind of user settings). However, be aware that we are considering rewriting Central Dashboard in Angular (#6000), so doing anything too drastic would be a waste of effort.

/approve

@google-oss-robot
Copy link

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: thesuperzapper

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@google-oss-robot google-oss-robot merged commit 7a674c1 into kubeflow:master Jun 21, 2021
chuckbelisle pushed a commit to StatCan/kubeflow that referenced this pull request Sep 2, 2022
* releasing: Add WG-Notebooks leads to release owners (kubeflow#5777)

Signed-off-by: Yannis Zarkadas <yanniszark@arrikto.com>

* Specify commonLabels for tensorboard-controller (kubeflow#5780)

* Remove jlewi as an approver (kubeflow#5786)

Removing myself as an owner. This should lead to better auto-assignment of code reviews.

* Update notebook server base images (kubeflow#5804)

* Update second layer docker images to new tags and some python deps (kubeflow#5809)

* Update image tags and python packages in pytorch and tensorflow full dockerfiles (kubeflow#5817)

* Add CI format checks for the Jupyter web app (kubeflow#5811)

* jwa(front): Add npm rule for checking the format

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* ci: Add common tasks for format checks

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* ci(jwa): Add format check tasks

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(back): Fix formatting

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(front): Fix formatting

Run `npm run format:write` on frontend

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(front): Include prettier in package.json

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(docker): Don't copy node_modules in Dockerfile

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* web-apps: Add global dockerignore file

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa: Remove unused dockerignore file

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(make): Don't include dockerignore and cleanup

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* initial notebook server images README.md (kubeflow#5818)

* tensorboard-controller: Fix scheduling unbound PVCs (kubeflow#5819)

When the TB controller attempts to schedule a RWO PVC it checks its
accessModes in the PVC status. The controller panics if the list is
empty.

This commit adds a check to ensure the list is not empty.

Signed-off-by: Ilias Katsakioris <elikatsis@arrikto.com>

* Fix profile-controller CRD pruning issue (kubeflow#5822)

* api: Add marker for preserving unknown fields in Plugins

Signed-off-by: Yannis Zarkadas <yanniszark@arrikto.com>

* manifests: Regenerate manifests

Signed-off-by: Yannis Zarkadas <yanniszark@arrikto.com>

* Remove Logo Trademarks from the Jupyter Web App and make logos configurable (kubeflow#5823)

* rebase: Make logos configurable in configmap and remove trademark references

Rebased to remove the changes to the package-lock.json

* review: add suggested changes and add image group section to README

* Make notebook limits configurable with a multiplication factor (kubeflow#5815)

* Make notebook limits configurable with a multiplication factor

* Make limits configurable under advanced section

* run prettier to format frontend code

* fix formatting and add rounding in backend

* Return error if limit is smaller than request

* Allow disabling limitFactor by setting it to none

* review: remove camelCase in python backend

* fix: update spawner_ui_config.yaml in manifests directory

* review: fix setting limits backend

* review: remove unnecessary check from backend

* notebook-servers: Update JupyterLab and add Git Extension (kubeflow#5846)

* chore: Add comment to reference RStudio license. (kubeflow#5884)

* CI/CD: add kustomize build tests (kubeflow#5919)

* Add CI format checks for the Volumes web app (kubeflow#5820)

* vwa(front): Add npm script to check the formatting

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* vwa(front): Update the package-lock.json

Run `npm install` to bring the package-lock.json up to date

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* vwa(front): Fix formatting

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* vwa(back): Fix formatting

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* vwa(docker): Remove unused dockerignore file

We have created a global dockerignore file for all the web apps in the
parent dir.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* vwa(docker): Don't copy node_modules in dockerfile

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* vwa(make): Don't include dockerignore

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* ci(vwa): Add format check tasks

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(docker): Copy only necessary files for build

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* review: Use docker context instead of cd ..

Don't use a `cd ..` and copy dockerignore files back and forth. Instead
we should use the Docker context and the global dockerignore file we
have for all the web apps.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* tensorboard-controller: fix binding issue (kubeflow#5925)

* JWA: Don't override assets with logos ConfigMap (kubeflow#5942)

* jwa(front): Add static logos in the app

The app does not contain the logos' svgs in its source code/static
files. This results in the icons to not show when developing locally.

This commit adds the svgs found in the logos ConfigMap to the static
files of the app as well.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(front): Change logos fetch url

Change the URLs of the logos from `static/assets/*` to
`static/assets/logos`.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(manifests): Don't override assets with logos

Mount the ConfigMap under the `static/assets/logos` directory to not
override the contents of the entire assets dir.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* Make it possible to add Namespaced menu items (kubeflow#5871)

* Make it possible to add Namespaced menu items

* Reduce the nesting levels

* FIX The number of gpu must be set as string in Kubernetes/Openshift (kubeflow#5891)

* Update form.py

* Update form.py

* Update form-default.component.ts

* fix whitespace issue

* feat(jupyter): add fonts as assets to service (kubeflow#5691)

* cwa(front): Ignore font files in assets

* feat(jupyter): add fonts as assets to service

* CRUD: fonts in common

* CWA: Remove link to css file

* jwa(front): Remove font assets from jupyter

Co-authored-by: Wendy Gaultier <wvgaultier@gmail.com>

* Update base notebook for Renovate compatibility (kubeflow#5955)

* Update base notebook for Renovate compatibility

* Separate kubectl and S6 arch

* Update jupyter, rstudio and vs code notebook images + allow rstudio in iframe (kubeflow#5961)

* Update jupyter, rstudio and vs code notebook images

* Remove sed and use substring removal

* Make menu bar scrollable again (kubeflow#5964)

* [fix]: Make jupyter-web-app parse workspace volume MountPath (kubeflow#5952)

* [fix]: Make jupyter-web-app parse workspace volume MountPath

- workspace volume path was fixed with "/home/jovyan"
- it should be enable to parse from jupyter-web-app-config's data

* change parsing key correctly

* Remove the f-strings in the Jupyter web app's backend (kubeflow#5680)

* Notebook servers: Add explanation about kernel not connecting (kubeflow#5920)

* fix(profile-controller): KNative probes (kubeflow#5848)

Knative control-plane components need to probe certain paths of Knative
Pods, in order to function correctly. These paths are:
- /healthz
- /metrics
- /ready
- /wait-for-drain

For this reason, we extend the Profile Controller to apply an Istio
AuthorizationPolicy that allows traffic to these HTTP paths for Pods in
user namespaces, as per Knative's instructions:
https://knative.dev/docs/serving/istio-authorization/#allowing-access-from-system-pods-by-paths

This only fixes requests through the public ingress. Cluster-internal
traffic and predictor-transformer use-cases are not supported yet. We
continue discussing these in kubeflow#5965.

Refs kubeflow#5965
Refs kserve/kserve#1558

Signed-off-by: Yurii Komar <subreptivus@gmail.com>

* feat(jupyter): Support hiding image registry/tag (kubeflow#5681)

* feat(jupyter): Support hiding image registry/tag

* feat(jupyter): Support hiding VSCode/RStudio images registry/tag

* rebasing onto upstream master

* fix: formatting with prettier and set default hideRegistry=true

* fix: move hideRegistry and hideTag from spawnerFormDefaults.image

* fix whitespace

* add tooltip for images

Co-authored-by: JessicaBarh <jessicabarhoma@gmail.com>

* Update dockerfiles and make compatible with Renovate (kubeflow#5968)

* Update dockerfiles and make compatible with Renovate

* Set memory for jupyter pytorch to the same as jupyter tensorflow

* Update protobuf

* Remove conda version and use substring expansion

* Update SQLAlchemy

* Update dill

* Create OWNERS file in .github folder for Renovate config + workflows (kubeflow#5983)

* Extend deadline for PyTorch image building (kubeflow#5991)

* Allow user to add/delete labels to user namespace using ConfigMap. Fix kubeflow#5712 (kubeflow#5761)

The profile-controller applies a hardcoded list of labels to Profile
namespaces. Make this list of labels configurable with a ConfigMap. The
new list of steps for the profile-controller is:

1. **(new)** Profile-controller starts and reads the file given by the
   `--namespace-labels-path` CLI flag (defaults to
   `/etc/profile-controller/namespace-labels.yaml`). This file contains
   a YAML dict of label key/values to add to each Profile namespace.
2. Profile-controller creates namespace.
3. Profile-controller merges existing namespace labels with given
   labels. A given label is applied ONLY if it doesn't exist already.
4. **(new)** If a given label value is empty (``), remove the label from
   the namespace if it already exists.

We are using a ConfigMap instead of a plain CLI flag, so that users can
change the list of labels without restarting the profile-controller.

Closes kubeflow#5712

Signed-off-by: Yannis Zarkadas <yanniszark@arrikto.com>

Co-authored-by: James Liu <jamxl@google.com>

Co-authored-by: Yannis Zarkadas <yanniszark@arrikto.com>

* Update last batch of notebook server Dockerfiles (kubeflow#5997)

* Update spawner_ui_config.yaml with new tags (kubeflow#5998)

* Update images in spawner_ui_config.yaml

* Update manifest image tags

* CRUD web apps:  i18n (kubeflow#5880)

* feat(jupyter): add modules for translations

* feat(jupyter): fix path for translations + add GPU translations

* feat(jupyter): add translations

* feat(jupyter): add translations to form-default

* feat(jupyter): add translations to affinity/tolerations

* feat(jupyter): add translations to Image Pull Policy

* feat(jupyter): remove imports from app.module.ts + fix translations

* web-apps(front): add TranslationModule to common for jupyter

* Implement i18n for jupyter frontend
- Including common components needed
- Refactor translation file

* web-apps(front): add i18n to messages from backend
Status tooltip text
Snackbar message

* web-apps(front): volumes and tensorboards i18n ini
Add the i18n feature for the other 2 projects

* feat(volume): add translations for volume

* feat(jupyter): refactor asset file

* Update asset files for all folders

* feat(tensorboards): add i18n
Update all asset files

* cwa: add new translations

* cwa: Add README documentation

* fix: formatting with prettier

* Fix tests

* Fix python format

* remove comment

Co-authored-by: saffaalvi <alvi118@uwindsor.ca>
Co-authored-by: Jose-Matsuda <tongster789@gmail.com>

* Remove deprecated JWA (kubeflow#5959)

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* Sync spawner yaml from source to manifests (kubeflow#5999)

* notebooks: Add additional PGP server for RStudio package validation (kubeflow#6010)

* Change PGP server for RStudio validation to pgp.surfnet.nl

* Add as secondary keyserver

* Add support for ServiceAccountName and AutomountServiceAccountName to admission-webhook (kubeflow#5939)

* WA: Use relative paths for fetching translation files (kubeflow#6034)

* jwa(front): Use relative path for i18n json

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* vwa(front): Use relative path for i18n json

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* twa(front): Use relative path for i18n json

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* Update CRUD web apps from Angular 8 to Angular 12 (kubeflow#6004)

* common: update angular core and cli to 9

ng update @angular/core@9 @angular/cli@9

* common: update material, cdk and cdk-experimental to 9

ng update @angular/cdk@9 @angular/material@9 @angular/cdk-experimental@9

* common: remove entryComponents and replace TestBed.get with TestBed.inject

Edit files manually

* common: make compatible with Angular 9

Edit files manually

* common: update fontawesome to 0.6.0

npm i @fortawesome/angular-fontawesome@0.6.0

* update kubeflow-common project dependencies

* common: add @angular/localize

ng add @angular/localize

* jupyter: update angular core and cli 8

ng update @angular/core@8 @angular/cli@8

* jupyter: update angular core and cli to 9

ng update @angular/core@9 @angular/cli@9

* jupyter: update material, cdk and cdk-experimental to 9

ng update @angular/cdk@9 @angular/material@9 @angular/cdk-experimental@9

* jupyter: remove entryComponents

Edit file manually

* jupyter: update fontawesome to 0.6.0

npm i @fortawesome/angular-fontawesome@0.6.0

* jupyter: add @angular/localize

ng add @angular/localize

* tensorboards: update angular core and cli to 9

ng update @angular/core@9 @angular/cli@9

* tensorboards: update material, cdk and cdk-experimental to 9

ng update @angular/cdk@9 @angular/material@9 @angular/cdk-experimental@9

* tensorboards: remove entryComponents and replace TestBed.get with TestBed.inje…

Edit files manually

* tensorboards: update fontawesome to 0.6.0

npm i @fortawesome/angular-fontawesome@0.6.0

* tensorboards: add @angular/localize

ng add @angular/localize

* Fix tensorboards-web-app run-dev in makefile

Edit file manually

* volumes: update angular core and cli to 9

ng update @angular/core@9 @angular/cli@9

* volumes: update material, cdk and cdk-experimental to 9

ng update @angular/cdk@9 @angular/material@9 @angular/cdk-experimental@9

* volumes: remove entryComponents and replace TestBed.get with TestBe…

Edit files manually

* volumes: update fontawesome to 0.6.0

npm i @fortawesome/angular-fontawesome@0.6.0

* volumes: add @angular/localize

ng add @angular/localize

* common: update angular core and cli to 10

ng update @angular/core@10 @angular/cli@10 --force

* common: update material, cdk and cdk-experimental to 10

ng update @angular/cdk@10 @angular/material@10 @angular/cdk-experimental@10

* update kubeflow-common project dependencies

Edit file manually

* common: update fontawesome to 0.7.0

npm i @fortawesome/angular-fontawesome@0.7.0

* common: fix font location

Edit file manually

* jupyter: update angular-fontawesome to 0.7.0

npm i @fortawesome/angular-fontawesome@0.7.0

* jupyter: update angular core and cli to 10

ng update @angular/core@10 @angular/cli@10

* jupyter: update material, cdk and cdk-experimental to 10

ng update @angular/cdk@10 @angular/material@10 @angular/cdk-experimental@10

* tensorboards: update angular-fontawesome to 0.7.0

npm i @fortawesome/angular-fontawesome@0.7.0

* tensorboards: update angular core and cli to 10

ng update @angular/core@10 @angular/cli@10

* tensorboards: update material, cdk and cdk-experimental to 10

ng update @angular/cdk@10 @angular/material@10 @angular/cdk-experimental@10

* volumes: update angular-fontawesome to 0.7.0

npm i @fortawesome/angular-fontawesome@0.7.0

* volumes: update angular core and cli to 10

ng update @angular/core@10 @angular/cli@10

* volumes: update material, cdk and cdk-experimental to 10

ng update @angular/cdk@10 @angular/material@10 @angular/cdk-experimental@10

* common: update angular-fontawesome to 0.8.2

npm i @fortawesome/angular-fontawesome@0.8.2

* common: update codelyzer to 6.0.2

npm i codelyzer@6.0.2

* common: install @angular-devkit/core@11

npm install --save-dev @angular-devkit/core@11

* common: update angular core and cli to 11

ng update @angular/core@11 @angular/cli@11

* common: update material, cdk and cdk-experimental to 11

ng update @angular/cdk@11 @angular/material@11 @angular/cdk-experimental@11

* update kubeflow-common project dependencies

Edit file manually

* jupyter: update angular-fontawesome to 0.8.2

npm i @fortawesome/angular-fontawesome@0.8.2

* jupyter: update codelyzer to 6.0.2

npm i codelyzer@6.0.2

* jupyter: update angular core and cli to 11

ng update @angular/core@11 @angular/cli@11

* jupyter: update material, cdk and cdk-experimental to 11

ng update @angular/cdk@11 @angular/material@11 @angular/cdk-experimental@11

* tensorboards: update angular-fontawesome to 0.8.2

npm i @fortawesome/angular-fontawesome@0.8.2

* tensorboards: update codelyzer to 6.0.2

npm i codelyzer@6.0.2

* tensorboards: update angular core and cli to 11

ng update @angular/core@11 @angular/cli@11

* tensorboards: update material, cdk and cdk-experimental to 11

ng update @angular/cdk@11 @angular/material@11 @angular/cdk-experimental@11

* volumes: update angular-fontawesome to 0.8.2

npm i @fortawesome/angular-fontawesome@0.8.2

* volumes: update codelyzer to 6.0.2

npm i codelyzer@6.0.2

* volumes: update angular core and cli to 11

ng update @angular/core@11 @angular/cli@11

* volumes: update material, cdk and cdk-experimental to 11

ng update @angular/cdk@11 @angular/material@11 @angular/cdk-experimental@11

* common: Fix CI failure

* jupyter: npm run format:write

npm run format:write

* volumes: npm run format:write

npm run format:write

* tensorboards: manually run prettier

prettier --write 'src/**/*.{js,ts,html,scss,css}'

* common: Fix MockComponent entryComponents for CI

Manually edit file

* common: update angular-fontawesome to 0.9.0

npm i @fortawesome/angular-fontawesome@0.9.0

* common: update angular core and cli to 12

ng update @angular/core@12 @angular/cli@12

* common: update material, cdk and cdk-experimental to 12

ng update @angular/cdk@12 @angular/material@12 @angular/cdk-experimental@12

* update kubeflow-common project dependencies

Edit file manually

* common: fix Intl.DateTimeFormatOptions

Edit file manually. Type of defaultDateOptions and defaultTimeOptions needs to explicitly be set to Intl.DateTimeFormatOptions.

* jupyter: update angular-fontawesome to 0.9.0

npm i @fortawesome/angular-fontawesome@0.9.0

* jupyter: update codelyzer to 6.0.2

npm i codelyzer@6.0.2

* jupyter: update angular core and cli to 12

ng update @angular/core@12 @angular/cli@12

* jupyter: update material, cdk and cdk-experimental to 12

ng update @angular/cdk@12 @angular/material@12 @angular/cdk-experimental@12

* tensorboards: update angular-fontawesome to 0.9.0

npm i @fortawesome/angular-fontawesome@0.9.0

* tensorboards: update angular core and cli to 12

ng update @angular/core@12 @angular/cli@12

* tensorboards: update material, cdk and cdk-experimental to 12

ng update @angular/cdk@12 @angular/material@12 @angular/cdk-experimental@12

* volumes: update angular-fontawesome to 0.9.0

npm i @fortawesome/angular-fontawesome@0.9.0

* volumes: update angular core and cli to 12

ng update @angular/core@12 @angular/cli@12

* volumes: update material, cdk and cdk-experimental to 12

ng update @angular/cdk@12 @angular/material@12 @angular/cdk-experimental@12

* common: update and audit packages

npm update && npm audit fix && npm i lodash-es@4.17.21 && npm i material-icons@0.7.3

* jupyter: update and audit packages

npm update && npm audit fix && npm i material-icons@0.7.3

* tensorboards: update and audit packages

npm update && npm audit fix && npm i material-icons@0.7.3

* volumes: update and audit packages

npm update && npm audit fix && npm i material-icons@0.7.3

* jupyter: npm run format:write

npm run format:write

* lists roadmap in reverse chronological order (kubeflow#6055)

Signed-off-by: Malini Bhandaru <mbhandaru@vmware.com>

* Correct missing predicates in controller watches. Fixes kubeflow#5326 (kubeflow#5873)

Co-authored-by: Filinto Duran <fduran@d2iq.com>

* fix(web-apps): Use Angular's i18n proposed implementation (kubeflow#6065)

* jwa(front): Add i18n for french in angular.json

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(front): Add i18n rules in package.json

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(front): i18n FormImage

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(front): i18n FormAdvancedOptions

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(front): i18n FormAffinityTolerations

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(front): i18n FormConfigurations

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(front): i18n FormCpuRam

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(front): i18n FormDataVolumes

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(front): i18n FormGpus

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(front): i18n FromName

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(front): i18n FormWorkspaceVolume

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(front): i18n Volume

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* web-apps(front): i18n BackendService

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* web-apps(front): Add localize to peerDependencies

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* web-apps(front): Import $localize in library

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* web-apps(front): i18n RokService

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(front): i18n Form

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(front): i18n BackendService

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* web-apps(front): i18n ConfirmDialog

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(front): i18n Index

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* web-apps(front): i18n ResourceTable

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* web-apps(front): i18n SnackBar

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* web-apps(front): i18n Form

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* web-app(front): i18n NameNamespaceSelector

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* web-apps(front): Remove ngx-translate

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(front): i18n App

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(front): Remove ngx-translate from package.json

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(back): Don't parse keys in status

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* vwa(back): Revert backend messages

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(front): Update translation language files

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* web-apps(back): Don't parse keys in status

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* vwa(front): Add i18n for french in angular.json

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* vwa(front): Add i18n rules in package.json

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* vwa(front): i18n Form

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* vwa(front): i18n Index

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* vwa(front): i18n BackendService

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* vwa(front): i18n App

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* vwa(front): Remove ngx-translate from package.json

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* vwa(front): i18n Form

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* vwa(front): Update translation language files

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* twa(back): Revert backend messages

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* twa(front): Add i18n rules in package.json

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* twa(front): Add i18n for french in angular.json

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* twa(front): i18n Index

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* twa(front): i18n BackendService

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* twa(front): i18n App

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* twa(front): Update translation language files

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* twa(front): Remove ngx-translate from package.json

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* wa: Update READMEs

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* web-apps(front): Add localize when testing

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* [CentralDashboard] Add entry for Models web app (kubeflow#6085)

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* Interact with the namespace selector (kubeflow#5995)

* Interact with the namespace selector

* Use namespace given as an argument

* Keep the index of selected item

* fix(jwa): Fix limits calculation when limitFactor is none (kubeflow#6058)

* jwa(front): Don't allow NaN values in limits

The UI should always catch a NaN value and don't add it in the form.
Currently this is the case for the cpu/memory limits.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(front): Limits should not be changed if dirty

If the user has manually edited the limits fields then the UI should not
try to automatically calculate them again, using the limitFactors.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* Update image tags in 1.4 release branch (kubeflow#6096)

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* Fix NaN in Kubeflow Notebooks (kubeflow#6092)

* Fix NaN in Kubeflow Notebooks

* add a validation check for NaN values

* fix: validation check if value is None

* lint: fix line length

* add section for 1.4; small updates for 1.3 section (kubeflow#6082)

* add section for 1.4; small updates for 1.3 section

* refer to training operators consistently; add link to Notebooks roadmap.

* address @Bobgy feedback regarding Kubeflow Pipelines portions of 1.4 roadmap

* Remove virtualservice timeout to prevent websocket disconnect (kubeflow#6126)

In the existing version, the 'timeout: 300s' added to the notebook's virtual service would cause websockets to disconnect at the 5 minute mark, causing the Jupyter Notebook web terminal function to hang. This is described in kubeflow#6124.

* fix(admission-webhook): attach namespace to pod request if pod does not have it (kubeflow#6052)

* fix(admission-webhook): attach namespace to pod request if pod does not have it

* Apply suggestions from code review

Add more explicit logs

* Add more explicit log

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* Cherry-pick: Notebooks base gpg fix (kubeflow#6139)

* Update images for RC 1 (kubeflow#6137)

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* Final preparations for 1.4 (kubeflow#6177)

* Improve README for example-notebook-servers (kubeflow#6165)

* jwa(front): Don't disable vendors with no GPUs (kubeflow#6171)

JWA should not block users from selecting GPUs if the current cluster
nodes do not have any GPUs attached to them. We've seen users that have
autoscaled nodegroups for GPUs, so a GPU node will be added to the
cluster once a Pod has requested it.

Refs: arrikto/dev#1484

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* Update the version to 1.4

1. Run the `python releasing/update-manifests-images v1.4` script to
   update all the images of our components to the `v1.4` tag
2. Update the VERSION file so that the post-submit script will build all
   images

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* Bump Golang version in PodDefaults, TensorBoard Controller and KFAM to 1.17 (kubeflow#6180)

* kfam: Upgrade go to 1.17

Update to a more recent docker image that has a newer version of
openssl.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* poddefaults: Upgrade go to 1.17

Update to a more recent docker image that has a newer version of
openssl.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* tensorboards: Upgrade go to 1.17

Update to a more recent docker image that has a newer version of
openssl.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* Automated cherry pick of kubeflow#4751: Add AWS IAM Role for ServiceAccount support in profile Cherry pick of kubeflow#4751 on v1.0-branch. kubeflow#4751: Add AWS IAM Role for ServiceAccount support in profile (kubeflow#4804)

* Add AWS IAM Role for ServiceAccount support in profile controller

* Add eks profile iam plugin instructions and examples

* Clean up codes and add unit tests

* Optimize character size of policy document

* Always set oidc audience to trust identity

* Support AssumeRoleWithWebIdentity call via

* feat(jupyter): add ci

* Update build.yml

* Update build.yml

* feat(jupyter): Update GPU options

* fix(access-management): Add patch from kubeflow/kubeflow!5202

* fix(npm): vulnerabilities

* fix(profile): Update webpack.config.js for kf profiles

* ci: Don't use upstream issue triage

* ci: Stop building jupyter-web-app component

The jupyter-web-app component is superceded by StatCan/jupyter-apis.
This commit stops building the redundant component.

* ci: Build/push centraldashboard component

* chore(fonts): move into service

* fix: rearrange dashboard cards

* fix: failing tests

* fix: Fix broken container scan

Run container scanning as a step after building rather than its own job
so that it can find the target image.

* fix: Security vulnerabilities

* Upgrades packages to fix high and critical severity vulnerabilities
* Refactor to use updated Kubernetes client API

* fix: add 'ws' module to package.json

* wip(centraldashboard): Manage multiple profiles

* feat: Manage multiple profiles

* feat(centraldashboard):Official Languages

* fix(centraldashboard): i18n english text and french youtube link

* Automated cherry pick of kubeflow#5404: Remove metadata link from centraldashboard (kubeflow#5412)

* fix(jupyter): resolve build error

Resolves a build error for the changes in #2

* feat(jupyter): add ci

* Update build.yml

* Update build.yml

* ci: Stop building jupyter-web-app component

The jupyter-web-app component is superceded by StatCan/jupyter-apis.
This commit stops building the redundant component.

* fix: rearrange dashboard cards

* fix(centraldashboard): remove config map usage + artifact menu and quicklink

* feat(pipelines): Fix pipelines on dashboard

* fix(centraldashboard): fix pipeline tests

* fix(centraldashboard): Remove configmap call + fix conflicts

* feat(centraldashboard): Add i18n to dashboard links

* cherry pick missing commits into v1.3-branch (kubeflow#5836)

* Update notebook server base images (kubeflow#5804)

(cherry picked from commit ebc0c4f)

* Update second layer docker images to new tags and some python deps (kubeflow#5809)

(cherry picked from commit 3dbc352)

* Update image tags and python packages in pytorch and tensorflow full dockerfiles (kubeflow#5817)

(cherry picked from commit e8250b9)

* Add CI format checks for the Jupyter web app (kubeflow#5811)

* jwa(front): Add npm rule for checking the format

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* ci: Add common tasks for format checks

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* ci(jwa): Add format check tasks

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(back): Fix formatting

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(front): Fix formatting

Run `npm run format:write` on frontend

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(front): Include prettier in package.json

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(docker): Don't copy node_modules in Dockerfile

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* web-apps: Add global dockerignore file

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa: Remove unused dockerignore file

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(make): Don't include dockerignore and cleanup

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>
(cherry picked from commit 43e03d4)

* initial notebook server images README.md (kubeflow#5818)

(cherry picked from commit 923a7c8)

* Fix profile-controller CRD pruning issue (kubeflow#5822)

* api: Add marker for preserving unknown fields in Plugins

Signed-off-by: Yannis Zarkadas <yanniszark@arrikto.com>

* manifests: Regenerate manifests

Signed-off-by: Yannis Zarkadas <yanniszark@arrikto.com>
(cherry picked from commit 8a12599)

* Remove Logo Trademarks from the Jupyter Web App and make logos configurable (kubeflow#5823)

* rebase: Make logos configurable in configmap and remove trademark references

Rebased to remove the changes to the package-lock.json

* review: add suggested changes and add image group section to README

(cherry picked from commit d73e468)

* Make notebook limits configurable with a multiplication factor (kubeflow#5815)

* Make notebook limits configurable with a multiplication factor

* Make limits configurable under advanced section

* run prettier to format frontend code

* fix formatting and add rounding in backend

* Return error if limit is smaller than request

* Allow disabling limitFactor by setting it to none

* review: remove camelCase in python backend

* fix: update spawner_ui_config.yaml in manifests directory

* review: fix setting limits backend

* review: remove unnecessary check from backend

(cherry picked from commit 2ed54bc)

Co-authored-by: DavidSpek <vanderspek.david@gmail.com>
Co-authored-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>
Co-authored-by: Yannis Zarkadas <yanniszark@arrikto.com>

* Release v1.3.0-rc.1 for kubeflow/kubeflow (kubeflow#5838)

Follow release procedure to release images and manifests for version
v1.3.0-rc.1

Signed-off-by: Yannis Zarkadas <yanniszark@arrikto.com>

* Cherry-picks for 1.3.1 (kubeflow#6007)

* chore: Add comment to reference RStudio license. (kubeflow#5884)

* CI/CD: add kustomize build tests (kubeflow#5919)

* Add CI format checks for the Volumes web app (kubeflow#5820)

* vwa(front): Add npm script to check the formatting

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* vwa(front): Update the package-lock.json

Run `npm install` to bring the package-lock.json up to date

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* vwa(front): Fix formatting

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* vwa(back): Fix formatting

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* vwa(docker): Remove unused dockerignore file

We have created a global dockerignore file for all the web apps in the
parent dir.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* vwa(docker): Don't copy node_modules in dockerfile

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* vwa(make): Don't include dockerignore

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* ci(vwa): Add format check tasks

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(docker): Copy only necessary files for build

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* review: Use docker context instead of cd ..

Don't use a `cd ..` and copy dockerignore files back and forth. Instead
we should use the Docker context and the global dockerignore file we
have for all the web apps.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* tensorboard-controller: fix binding issue (kubeflow#5925)

* JWA: Don't override assets with logos ConfigMap (kubeflow#5942)

* jwa(front): Add static logos in the app

The app does not contain the logos' svgs in its source code/static
files. This results in the icons to not show when developing locally.

This commit adds the svgs found in the logos ConfigMap to the static
files of the app as well.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(front): Change logos fetch url

Change the URLs of the logos from `static/assets/*` to
`static/assets/logos`.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(manifests): Don't override assets with logos

Mount the ConfigMap under the `static/assets/logos` directory to not
override the contents of the entire assets dir.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* FIX The number of gpu must be set as string in Kubernetes/Openshift (kubeflow#5891)

* Update form.py

* Update form.py

* Update form-default.component.ts

* fix whitespace issue

* Update base notebook for Renovate compatibility (kubeflow#5955)

* Update base notebook for Renovate compatibility

* Separate kubectl and S6 arch

* Update jupyter, rstudio and vs code notebook images + allow rstudio in iframe (kubeflow#5961)

* Update jupyter, rstudio and vs code notebook images

* Remove sed and use substring removal

* Make menu bar scrollable again (kubeflow#5964)

* [fix]: Make jupyter-web-app parse workspace volume MountPath (kubeflow#5952)

* [fix]: Make jupyter-web-app parse workspace volume MountPath

- workspace volume path was fixed with "/home/jovyan"
- it should be enable to parse from jupyter-web-app-config's data

* change parsing key correctly

* Remove the f-strings in the Jupyter web app's backend (kubeflow#5680)

* Notebook servers: Add explanation about kernel not connecting (kubeflow#5920)

* fix(profile-controller): KNative probes (kubeflow#5848)

Knative control-plane components need to probe certain paths of Knative
Pods, in order to function correctly. These paths are:
- /healthz
- /metrics
- /ready
- /wait-for-drain

For this reason, we extend the Profile Controller to apply an Istio
AuthorizationPolicy that allows traffic to these HTTP paths for Pods in
user namespaces, as per Knative's instructions:
https://knative.dev/docs/serving/istio-authorization/#allowing-access-from-system-pods-by-paths

This only fixes requests through the public ingress. Cluster-internal
traffic and predictor-transformer use-cases are not supported yet. We
continue discussing these in kubeflow#5965.

Refs kubeflow#5965
Refs kserve/kserve#1558

Signed-off-by: Yurii Komar <subreptivus@gmail.com>

* Update dockerfiles and make compatible with Renovate (kubeflow#5968)

* Update dockerfiles and make compatible with Renovate

* Set memory for jupyter pytorch to the same as jupyter tensorflow

* Update protobuf

* Remove conda version and use substring expansion

* Update SQLAlchemy

* Update dill

* Extend deadline for PyTorch image building (kubeflow#5991)

* Update last batch of notebook server Dockerfiles (kubeflow#5997)

* notebooks: Add additional PGP server for RStudio package validation (kubeflow#6010)

* Change PGP server for RStudio validation to pgp.surfnet.nl

* Add as secondary keyserver

* use eslint versions from upstream v1.3 branch

* fix eslint errors in pipelines.js

* fix: test with namespace for pipeline-cards

* fix: wrong text for test

* centraldashboard: Update node and use latest-stable (kubeflow#6260)

Change the tests to NOT fetch Chromium from the Edge branch, which is develop,
but instead use latest-stable. We saw that edge can have problems from times to
times.

In the same commit we also update the node version to fix CVEs with the current
v12.18.3 version that we had.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>
(cherry picked from commit bedda47)

* eslint auto fixes

* fix(vuln): Update base image

Update base image and fix json-schema
Comment out vulnerability scan (should be reworked to actually block)

* feat(ci): add notebook-controller workflow

* fix(ci): update notebook-controller ci (#86)

Co-authored-by: jumana-s <salwa.mohamed@canda.ca>

* fix(ci): push to acr once (#87)

Co-authored-by: jumana-s <salwa.mohamed@canda.ca>

* feat(notebook-controller): add readiness probe (#85)

* fix(npm): fix vulnerabilities

* install(packages): overcome CVEs (#90)

* install(packages): overcome CVEs
* regenerate package-lock.json
* Fix package version

Co-authored-by: Bryan Paget <bryan.paget@statcan.gc.ca>
Co-authored-by: Wendy V Gaultier <wendyvgaultier@gmail.com>

Signed-off-by: Yannis Zarkadas <yanniszark@arrikto.com>
Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>
Signed-off-by: Ilias Katsakioris <elikatsis@arrikto.com>
Signed-off-by: Yurii Komar <subreptivus@gmail.com>
Signed-off-by: Malini Bhandaru <mbhandaru@vmware.com>
Co-authored-by: Yannis Zarkadas <yanniszark@arrikto.com>
Co-authored-by: DavidSpek <vanderspek.david@gmail.com>
Co-authored-by: Jeremy Lewi <jeremy+github@lewi.us>
Co-authored-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>
Co-authored-by: Mathew Wicks <thesuperzapper@users.noreply.github.com>
Co-authored-by: Ilias Katsakioris <elikatsis@arrikto.com>
Co-authored-by: James Liu <37026441+zijianjoy@users.noreply.github.com>
Co-authored-by: toshi_k <high_luin@yahoo.co.jp>
Co-authored-by: juliusvonkohout <45896133+juliusvonkohout@users.noreply.github.com>
Co-authored-by: Saffa Alvi <alvi118@uwindsor.ca>
Co-authored-by: Wendy Gaultier <wvgaultier@gmail.com>
Co-authored-by: Jaeyeon Kim <anencore94@gmail.com>
Co-authored-by: Yurii Komar <Subreptivus@gmail.com>
Co-authored-by: JessicaBarh <jessicabarhoma@gmail.com>
Co-authored-by: Jose-Matsuda <tongster789@gmail.com>
Co-authored-by: Stephen Hopper <50119450+hopper-signifyd@users.noreply.github.com>
Co-authored-by: Malini Bhandaru <mbhandaru@vmware.com>
Co-authored-by: Filinto Duran <duranto@gmail.com>
Co-authored-by: Filinto Duran <fduran@d2iq.com>
Co-authored-by: Jiaxin Shan <seedjeffwan@gmail.com>
Co-authored-by: Frances Zsurka <franceszsurka@gmail.com>
Co-authored-by: William H <sylus1984@gmail.com>
Co-authored-by: Zachary Seguin <zachary@zacharyseguin.ca>
Co-authored-by: Zachary Seguin <zachary.seguin@canada.ca>
Co-authored-by: Brendan Gadd <brendangadd@gmail.com>
Co-authored-by: wg102 <wgaul102@uottawa.ca>
Co-authored-by: frazs <frances.zsurka@gmail.com>
Co-authored-by: Skye Turriff <turriff.skye@gmail.com>
Co-authored-by: Jose-Matsuda <jose.matsuda@canada.ca>
Co-authored-by: jumana-s <salwa.mohamed@canda.ca>
Co-authored-by: Salwa <51963397+jumana-s@users.noreply.github.com>
Co-authored-by: Wendy V Gaultier <wendyvgaultier@gmail.com>
Co-authored-by: Bryan Paget <bbrryyaann@protonmail.com>
Co-authored-by: Bryan Paget <bryan.paget@statcan.gc.ca>
bryanpaget added a commit to StatCan/kubeflow that referenced this pull request Sep 9, 2022
* releasing: Add WG-Notebooks leads to release owners (kubeflow#5777)

Signed-off-by: Yannis Zarkadas <yanniszark@arrikto.com>

* Specify commonLabels for tensorboard-controller (kubeflow#5780)

* Remove jlewi as an approver (kubeflow#5786)

Removing myself as an owner. This should lead to better auto-assignment of code reviews.

* Update notebook server base images (kubeflow#5804)

* Update second layer docker images to new tags and some python deps (kubeflow#5809)

* Update image tags and python packages in pytorch and tensorflow full dockerfiles (kubeflow#5817)

* Add CI format checks for the Jupyter web app (kubeflow#5811)

* jwa(front): Add npm rule for checking the format

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* ci: Add common tasks for format checks

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* ci(jwa): Add format check tasks

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(back): Fix formatting

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(front): Fix formatting

Run `npm run format:write` on frontend

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(front): Include prettier in package.json

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(docker): Don't copy node_modules in Dockerfile

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* web-apps: Add global dockerignore file

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa: Remove unused dockerignore file

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(make): Don't include dockerignore and cleanup

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* initial notebook server images README.md (kubeflow#5818)

* tensorboard-controller: Fix scheduling unbound PVCs (kubeflow#5819)

When the TB controller attempts to schedule a RWO PVC it checks its
accessModes in the PVC status. The controller panics if the list is
empty.

This commit adds a check to ensure the list is not empty.

Signed-off-by: Ilias Katsakioris <elikatsis@arrikto.com>

* Fix profile-controller CRD pruning issue (kubeflow#5822)

* api: Add marker for preserving unknown fields in Plugins

Signed-off-by: Yannis Zarkadas <yanniszark@arrikto.com>

* manifests: Regenerate manifests

Signed-off-by: Yannis Zarkadas <yanniszark@arrikto.com>

* Remove Logo Trademarks from the Jupyter Web App and make logos configurable (kubeflow#5823)

* rebase: Make logos configurable in configmap and remove trademark references

Rebased to remove the changes to the package-lock.json

* review: add suggested changes and add image group section to README

* Make notebook limits configurable with a multiplication factor (kubeflow#5815)

* Make notebook limits configurable with a multiplication factor

* Make limits configurable under advanced section

* run prettier to format frontend code

* fix formatting and add rounding in backend

* Return error if limit is smaller than request

* Allow disabling limitFactor by setting it to none

* review: remove camelCase in python backend

* fix: update spawner_ui_config.yaml in manifests directory

* review: fix setting limits backend

* review: remove unnecessary check from backend

* notebook-servers: Update JupyterLab and add Git Extension (kubeflow#5846)

* chore: Add comment to reference RStudio license. (kubeflow#5884)

* CI/CD: add kustomize build tests (kubeflow#5919)

* Add CI format checks for the Volumes web app (kubeflow#5820)

* vwa(front): Add npm script to check the formatting

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* vwa(front): Update the package-lock.json

Run `npm install` to bring the package-lock.json up to date

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* vwa(front): Fix formatting

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* vwa(back): Fix formatting

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* vwa(docker): Remove unused dockerignore file

We have created a global dockerignore file for all the web apps in the
parent dir.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* vwa(docker): Don't copy node_modules in dockerfile

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* vwa(make): Don't include dockerignore

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* ci(vwa): Add format check tasks

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(docker): Copy only necessary files for build

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* review: Use docker context instead of cd ..

Don't use a `cd ..` and copy dockerignore files back and forth. Instead
we should use the Docker context and the global dockerignore file we
have for all the web apps.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* tensorboard-controller: fix binding issue (kubeflow#5925)

* JWA: Don't override assets with logos ConfigMap (kubeflow#5942)

* jwa(front): Add static logos in the app

The app does not contain the logos' svgs in its source code/static
files. This results in the icons to not show when developing locally.

This commit adds the svgs found in the logos ConfigMap to the static
files of the app as well.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(front): Change logos fetch url

Change the URLs of the logos from `static/assets/*` to
`static/assets/logos`.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(manifests): Don't override assets with logos

Mount the ConfigMap under the `static/assets/logos` directory to not
override the contents of the entire assets dir.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* Make it possible to add Namespaced menu items (kubeflow#5871)

* Make it possible to add Namespaced menu items

* Reduce the nesting levels

* FIX The number of gpu must be set as string in Kubernetes/Openshift (kubeflow#5891)

* Update form.py

* Update form.py

* Update form-default.component.ts

* fix whitespace issue

* feat(jupyter): add fonts as assets to service (kubeflow#5691)

* cwa(front): Ignore font files in assets

* feat(jupyter): add fonts as assets to service

* CRUD: fonts in common

* CWA: Remove link to css file

* jwa(front): Remove font assets from jupyter

Co-authored-by: Wendy Gaultier <wvgaultier@gmail.com>

* Update base notebook for Renovate compatibility (kubeflow#5955)

* Update base notebook for Renovate compatibility

* Separate kubectl and S6 arch

* Update jupyter, rstudio and vs code notebook images + allow rstudio in iframe (kubeflow#5961)

* Update jupyter, rstudio and vs code notebook images

* Remove sed and use substring removal

* Make menu bar scrollable again (kubeflow#5964)

* [fix]: Make jupyter-web-app parse workspace volume MountPath (kubeflow#5952)

* [fix]: Make jupyter-web-app parse workspace volume MountPath

- workspace volume path was fixed with "/home/jovyan"
- it should be enable to parse from jupyter-web-app-config's data

* change parsing key correctly

* Remove the f-strings in the Jupyter web app's backend (kubeflow#5680)

* Notebook servers: Add explanation about kernel not connecting (kubeflow#5920)

* fix(profile-controller): KNative probes (kubeflow#5848)

Knative control-plane components need to probe certain paths of Knative
Pods, in order to function correctly. These paths are:
- /healthz
- /metrics
- /ready
- /wait-for-drain

For this reason, we extend the Profile Controller to apply an Istio
AuthorizationPolicy that allows traffic to these HTTP paths for Pods in
user namespaces, as per Knative's instructions:
https://knative.dev/docs/serving/istio-authorization/#allowing-access-from-system-pods-by-paths

This only fixes requests through the public ingress. Cluster-internal
traffic and predictor-transformer use-cases are not supported yet. We
continue discussing these in kubeflow#5965.

Refs kubeflow#5965
Refs kserve/kserve#1558

Signed-off-by: Yurii Komar <subreptivus@gmail.com>

* feat(jupyter): Support hiding image registry/tag (kubeflow#5681)

* feat(jupyter): Support hiding image registry/tag

* feat(jupyter): Support hiding VSCode/RStudio images registry/tag

* rebasing onto upstream master

* fix: formatting with prettier and set default hideRegistry=true

* fix: move hideRegistry and hideTag from spawnerFormDefaults.image

* fix whitespace

* add tooltip for images

Co-authored-by: JessicaBarh <jessicabarhoma@gmail.com>

* Update dockerfiles and make compatible with Renovate (kubeflow#5968)

* Update dockerfiles and make compatible with Renovate

* Set memory for jupyter pytorch to the same as jupyter tensorflow

* Update protobuf

* Remove conda version and use substring expansion

* Update SQLAlchemy

* Update dill

* Create OWNERS file in .github folder for Renovate config + workflows (kubeflow#5983)

* Extend deadline for PyTorch image building (kubeflow#5991)

* Allow user to add/delete labels to user namespace using ConfigMap. Fix kubeflow#5712 (kubeflow#5761)

The profile-controller applies a hardcoded list of labels to Profile
namespaces. Make this list of labels configurable with a ConfigMap. The
new list of steps for the profile-controller is:

1. **(new)** Profile-controller starts and reads the file given by the
   `--namespace-labels-path` CLI flag (defaults to
   `/etc/profile-controller/namespace-labels.yaml`). This file contains
   a YAML dict of label key/values to add to each Profile namespace.
2. Profile-controller creates namespace.
3. Profile-controller merges existing namespace labels with given
   labels. A given label is applied ONLY if it doesn't exist already.
4. **(new)** If a given label value is empty (``), remove the label from
   the namespace if it already exists.

We are using a ConfigMap instead of a plain CLI flag, so that users can
change the list of labels without restarting the profile-controller.

Closes kubeflow#5712

Signed-off-by: Yannis Zarkadas <yanniszark@arrikto.com>

Co-authored-by: James Liu <jamxl@google.com>

Co-authored-by: Yannis Zarkadas <yanniszark@arrikto.com>

* Update last batch of notebook server Dockerfiles (kubeflow#5997)

* Update spawner_ui_config.yaml with new tags (kubeflow#5998)

* Update images in spawner_ui_config.yaml

* Update manifest image tags

* CRUD web apps:  i18n (kubeflow#5880)

* feat(jupyter): add modules for translations

* feat(jupyter): fix path for translations + add GPU translations

* feat(jupyter): add translations

* feat(jupyter): add translations to form-default

* feat(jupyter): add translations to affinity/tolerations

* feat(jupyter): add translations to Image Pull Policy

* feat(jupyter): remove imports from app.module.ts + fix translations

* web-apps(front): add TranslationModule to common for jupyter

* Implement i18n for jupyter frontend
- Including common components needed
- Refactor translation file

* web-apps(front): add i18n to messages from backend
Status tooltip text
Snackbar message

* web-apps(front): volumes and tensorboards i18n ini
Add the i18n feature for the other 2 projects

* feat(volume): add translations for volume

* feat(jupyter): refactor asset file

* Update asset files for all folders

* feat(tensorboards): add i18n
Update all asset files

* cwa: add new translations

* cwa: Add README documentation

* fix: formatting with prettier

* Fix tests

* Fix python format

* remove comment

Co-authored-by: saffaalvi <alvi118@uwindsor.ca>
Co-authored-by: Jose-Matsuda <tongster789@gmail.com>

* Remove deprecated JWA (kubeflow#5959)

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* Sync spawner yaml from source to manifests (kubeflow#5999)

* notebooks: Add additional PGP server for RStudio package validation (kubeflow#6010)

* Change PGP server for RStudio validation to pgp.surfnet.nl

* Add as secondary keyserver

* Add support for ServiceAccountName and AutomountServiceAccountName to admission-webhook (kubeflow#5939)

* WA: Use relative paths for fetching translation files (kubeflow#6034)

* jwa(front): Use relative path for i18n json

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* vwa(front): Use relative path for i18n json

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* twa(front): Use relative path for i18n json

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* Update CRUD web apps from Angular 8 to Angular 12 (kubeflow#6004)

* common: update angular core and cli to 9

ng update @angular/core@9 @angular/cli@9

* common: update material, cdk and cdk-experimental to 9

ng update @angular/cdk@9 @angular/material@9 @angular/cdk-experimental@9

* common: remove entryComponents and replace TestBed.get with TestBed.inject

Edit files manually

* common: make compatible with Angular 9

Edit files manually

* common: update fontawesome to 0.6.0

npm i @fortawesome/angular-fontawesome@0.6.0

* update kubeflow-common project dependencies

* common: add @angular/localize

ng add @angular/localize

* jupyter: update angular core and cli 8

ng update @angular/core@8 @angular/cli@8

* jupyter: update angular core and cli to 9

ng update @angular/core@9 @angular/cli@9

* jupyter: update material, cdk and cdk-experimental to 9

ng update @angular/cdk@9 @angular/material@9 @angular/cdk-experimental@9

* jupyter: remove entryComponents

Edit file manually

* jupyter: update fontawesome to 0.6.0

npm i @fortawesome/angular-fontawesome@0.6.0

* jupyter: add @angular/localize

ng add @angular/localize

* tensorboards: update angular core and cli to 9

ng update @angular/core@9 @angular/cli@9

* tensorboards: update material, cdk and cdk-experimental to 9

ng update @angular/cdk@9 @angular/material@9 @angular/cdk-experimental@9

* tensorboards: remove entryComponents and replace TestBed.get with TestBed.inje…

Edit files manually

* tensorboards: update fontawesome to 0.6.0

npm i @fortawesome/angular-fontawesome@0.6.0

* tensorboards: add @angular/localize

ng add @angular/localize

* Fix tensorboards-web-app run-dev in makefile

Edit file manually

* volumes: update angular core and cli to 9

ng update @angular/core@9 @angular/cli@9

* volumes: update material, cdk and cdk-experimental to 9

ng update @angular/cdk@9 @angular/material@9 @angular/cdk-experimental@9

* volumes: remove entryComponents and replace TestBed.get with TestBe…

Edit files manually

* volumes: update fontawesome to 0.6.0

npm i @fortawesome/angular-fontawesome@0.6.0

* volumes: add @angular/localize

ng add @angular/localize

* common: update angular core and cli to 10

ng update @angular/core@10 @angular/cli@10 --force

* common: update material, cdk and cdk-experimental to 10

ng update @angular/cdk@10 @angular/material@10 @angular/cdk-experimental@10

* update kubeflow-common project dependencies

Edit file manually

* common: update fontawesome to 0.7.0

npm i @fortawesome/angular-fontawesome@0.7.0

* common: fix font location

Edit file manually

* jupyter: update angular-fontawesome to 0.7.0

npm i @fortawesome/angular-fontawesome@0.7.0

* jupyter: update angular core and cli to 10

ng update @angular/core@10 @angular/cli@10

* jupyter: update material, cdk and cdk-experimental to 10

ng update @angular/cdk@10 @angular/material@10 @angular/cdk-experimental@10

* tensorboards: update angular-fontawesome to 0.7.0

npm i @fortawesome/angular-fontawesome@0.7.0

* tensorboards: update angular core and cli to 10

ng update @angular/core@10 @angular/cli@10

* tensorboards: update material, cdk and cdk-experimental to 10

ng update @angular/cdk@10 @angular/material@10 @angular/cdk-experimental@10

* volumes: update angular-fontawesome to 0.7.0

npm i @fortawesome/angular-fontawesome@0.7.0

* volumes: update angular core and cli to 10

ng update @angular/core@10 @angular/cli@10

* volumes: update material, cdk and cdk-experimental to 10

ng update @angular/cdk@10 @angular/material@10 @angular/cdk-experimental@10

* common: update angular-fontawesome to 0.8.2

npm i @fortawesome/angular-fontawesome@0.8.2

* common: update codelyzer to 6.0.2

npm i codelyzer@6.0.2

* common: install @angular-devkit/core@11

npm install --save-dev @angular-devkit/core@11

* common: update angular core and cli to 11

ng update @angular/core@11 @angular/cli@11

* common: update material, cdk and cdk-experimental to 11

ng update @angular/cdk@11 @angular/material@11 @angular/cdk-experimental@11

* update kubeflow-common project dependencies

Edit file manually

* jupyter: update angular-fontawesome to 0.8.2

npm i @fortawesome/angular-fontawesome@0.8.2

* jupyter: update codelyzer to 6.0.2

npm i codelyzer@6.0.2

* jupyter: update angular core and cli to 11

ng update @angular/core@11 @angular/cli@11

* jupyter: update material, cdk and cdk-experimental to 11

ng update @angular/cdk@11 @angular/material@11 @angular/cdk-experimental@11

* tensorboards: update angular-fontawesome to 0.8.2

npm i @fortawesome/angular-fontawesome@0.8.2

* tensorboards: update codelyzer to 6.0.2

npm i codelyzer@6.0.2

* tensorboards: update angular core and cli to 11

ng update @angular/core@11 @angular/cli@11

* tensorboards: update material, cdk and cdk-experimental to 11

ng update @angular/cdk@11 @angular/material@11 @angular/cdk-experimental@11

* volumes: update angular-fontawesome to 0.8.2

npm i @fortawesome/angular-fontawesome@0.8.2

* volumes: update codelyzer to 6.0.2

npm i codelyzer@6.0.2

* volumes: update angular core and cli to 11

ng update @angular/core@11 @angular/cli@11

* volumes: update material, cdk and cdk-experimental to 11

ng update @angular/cdk@11 @angular/material@11 @angular/cdk-experimental@11

* common: Fix CI failure

* jupyter: npm run format:write

npm run format:write

* volumes: npm run format:write

npm run format:write

* tensorboards: manually run prettier

prettier --write 'src/**/*.{js,ts,html,scss,css}'

* common: Fix MockComponent entryComponents for CI

Manually edit file

* common: update angular-fontawesome to 0.9.0

npm i @fortawesome/angular-fontawesome@0.9.0

* common: update angular core and cli to 12

ng update @angular/core@12 @angular/cli@12

* common: update material, cdk and cdk-experimental to 12

ng update @angular/cdk@12 @angular/material@12 @angular/cdk-experimental@12

* update kubeflow-common project dependencies

Edit file manually

* common: fix Intl.DateTimeFormatOptions

Edit file manually. Type of defaultDateOptions and defaultTimeOptions needs to explicitly be set to Intl.DateTimeFormatOptions.

* jupyter: update angular-fontawesome to 0.9.0

npm i @fortawesome/angular-fontawesome@0.9.0

* jupyter: update codelyzer to 6.0.2

npm i codelyzer@6.0.2

* jupyter: update angular core and cli to 12

ng update @angular/core@12 @angular/cli@12

* jupyter: update material, cdk and cdk-experimental to 12

ng update @angular/cdk@12 @angular/material@12 @angular/cdk-experimental@12

* tensorboards: update angular-fontawesome to 0.9.0

npm i @fortawesome/angular-fontawesome@0.9.0

* tensorboards: update angular core and cli to 12

ng update @angular/core@12 @angular/cli@12

* tensorboards: update material, cdk and cdk-experimental to 12

ng update @angular/cdk@12 @angular/material@12 @angular/cdk-experimental@12

* volumes: update angular-fontawesome to 0.9.0

npm i @fortawesome/angular-fontawesome@0.9.0

* volumes: update angular core and cli to 12

ng update @angular/core@12 @angular/cli@12

* volumes: update material, cdk and cdk-experimental to 12

ng update @angular/cdk@12 @angular/material@12 @angular/cdk-experimental@12

* common: update and audit packages

npm update && npm audit fix && npm i lodash-es@4.17.21 && npm i material-icons@0.7.3

* jupyter: update and audit packages

npm update && npm audit fix && npm i material-icons@0.7.3

* tensorboards: update and audit packages

npm update && npm audit fix && npm i material-icons@0.7.3

* volumes: update and audit packages

npm update && npm audit fix && npm i material-icons@0.7.3

* jupyter: npm run format:write

npm run format:write

* lists roadmap in reverse chronological order (kubeflow#6055)

Signed-off-by: Malini Bhandaru <mbhandaru@vmware.com>

* Correct missing predicates in controller watches. Fixes kubeflow#5326 (kubeflow#5873)

Co-authored-by: Filinto Duran <fduran@d2iq.com>

* fix(web-apps): Use Angular's i18n proposed implementation (kubeflow#6065)

* jwa(front): Add i18n for french in angular.json

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(front): Add i18n rules in package.json

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(front): i18n FormImage

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(front): i18n FormAdvancedOptions

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(front): i18n FormAffinityTolerations

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(front): i18n FormConfigurations

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(front): i18n FormCpuRam

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(front): i18n FormDataVolumes

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(front): i18n FormGpus

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(front): i18n FromName

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(front): i18n FormWorkspaceVolume

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(front): i18n Volume

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* web-apps(front): i18n BackendService

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* web-apps(front): Add localize to peerDependencies

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* web-apps(front): Import $localize in library

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* web-apps(front): i18n RokService

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(front): i18n Form

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(front): i18n BackendService

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* web-apps(front): i18n ConfirmDialog

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(front): i18n Index

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* web-apps(front): i18n ResourceTable

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* web-apps(front): i18n SnackBar

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* web-apps(front): i18n Form

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* web-app(front): i18n NameNamespaceSelector

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* web-apps(front): Remove ngx-translate

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(front): i18n App

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(front): Remove ngx-translate from package.json

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(back): Don't parse keys in status

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* vwa(back): Revert backend messages

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(front): Update translation language files

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* web-apps(back): Don't parse keys in status

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* vwa(front): Add i18n for french in angular.json

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* vwa(front): Add i18n rules in package.json

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* vwa(front): i18n Form

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* vwa(front): i18n Index

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* vwa(front): i18n BackendService

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* vwa(front): i18n App

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* vwa(front): Remove ngx-translate from package.json

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* vwa(front): i18n Form

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* vwa(front): Update translation language files

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* twa(back): Revert backend messages

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* twa(front): Add i18n rules in package.json

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* twa(front): Add i18n for french in angular.json

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* twa(front): i18n Index

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* twa(front): i18n BackendService

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* twa(front): i18n App

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* twa(front): Update translation language files

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* twa(front): Remove ngx-translate from package.json

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* wa: Update READMEs

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* web-apps(front): Add localize when testing

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* [CentralDashboard] Add entry for Models web app (kubeflow#6085)

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* Interact with the namespace selector (kubeflow#5995)

* Interact with the namespace selector

* Use namespace given as an argument

* Keep the index of selected item

* fix(jwa): Fix limits calculation when limitFactor is none (kubeflow#6058)

* jwa(front): Don't allow NaN values in limits

The UI should always catch a NaN value and don't add it in the form.
Currently this is the case for the cpu/memory limits.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(front): Limits should not be changed if dirty

If the user has manually edited the limits fields then the UI should not
try to automatically calculate them again, using the limitFactors.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* Update image tags in 1.4 release branch (kubeflow#6096)

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* Fix NaN in Kubeflow Notebooks (kubeflow#6092)

* Fix NaN in Kubeflow Notebooks

* add a validation check for NaN values

* fix: validation check if value is None

* lint: fix line length

* add section for 1.4; small updates for 1.3 section (kubeflow#6082)

* add section for 1.4; small updates for 1.3 section

* refer to training operators consistently; add link to Notebooks roadmap.

* address @Bobgy feedback regarding Kubeflow Pipelines portions of 1.4 roadmap

* Remove virtualservice timeout to prevent websocket disconnect (kubeflow#6126)

In the existing version, the 'timeout: 300s' added to the notebook's virtual service would cause websockets to disconnect at the 5 minute mark, causing the Jupyter Notebook web terminal function to hang. This is described in kubeflow#6124.

* fix(admission-webhook): attach namespace to pod request if pod does not have it (kubeflow#6052)

* fix(admission-webhook): attach namespace to pod request if pod does not have it

* Apply suggestions from code review

Add more explicit logs

* Add more explicit log

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* Cherry-pick: Notebooks base gpg fix (kubeflow#6139)

* Update images for RC 1 (kubeflow#6137)

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* Final preparations for 1.4 (kubeflow#6177)

* Improve README for example-notebook-servers (kubeflow#6165)

* jwa(front): Don't disable vendors with no GPUs (kubeflow#6171)

JWA should not block users from selecting GPUs if the current cluster
nodes do not have any GPUs attached to them. We've seen users that have
autoscaled nodegroups for GPUs, so a GPU node will be added to the
cluster once a Pod has requested it.

Refs: arrikto/dev#1484

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* Update the version to 1.4

1. Run the `python releasing/update-manifests-images v1.4` script to
   update all the images of our components to the `v1.4` tag
2. Update the VERSION file so that the post-submit script will build all
   images

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* Bump Golang version in PodDefaults, TensorBoard Controller and KFAM to 1.17 (kubeflow#6180)

* kfam: Upgrade go to 1.17

Update to a more recent docker image that has a newer version of
openssl.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* poddefaults: Upgrade go to 1.17

Update to a more recent docker image that has a newer version of
openssl.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* tensorboards: Upgrade go to 1.17

Update to a more recent docker image that has a newer version of
openssl.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* Automated cherry pick of kubeflow#4751: Add AWS IAM Role for ServiceAccount support in profile Cherry pick of kubeflow#4751 on v1.0-branch. kubeflow#4751: Add AWS IAM Role for ServiceAccount support in profile (kubeflow#4804)

* Add AWS IAM Role for ServiceAccount support in profile controller

* Add eks profile iam plugin instructions and examples

* Clean up codes and add unit tests

* Optimize character size of policy document

* Always set oidc audience to trust identity

* Support AssumeRoleWithWebIdentity call via

* feat(jupyter): add ci

* Update build.yml

* Update build.yml

* feat(jupyter): Update GPU options

* fix(access-management): Add patch from kubeflow/kubeflow!5202

* fix(npm): vulnerabilities

* fix(profile): Update webpack.config.js for kf profiles

* ci: Don't use upstream issue triage

* ci: Stop building jupyter-web-app component

The jupyter-web-app component is superceded by StatCan/jupyter-apis.
This commit stops building the redundant component.

* ci: Build/push centraldashboard component

* chore(fonts): move into service

* fix: rearrange dashboard cards

* fix: failing tests

* fix: Fix broken container scan

Run container scanning as a step after building rather than its own job
so that it can find the target image.

* fix: Security vulnerabilities

* Upgrades packages to fix high and critical severity vulnerabilities
* Refactor to use updated Kubernetes client API

* fix: add 'ws' module to package.json

* wip(centraldashboard): Manage multiple profiles

* feat: Manage multiple profiles

* feat(centraldashboard):Official Languages

* fix(centraldashboard): i18n english text and french youtube link

* Automated cherry pick of kubeflow#5404: Remove metadata link from centraldashboard (kubeflow#5412)

* fix(jupyter): resolve build error

Resolves a build error for the changes in #2

* feat(jupyter): add ci

* Update build.yml

* Update build.yml

* ci: Stop building jupyter-web-app component

The jupyter-web-app component is superceded by StatCan/jupyter-apis.
This commit stops building the redundant component.

* fix: rearrange dashboard cards

* fix(centraldashboard): remove config map usage + artifact menu and quicklink

* feat(pipelines): Fix pipelines on dashboard

* fix(centraldashboard): fix pipeline tests

* fix(centraldashboard): Remove configmap call + fix conflicts

* feat(centraldashboard): Add i18n to dashboard links

* cherry pick missing commits into v1.3-branch (kubeflow#5836)

* Update notebook server base images (kubeflow#5804)

(cherry picked from commit ebc0c4f)

* Update second layer docker images to new tags and some python deps (kubeflow#5809)

(cherry picked from commit 3dbc352)

* Update image tags and python packages in pytorch and tensorflow full dockerfiles (kubeflow#5817)

(cherry picked from commit e8250b9)

* Add CI format checks for the Jupyter web app (kubeflow#5811)

* jwa(front): Add npm rule for checking the format

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* ci: Add common tasks for format checks

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* ci(jwa): Add format check tasks

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(back): Fix formatting

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(front): Fix formatting

Run `npm run format:write` on frontend

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(front): Include prettier in package.json

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(docker): Don't copy node_modules in Dockerfile

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* web-apps: Add global dockerignore file

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa: Remove unused dockerignore file

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(make): Don't include dockerignore and cleanup

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>
(cherry picked from commit 43e03d4)

* initial notebook server images README.md (kubeflow#5818)

(cherry picked from commit 923a7c8)

* Fix profile-controller CRD pruning issue (kubeflow#5822)

* api: Add marker for preserving unknown fields in Plugins

Signed-off-by: Yannis Zarkadas <yanniszark@arrikto.com>

* manifests: Regenerate manifests

Signed-off-by: Yannis Zarkadas <yanniszark@arrikto.com>
(cherry picked from commit 8a12599)

* Remove Logo Trademarks from the Jupyter Web App and make logos configurable (kubeflow#5823)

* rebase: Make logos configurable in configmap and remove trademark references

Rebased to remove the changes to the package-lock.json

* review: add suggested changes and add image group section to README

(cherry picked from commit d73e468)

* Make notebook limits configurable with a multiplication factor (kubeflow#5815)

* Make notebook limits configurable with a multiplication factor

* Make limits configurable under advanced section

* run prettier to format frontend code

* fix formatting and add rounding in backend

* Return error if limit is smaller than request

* Allow disabling limitFactor by setting it to none

* review: remove camelCase in python backend

* fix: update spawner_ui_config.yaml in manifests directory

* review: fix setting limits backend

* review: remove unnecessary check from backend

(cherry picked from commit 2ed54bc)

Co-authored-by: DavidSpek <vanderspek.david@gmail.com>
Co-authored-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>
Co-authored-by: Yannis Zarkadas <yanniszark@arrikto.com>

* Release v1.3.0-rc.1 for kubeflow/kubeflow (kubeflow#5838)

Follow release procedure to release images and manifests for version
v1.3.0-rc.1

Signed-off-by: Yannis Zarkadas <yanniszark@arrikto.com>

* Cherry-picks for 1.3.1 (kubeflow#6007)

* chore: Add comment to reference RStudio license. (kubeflow#5884)

* CI/CD: add kustomize build tests (kubeflow#5919)

* Add CI format checks for the Volumes web app (kubeflow#5820)

* vwa(front): Add npm script to check the formatting

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* vwa(front): Update the package-lock.json

Run `npm install` to bring the package-lock.json up to date

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* vwa(front): Fix formatting

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* vwa(back): Fix formatting

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* vwa(docker): Remove unused dockerignore file

We have created a global dockerignore file for all the web apps in the
parent dir.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* vwa(docker): Don't copy node_modules in dockerfile

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* vwa(make): Don't include dockerignore

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* ci(vwa): Add format check tasks

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(docker): Copy only necessary files for build

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* review: Use docker context instead of cd ..

Don't use a `cd ..` and copy dockerignore files back and forth. Instead
we should use the Docker context and the global dockerignore file we
have for all the web apps.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* tensorboard-controller: fix binding issue (kubeflow#5925)

* JWA: Don't override assets with logos ConfigMap (kubeflow#5942)

* jwa(front): Add static logos in the app

The app does not contain the logos' svgs in its source code/static
files. This results in the icons to not show when developing locally.

This commit adds the svgs found in the logos ConfigMap to the static
files of the app as well.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(front): Change logos fetch url

Change the URLs of the logos from `static/assets/*` to
`static/assets/logos`.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(manifests): Don't override assets with logos

Mount the ConfigMap under the `static/assets/logos` directory to not
override the contents of the entire assets dir.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* FIX The number of gpu must be set as string in Kubernetes/Openshift (kubeflow#5891)

* Update form.py

* Update form.py

* Update form-default.component.ts

* fix whitespace issue

* Update base notebook for Renovate compatibility (kubeflow#5955)

* Update base notebook for Renovate compatibility

* Separate kubectl and S6 arch

* Update jupyter, rstudio and vs code notebook images + allow rstudio in iframe (kubeflow#5961)

* Update jupyter, rstudio and vs code notebook images

* Remove sed and use substring removal

* Make menu bar scrollable again (kubeflow#5964)

* [fix]: Make jupyter-web-app parse workspace volume MountPath (kubeflow#5952)

* [fix]: Make jupyter-web-app parse workspace volume MountPath

- workspace volume path was fixed with "/home/jovyan"
- it should be enable to parse from jupyter-web-app-config's data

* change parsing key correctly

* Remove the f-strings in the Jupyter web app's backend (kubeflow#5680)

* Notebook servers: Add explanation about kernel not connecting (kubeflow#5920)

* fix(profile-controller): KNative probes (kubeflow#5848)

Knative control-plane components need to probe certain paths of Knative
Pods, in order to function correctly. These paths are:
- /healthz
- /metrics
- /ready
- /wait-for-drain

For this reason, we extend the Profile Controller to apply an Istio
AuthorizationPolicy that allows traffic to these HTTP paths for Pods in
user namespaces, as per Knative's instructions:
https://knative.dev/docs/serving/istio-authorization/#allowing-access-from-system-pods-by-paths

This only fixes requests through the public ingress. Cluster-internal
traffic and predictor-transformer use-cases are not supported yet. We
continue discussing these in kubeflow#5965.

Refs kubeflow#5965
Refs kserve/kserve#1558

Signed-off-by: Yurii Komar <subreptivus@gmail.com>

* Update dockerfiles and make compatible with Renovate (kubeflow#5968)

* Update dockerfiles and make compatible with Renovate

* Set memory for jupyter pytorch to the same as jupyter tensorflow

* Update protobuf

* Remove conda version and use substring expansion

* Update SQLAlchemy

* Update dill

* Extend deadline for PyTorch image building (kubeflow#5991)

* Update last batch of notebook server Dockerfiles (kubeflow#5997)

* notebooks: Add additional PGP server for RStudio package validation (kubeflow#6010)

* Change PGP server for RStudio validation to pgp.surfnet.nl

* Add as secondary keyserver

* use eslint versions from upstream v1.3 branch

* fix eslint errors in pipelines.js

* fix: test with namespace for pipeline-cards

* fix: wrong text for test

* centraldashboard: Update node and use latest-stable (kubeflow#6260)

Change the tests to NOT fetch Chromium from the Edge branch, which is develop,
but instead use latest-stable. We saw that edge can have problems from times to
times.

In the same commit we also update the node version to fix CVEs with the current
v12.18.3 version that we had.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>
(cherry picked from commit bedda47)

* eslint auto fixes

* fix(vuln): Update base image

Update base image and fix json-schema
Comment out vulnerability scan (should be reworked to actually block)

* feat(ci): add notebook-controller workflow

* fix(ci): update notebook-controller ci (#86)

Co-authored-by: jumana-s <salwa.mohamed@canda.ca>

* fix(ci): push to acr once (#87)

Co-authored-by: jumana-s <salwa.mohamed@canda.ca>

* feat(notebook-controller): add readiness probe (#85)

* fix(npm): fix vulnerabilities

* install(packages): overcome CVEs (#90)

* install(packages): overcome CVEs
* regenerate package-lock.json
* Fix package version

Co-authored-by: Bryan Paget <bryan.paget@statcan.gc.ca>
Co-authored-by: Wendy V Gaultier <wendyvgaultier@gmail.com>

Signed-off-by: Yannis Zarkadas <yanniszark@arrikto.com>
Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>
Signed-off-by: Ilias Katsakioris <elikatsis@arrikto.com>
Signed-off-by: Yurii Komar <subreptivus@gmail.com>
Signed-off-by: Malini Bhandaru <mbhandaru@vmware.com>
Co-authored-by: Yannis Zarkadas <yanniszark@arrikto.com>
Co-authored-by: DavidSpek <vanderspek.david@gmail.com>
Co-authored-by: Jeremy Lewi <jeremy+github@lewi.us>
Co-authored-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>
Co-authored-by: Mathew Wicks <thesuperzapper@users.noreply.github.com>
Co-authored-by: Ilias Katsakioris <elikatsis@arrikto.com>
Co-authored-by: James Liu <37026441+zijianjoy@users.noreply.github.com>
Co-authored-by: toshi_k <high_luin@yahoo.co.jp>
Co-authored-by: juliusvonkohout <45896133+juliusvonkohout@users.noreply.github.com>
Co-authored-by: Saffa Alvi <alvi118@uwindsor.ca>
Co-authored-by: Wendy Gaultier <wvgaultier@gmail.com>
Co-authored-by: Jaeyeon Kim <anencore94@gmail.com>
Co-authored-by: Yurii Komar <Subreptivus@gmail.com>
Co-authored-by: JessicaBarh <jessicabarhoma@gmail.com>
Co-authored-by: Jose-Matsuda <tongster789@gmail.com>
Co-authored-by: Stephen Hopper <50119450+hopper-signifyd@users.noreply.github.com>
Co-authored-by: Malini Bhandaru <mbhandaru@vmware.com>
Co-authored-by: Filinto Duran <duranto@gmail.com>
Co-authored-by: Filinto Duran <fduran@d2iq.com>
Co-authored-by: Jiaxin Shan <seedjeffwan@gmail.com>
Co-authored-by: Frances Zsurka <franceszsurka@gmail.com>
Co-authored-by: William H <sylus1984@gmail.com>
Co-authored-by: Zachary Seguin <zachary@zacharyseguin.ca>
Co-authored-by: Zachary Seguin <zachary.seguin@canada.ca>
Co-authored-by: Brendan Gadd <brendangadd@gmail.com>
Co-authored-by: wg102 <wgaul102@uottawa.ca>
Co-authored-by: frazs <frances.zsurka@gmail.com>
Co-authored-by: Skye Turriff <turriff.skye@gmail.com>
Co-authored-by: Jose-Matsuda <jose.matsuda@canada.ca>
Co-authored-by: jumana-s <salwa.mohamed@canda.ca>
Co-authored-by: Salwa <51963397+jumana-s@users.noreply.github.com>
Co-authored-by: Wendy V Gaultier <wendyvgaultier@gmail.com>
Co-authored-by: Bryan Paget <bbrryyaann@protonmail.com>
Co-authored-by: Bryan Paget <bryan.paget@statcan.gc.ca>
chuckbelisle pushed a commit to StatCan/kubeflow that referenced this pull request Sep 20, 2022
* releasing: Add WG-Notebooks leads to release owners (#5777)

Signed-off-by: Yannis Zarkadas <yanniszark@arrikto.com>

* Specify commonLabels for tensorboard-controller (#5780)

* Remove jlewi as an approver (#5786)

Removing myself as an owner. This should lead to better auto-assignment of code reviews.

* Update notebook server base images (#5804)

* Update second layer docker images to new tags and some python deps (#5809)

* Update image tags and python packages in pytorch and tensorflow full dockerfiles (#5817)

* Add CI format checks for the Jupyter web app (#5811)

* jwa(front): Add npm rule for checking the format

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* ci: Add common tasks for format checks

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* ci(jwa): Add format check tasks

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(back): Fix formatting

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(front): Fix formatting

Run `npm run format:write` on frontend

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(front): Include prettier in package.json

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(docker): Don't copy node_modules in Dockerfile

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* web-apps: Add global dockerignore file

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa: Remove unused dockerignore file

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(make): Don't include dockerignore and cleanup

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* initial notebook server images README.md (#5818)

* tensorboard-controller: Fix scheduling unbound PVCs (#5819)

When the TB controller attempts to schedule a RWO PVC it checks its
accessModes in the PVC status. The controller panics if the list is
empty.

This commit adds a check to ensure the list is not empty.

Signed-off-by: Ilias Katsakioris <elikatsis@arrikto.com>

* Fix profile-controller CRD pruning issue (#5822)

* api: Add marker for preserving unknown fields in Plugins

Signed-off-by: Yannis Zarkadas <yanniszark@arrikto.com>

* manifests: Regenerate manifests

Signed-off-by: Yannis Zarkadas <yanniszark@arrikto.com>

* Remove Logo Trademarks from the Jupyter Web App and make logos configurable (#5823)

* rebase: Make logos configurable in configmap and remove trademark references

Rebased to remove the changes to the package-lock.json

* review: add suggested changes and add image group section to README

* Make notebook limits configurable with a multiplication factor (#5815)

* Make notebook limits configurable with a multiplication factor

* Make limits configurable under advanced section

* run prettier to format frontend code

* fix formatting and add rounding in backend

* Return error if limit is smaller than request

* Allow disabling limitFactor by setting it to none

* review: remove camelCase in python backend

* fix: update spawner_ui_config.yaml in manifests directory

* review: fix setting limits backend

* review: remove unnecessary check from backend

* notebook-servers: Update JupyterLab and add Git Extension (#5846)

* chore: Add comment to reference RStudio license. (#5884)

* CI/CD: add kustomize build tests (#5919)

* Add CI format checks for the Volumes web app (#5820)

* vwa(front): Add npm script to check the formatting

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* vwa(front): Update the package-lock.json

Run `npm install` to bring the package-lock.json up to date

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* vwa(front): Fix formatting

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* vwa(back): Fix formatting

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* vwa(docker): Remove unused dockerignore file

We have created a global dockerignore file for all the web apps in the
parent dir.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* vwa(docker): Don't copy node_modules in dockerfile

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* vwa(make): Don't include dockerignore

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* ci(vwa): Add format check tasks

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(docker): Copy only necessary files for build

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* review: Use docker context instead of cd ..

Don't use a `cd ..` and copy dockerignore files back and forth. Instead
we should use the Docker context and the global dockerignore file we
have for all the web apps.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* tensorboard-controller: fix binding issue (#5925)

* JWA: Don't override assets with logos ConfigMap (#5942)

* jwa(front): Add static logos in the app

The app does not contain the logos' svgs in its source code/static
files. This results in the icons to not show when developing locally.

This commit adds the svgs found in the logos ConfigMap to the static
files of the app as well.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(front): Change logos fetch url

Change the URLs of the logos from `static/assets/*` to
`static/assets/logos`.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(manifests): Don't override assets with logos

Mount the ConfigMap under the `static/assets/logos` directory to not
override the contents of the entire assets dir.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* Make it possible to add Namespaced menu items (#5871)

* Make it possible to add Namespaced menu items

* Reduce the nesting levels

* FIX The number of gpu must be set as string in Kubernetes/Openshift (#5891)

* Update form.py

* Update form.py

* Update form-default.component.ts

* fix whitespace issue

* feat(jupyter): add fonts as assets to service (#5691)

* cwa(front): Ignore font files in assets

* feat(jupyter): add fonts as assets to service

* CRUD: fonts in common

* CWA: Remove link to css file

* jwa(front): Remove font assets from jupyter

Co-authored-by: Wendy Gaultier <wvgaultier@gmail.com>

* Update base notebook for Renovate compatibility (#5955)

* Update base notebook for Renovate compatibility

* Separate kubectl and S6 arch

* Update jupyter, rstudio and vs code notebook images + allow rstudio in iframe (#5961)

* Update jupyter, rstudio and vs code notebook images

* Remove sed and use substring removal

* Make menu bar scrollable again (#5964)

* [fix]: Make jupyter-web-app parse workspace volume MountPath (#5952)

* [fix]: Make jupyter-web-app parse workspace volume MountPath

- workspace volume path was fixed with "/home/jovyan"
- it should be enable to parse from jupyter-web-app-config's data

* change parsing key correctly

* Remove the f-strings in the Jupyter web app's backend (#5680)

* Notebook servers: Add explanation about kernel not connecting (#5920)

* fix(profile-controller): KNative probes (#5848)

Knative control-plane components need to probe certain paths of Knative
Pods, in order to function correctly. These paths are:
- /healthz
- /metrics
- /ready
- /wait-for-drain

For this reason, we extend the Profile Controller to apply an Istio
AuthorizationPolicy that allows traffic to these HTTP paths for Pods in
user namespaces, as per Knative's instructions:
https://knative.dev/docs/serving/istio-authorization/#allowing-access-from-system-pods-by-paths

This only fixes requests through the public ingress. Cluster-internal
traffic and predictor-transformer use-cases are not supported yet. We
continue discussing these in kubeflow#5965.

Refs kubeflow#5965
Refs kubeflow/kfserving#1558

Signed-off-by: Yurii Komar <subreptivus@gmail.com>

* feat(jupyter): Support hiding image registry/tag (#5681)

* feat(jupyter): Support hiding image registry/tag

* feat(jupyter): Support hiding VSCode/RStudio images registry/tag

* rebasing onto upstream master

* fix: formatting with prettier and set default hideRegistry=true

* fix: move hideRegistry and hideTag from spawnerFormDefaults.image

* fix whitespace

* add tooltip for images

Co-authored-by: JessicaBarh <jessicabarhoma@gmail.com>

* Update dockerfiles and make compatible with Renovate (#5968)

* Update dockerfiles and make compatible with Renovate

* Set memory for jupyter pytorch to the same as jupyter tensorflow

* Update protobuf

* Remove conda version and use substring expansion

* Update SQLAlchemy

* Update dill

* Create OWNERS file in .github folder for Renovate config + workflows (#5983)

* Extend deadline for PyTorch image building (#5991)

* Allow user to add/delete labels to user namespace using ConfigMap. Fix #5712 (#5761)

The profile-controller applies a hardcoded list of labels to Profile
namespaces. Make this list of labels configurable with a ConfigMap. The
new list of steps for the profile-controller is:

1. **(new)** Profile-controller starts and reads the file given by the
   `--namespace-labels-path` CLI flag (defaults to
   `/etc/profile-controller/namespace-labels.yaml`). This file contains
   a YAML dict of label key/values to add to each Profile namespace.
2. Profile-controller creates namespace.
3. Profile-controller merges existing namespace labels with given
   labels. A given label is applied ONLY if it doesn't exist already.
4. **(new)** If a given label value is empty (``), remove the label from
   the namespace if it already exists.

We are using a ConfigMap instead of a plain CLI flag, so that users can
change the list of labels without restarting the profile-controller.

Closes #5712

Signed-off-by: Yannis Zarkadas <yanniszark@arrikto.com>

Co-authored-by: James Liu <jamxl@google.com>

Co-authored-by: Yannis Zarkadas <yanniszark@arrikto.com>

* Update last batch of notebook server Dockerfiles (#5997)

* Update spawner_ui_config.yaml with new tags (#5998)

* Update images in spawner_ui_config.yaml

* Update manifest image tags

* CRUD web apps:  i18n (#5880)

* feat(jupyter): add modules for translations

* feat(jupyter): fix path for translations + add GPU translations

* feat(jupyter): add translations

* feat(jupyter): add translations to form-default

* feat(jupyter): add translations to affinity/tolerations

* feat(jupyter): add translations to Image Pull Policy

* feat(jupyter): remove imports from app.module.ts + fix translations

* web-apps(front): add TranslationModule to common for jupyter

* Implement i18n for jupyter frontend
- Including common components needed
- Refactor translation file

* web-apps(front): add i18n to messages from backend
Status tooltip text
Snackbar message

* web-apps(front): volumes and tensorboards i18n ini
Add the i18n feature for the other 2 projects

* feat(volume): add translations for volume

* feat(jupyter): refactor asset file

* Update asset files for all folders

* feat(tensorboards): add i18n
Update all asset files

* cwa: add new translations

* cwa: Add README documentation

* fix: formatting with prettier

* Fix tests

* Fix python format

* remove comment

Co-authored-by: saffaalvi <alvi118@uwindsor.ca>
Co-authored-by: Jose-Matsuda <tongster789@gmail.com>

* Remove deprecated JWA (#5959)

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* Sync spawner yaml from source to manifests (#5999)

* notebooks: Add additional PGP server for RStudio package validation (#6010)

* Change PGP server for RStudio validation to pgp.surfnet.nl

* Add as secondary keyserver

* Add support for ServiceAccountName and AutomountServiceAccountName to admission-webhook (#5939)

* WA: Use relative paths for fetching translation files (#6034)

* jwa(front): Use relative path for i18n json

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* vwa(front): Use relative path for i18n json

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* twa(front): Use relative path for i18n json

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* Update CRUD web apps from Angular 8 to Angular 12 (#6004)

* common: update angular core and cli to 9

ng update @angular/core@9 @angular/cli@9

* common: update material, cdk and cdk-experimental to 9

ng update @angular/cdk@9 @angular/material@9 @angular/cdk-experimental@9

* common: remove entryComponents and replace TestBed.get with TestBed.inject

Edit files manually

* common: make compatible with Angular 9

Edit files manually

* common: update fontawesome to 0.6.0

npm i @fortawesome/angular-fontawesome@0.6.0

* update kubeflow-common project dependencies

* common: add @angular/localize

ng add @angular/localize

* jupyter: update angular core and cli 8

ng update @angular/core@8 @angular/cli@8

* jupyter: update angular core and cli to 9

ng update @angular/core@9 @angular/cli@9

* jupyter: update material, cdk and cdk-experimental to 9

ng update @angular/cdk@9 @angular/material@9 @angular/cdk-experimental@9

* jupyter: remove entryComponents

Edit file manually

* jupyter: update fontawesome to 0.6.0

npm i @fortawesome/angular-fontawesome@0.6.0

* jupyter: add @angular/localize

ng add @angular/localize

* tensorboards: update angular core and cli to 9

ng update @angular/core@9 @angular/cli@9

* tensorboards: update material, cdk and cdk-experimental to 9

ng update @angular/cdk@9 @angular/material@9 @angular/cdk-experimental@9

* tensorboards: remove entryComponents and replace TestBed.get with TestBed.inje…

Edit files manually

* tensorboards: update fontawesome to 0.6.0

npm i @fortawesome/angular-fontawesome@0.6.0

* tensorboards: add @angular/localize

ng add @angular/localize

* Fix tensorboards-web-app run-dev in makefile

Edit file manually

* volumes: update angular core and cli to 9

ng update @angular/core@9 @angular/cli@9

* volumes: update material, cdk and cdk-experimental to 9

ng update @angular/cdk@9 @angular/material@9 @angular/cdk-experimental@9

* volumes: remove entryComponents and replace TestBed.get with TestBe…

Edit files manually

* volumes: update fontawesome to 0.6.0

npm i @fortawesome/angular-fontawesome@0.6.0

* volumes: add @angular/localize

ng add @angular/localize

* common: update angular core and cli to 10

ng update @angular/core@10 @angular/cli@10 --force

* common: update material, cdk and cdk-experimental to 10

ng update @angular/cdk@10 @angular/material@10 @angular/cdk-experimental@10

* update kubeflow-common project dependencies

Edit file manually

* common: update fontawesome to 0.7.0

npm i @fortawesome/angular-fontawesome@0.7.0

* common: fix font location

Edit file manually

* jupyter: update angular-fontawesome to 0.7.0

npm i @fortawesome/angular-fontawesome@0.7.0

* jupyter: update angular core and cli to 10

ng update @angular/core@10 @angular/cli@10

* jupyter: update material, cdk and cdk-experimental to 10

ng update @angular/cdk@10 @angular/material@10 @angular/cdk-experimental@10

* tensorboards: update angular-fontawesome to 0.7.0

npm i @fortawesome/angular-fontawesome@0.7.0

* tensorboards: update angular core and cli to 10

ng update @angular/core@10 @angular/cli@10

* tensorboards: update material, cdk and cdk-experimental to 10

ng update @angular/cdk@10 @angular/material@10 @angular/cdk-experimental@10

* volumes: update angular-fontawesome to 0.7.0

npm i @fortawesome/angular-fontawesome@0.7.0

* volumes: update angular core and cli to 10

ng update @angular/core@10 @angular/cli@10

* volumes: update material, cdk and cdk-experimental to 10

ng update @angular/cdk@10 @angular/material@10 @angular/cdk-experimental@10

* common: update angular-fontawesome to 0.8.2

npm i @fortawesome/angular-fontawesome@0.8.2

* common: update codelyzer to 6.0.2

npm i codelyzer@6.0.2

* common: install @angular-devkit/core@11

npm install --save-dev @angular-devkit/core@11

* common: update angular core and cli to 11

ng update @angular/core@11 @angular/cli@11

* common: update material, cdk and cdk-experimental to 11

ng update @angular/cdk@11 @angular/material@11 @angular/cdk-experimental@11

* update kubeflow-common project dependencies

Edit file manually

* jupyter: update angular-fontawesome to 0.8.2

npm i @fortawesome/angular-fontawesome@0.8.2

* jupyter: update codelyzer to 6.0.2

npm i codelyzer@6.0.2

* jupyter: update angular core and cli to 11

ng update @angular/core@11 @angular/cli@11

* jupyter: update material, cdk and cdk-experimental to 11

ng update @angular/cdk@11 @angular/material@11 @angular/cdk-experimental@11

* tensorboards: update angular-fontawesome to 0.8.2

npm i @fortawesome/angular-fontawesome@0.8.2

* tensorboards: update codelyzer to 6.0.2

npm i codelyzer@6.0.2

* tensorboards: update angular core and cli to 11

ng update @angular/core@11 @angular/cli@11

* tensorboards: update material, cdk and cdk-experimental to 11

ng update @angular/cdk@11 @angular/material@11 @angular/cdk-experimental@11

* volumes: update angular-fontawesome to 0.8.2

npm i @fortawesome/angular-fontawesome@0.8.2

* volumes: update codelyzer to 6.0.2

npm i codelyzer@6.0.2

* volumes: update angular core and cli to 11

ng update @angular/core@11 @angular/cli@11

* volumes: update material, cdk and cdk-experimental to 11

ng update @angular/cdk@11 @angular/material@11 @angular/cdk-experimental@11

* common: Fix CI failure

* jupyter: npm run format:write

npm run format:write

* volumes: npm run format:write

npm run format:write

* tensorboards: manually run prettier

prettier --write 'src/**/*.{js,ts,html,scss,css}'

* common: Fix MockComponent entryComponents for CI

Manually edit file

* common: update angular-fontawesome to 0.9.0

npm i @fortawesome/angular-fontawesome@0.9.0

* common: update angular core and cli to 12

ng update @angular/core@12 @angular/cli@12

* common: update material, cdk and cdk-experimental to 12

ng update @angular/cdk@12 @angular/material@12 @angular/cdk-experimental@12

* update kubeflow-common project dependencies

Edit file manually

* common: fix Intl.DateTimeFormatOptions

Edit file manually. Type of defaultDateOptions and defaultTimeOptions needs to explicitly be set to Intl.DateTimeFormatOptions.

* jupyter: update angular-fontawesome to 0.9.0

npm i @fortawesome/angular-fontawesome@0.9.0

* jupyter: update codelyzer to 6.0.2

npm i codelyzer@6.0.2

* jupyter: update angular core and cli to 12

ng update @angular/core@12 @angular/cli@12

* jupyter: update material, cdk and cdk-experimental to 12

ng update @angular/cdk@12 @angular/material@12 @angular/cdk-experimental@12

* tensorboards: update angular-fontawesome to 0.9.0

npm i @fortawesome/angular-fontawesome@0.9.0

* tensorboards: update angular core and cli to 12

ng update @angular/core@12 @angular/cli@12

* tensorboards: update material, cdk and cdk-experimental to 12

ng update @angular/cdk@12 @angular/material@12 @angular/cdk-experimental@12

* volumes: update angular-fontawesome to 0.9.0

npm i @fortawesome/angular-fontawesome@0.9.0

* volumes: update angular core and cli to 12

ng update @angular/core@12 @angular/cli@12

* volumes: update material, cdk and cdk-experimental to 12

ng update @angular/cdk@12 @angular/material@12 @angular/cdk-experimental@12

* common: update and audit packages

npm update && npm audit fix && npm i lodash-es@4.17.21 && npm i material-icons@0.7.3

* jupyter: update and audit packages

npm update && npm audit fix && npm i material-icons@0.7.3

* tensorboards: update and audit packages

npm update && npm audit fix && npm i material-icons@0.7.3

* volumes: update and audit packages

npm update && npm audit fix && npm i material-icons@0.7.3

* jupyter: npm run format:write

npm run format:write

* lists roadmap in reverse chronological order (#6055)

Signed-off-by: Malini Bhandaru <mbhandaru@vmware.com>

* Correct missing predicates in controller watches. Fixes #5326 (#5873)

Co-authored-by: Filinto Duran <fduran@d2iq.com>

* fix(web-apps): Use Angular's i18n proposed implementation (#6065)

* jwa(front): Add i18n for french in angular.json

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(front): Add i18n rules in package.json

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(front): i18n FormImage

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(front): i18n FormAdvancedOptions

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(front): i18n FormAffinityTolerations

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(front): i18n FormConfigurations

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(front): i18n FormCpuRam

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(front): i18n FormDataVolumes

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(front): i18n FormGpus

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(front): i18n FromName

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(front): i18n FormWorkspaceVolume

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(front): i18n Volume

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* web-apps(front): i18n BackendService

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* web-apps(front): Add localize to peerDependencies

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* web-apps(front): Import $localize in library

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* web-apps(front): i18n RokService

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(front): i18n Form

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(front): i18n BackendService

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* web-apps(front): i18n ConfirmDialog

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(front): i18n Index

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* web-apps(front): i18n ResourceTable

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* web-apps(front): i18n SnackBar

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* web-apps(front): i18n Form

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* web-app(front): i18n NameNamespaceSelector

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* web-apps(front): Remove ngx-translate

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(front): i18n App

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(front): Remove ngx-translate from package.json

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(back): Don't parse keys in status

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* vwa(back): Revert backend messages

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(front): Update translation language files

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* web-apps(back): Don't parse keys in status

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* vwa(front): Add i18n for french in angular.json

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* vwa(front): Add i18n rules in package.json

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* vwa(front): i18n Form

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* vwa(front): i18n Index

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* vwa(front): i18n BackendService

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* vwa(front): i18n App

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* vwa(front): Remove ngx-translate from package.json

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* vwa(front): i18n Form

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* vwa(front): Update translation language files

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* twa(back): Revert backend messages

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* twa(front): Add i18n rules in package.json

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* twa(front): Add i18n for french in angular.json

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* twa(front): i18n Index

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* twa(front): i18n BackendService

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* twa(front): i18n App

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* twa(front): Update translation language files

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* twa(front): Remove ngx-translate from package.json

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* wa: Update READMEs

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* web-apps(front): Add localize when testing

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* [CentralDashboard] Add entry for Models web app (#6085)

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* Interact with the namespace selector (#5995)

* Interact with the namespace selector

* Use namespace given as an argument

* Keep the index of selected item

* fix(jwa): Fix limits calculation when limitFactor is none (#6058)

* jwa(front): Don't allow NaN values in limits

The UI should always catch a NaN value and don't add it in the form.
Currently this is the case for the cpu/memory limits.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(front): Limits should not be changed if dirty

If the user has manually edited the limits fields then the UI should not
try to automatically calculate them again, using the limitFactors.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* Fix NaN in Kubeflow Notebooks (#6092)

* Fix NaN in Kubeflow Notebooks

* add a validation check for NaN values

* fix: validation check if value is None

* lint: fix line length

* add section for 1.4; small updates for 1.3 section (#6082)

* add section for 1.4; small updates for 1.3 section

* refer to training operators consistently; add link to Notebooks roadmap.

* address @Bobgy feedback regarding Kubeflow Pipelines portions of 1.4 roadmap

* Remove virtualservice timeout to prevent websocket disconnect (#6126)

In the existing version, the 'timeout: 300s' added to the notebook's virtual service would cause websockets to disconnect at the 5 minute mark, causing the Jupyter Notebook web terminal function to hang. This is described in https://github.com/kubeflow/kubeflow/issues/6124.

* fix(admission-webhook): attach namespace to pod request if pod does not have it (#6052)

* fix(admission-webhook): attach namespace to pod request if pod does not have it

* Apply suggestions from code review

Add more explicit logs

Co-authored-by: Erhan Kesken <ekesken@users.noreply.github.com>

* Add more explicit log

Co-authored-by: Erhan Kesken <ekesken@users.noreply.github.com>

Co-authored-by: Erhan Kesken <ekesken@users.noreply.github.com>

* Fix notebook base image gpg keys (#6138)

* update links for new testing infra (#6144)

* Add more root OWNERS (#6146)

* Improve README for example-notebook-servers (#6165)

* jwa(front): Don't disable vendors with no GPUs (#6171)

JWA should not block users from selecting GPUs if the current cluster
nodes do not have any GPUs attached to them. We've seen users that have
autoscaled nodegroups for GPUs, so a GPU node will be added to the
cluster once a Pod has requested it.

Refs: arrikto/dev#1484

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* Bump Golang version in PodDefaults, TensorBoard Controller and KFAM to 1.17 (#6180)

* kfam: Upgrade go to 1.17

Update to a more recent docker image that has a newer version of
openssl.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* poddefaults: Upgrade go to 1.17

Update to a more recent docker image that has a newer version of
openssl.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* tensorboards: Upgrade go to 1.17

Update to a more recent docker image that has a newer version of
openssl.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* Update Notebooks README regarding custom packages (#6176)

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* Fix reference to PodPreset as PodDefault (#6184)

* fix: tensorboard-controller is killed due to out of memory (#6148)

* Update manager.yaml

* Update manager.yaml

* fix(web-apps): Use relative path for --deploy-url build. #5566 (#6206)

* fix(web-apps): Use relative path for --base-href build

* volume web app should use volume base-href

* Document for cutting Kubeflow website release. (#6198)

* Document for cutting Kubeflow website release.

* Update docs_dev/releasing.md

Co-authored-by: Shannon Bradshaw <shannon.bradshaw@arrikto.com>

* Update docs_dev/releasing.md

Co-authored-by: Shannon Bradshaw <shannon.bradshaw@arrikto.com>

* address comment

* canonical name

Co-authored-by: Shannon Bradshaw <shannon.bradshaw@arrikto.com>

* Change namespace label for Katib metrics collector injection (#6225)

* fix(backend): tensorboard-controller does not work because of missing permissions (#6216)

* add(web-apps): i18n Fr translations (#6122)

* chore(docs): It's better to refer to the cli documents, not repo's readme (#6239)

* fix(docs): Fix the 404 K8s docs link (#6236)

* nit: typo in README.md (#6211)

* web-apps(front) : change error msg for min (#6193)

* Centraldashboard/add uncaughtexception handler (#6203)

* Central dashboard - Add global uncaught exception handler

* fix comments

* Fix(frontend): Upgrade ts node to v4 (#6242)

* Upgrade ts-node to v10

* Update lock

* admission-webhook-manifests: Update the cert manager from v1alpha2 to v1 (#6230)

* Short term updates to Published Kubeflow Roadmap (#6248)

3 changes - Add info on Kubeflow 1.5 and 1.4.1.   Update roadmap to show that Kubeflow 1.4 was delivered.

* Upgrade the google cloud monitoring node.js client (#6210)

* Upgrade the google cloud monitoring node.js client

* Update package-lock.json

* Upgrade TypeScript to v4

* Update package-lock.json

* run tslint

* fix dashboard sending malformed headers to kfam (#6186)

* correct typo - track to tracked (#6264)

corrected typo "track" to "tracked"

* centraldashboard: Update node and use latest-stable (#6260)

Change the tests to NOT fetch Chromium from the Edge branch, which is develop,
but instead use latest-stable. We saw that edge can have problems from times to
times.

In the same commit we also update the node version to fix CVEs with the current
v12.18.3 version that we had.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* Tell user to select a namespace to access namespaced menu item (#6181)

* Fix unnecessary updates of istio AuthorizationPolicy (#6269)

* Introduce a basic proposals flow for more advanced features (#6272)

* Introduce a proposals dir for Notebooks

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* Move the proposals dir under components

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* Secure access to KFAM (#6077)

* fix(notebooks) make culling work with multi-user (#5128) (#5980)

* JWA: Very basic E2E tests to run locally (#6005)

* web-apps(front): Add test attrs selectors

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(front): Update gitignore for cypress files

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(front): Install cypress & npm scripts for e2e

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(e2e): Add first cypress tests

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa: Update the README for e2e instructions

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* chore: remove Bobgy from OWNERS

* frontend: Use responsive tables instead of cards (#6316)

* jwa(front): Make ng lint work by ignoring e2e/tsconfig.json

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>
Reviewed-by: Tasos Alexiou <tasos@arrikto.com>

* web-apps(front): Make the table responsive

We introduced all of our changes in the underlying TableComponent. This
component can then be either used independently or inside a material
card.

The changes we did in that component are:
1. Create Output() emitters, since this component can be used directly
2. The config object for a Table row now supports a `style` prop for
   defining the list of CSS styles to be applied
2. Remove the truncate classes (small, medium, large) and only have
   a boolean value. The user can define the width directly now via the
   `style` property in the row's config
3. Modify the classes for aligning contents right and left

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>
Reviewed-by: Tasos Alexiou <tasos@arrikto.com>

* web-apps(front): Add table paginator

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>
Reviewed-by: Tasos Alexiou <tasos@arrikto.com>

* web-apps(front): Add padding to titlebar text

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>
Reviewed-by: Tasos Alexiou <tasos@arrikto.com>

* web-apps(front): Use bigger font for toolbar title

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>
Reviewed-by: Tasos Alexiou <tasos@arrikto.com>

* jwa(front): Use responsive table and toolbar

We refactor both the index and form pages of the app to:
1. Add a top row toolbar with the title of the app and the button to
   create a new Notebook
2. Replace the card with a responsive table that shows the items. The
   component also has a paginator

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>
Reviewed-by: Tasos Alexiou <tasos@arrikto.com>

* vwa(front): Use responsive table and toolbar

This commit:
1. Adds a toolbar at the top of the index page with the title of the app
   and the button to create a new volume
2. Replaces the card with a responsive table

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>
Reviewed-by: Tasos Alexiou <tasos@arrikto.com>

* twa(front): Use responsive table and toolbar

This commit:
1. Adds a toolbar at the top of the index page with the title of the app
   and a button to create a new TensorBoard instance
2. Replaces the card with a resopnsive table

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>
Reviewed-by: Tasos Alexiou <tasos@arrikto.com>

* fix the format

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* wa(front): Add npm script for running unit tests in docker

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* fix unit tests failing

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* Don't use asterisk in Dockerfiles (#6318)

* wa(docker): Don't copy node_modules

To ensure Kaniko is not copying stale node_moduels folders, even though
we have a dockerignore file, we are explicitly only copying the source
code.

We have seen the build system with Kaniko to fail, due to NFS stale
instances with files in node_modules and expect that this is the root
cause.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* twa: Update makefile to use global dockerignore

The dockerfile for TWA was copying over the local dockerignore. This was
overriding the global one we had for all the web apps.

This commit updates the Makefile of the app to use the global
dockerignore that all the apps should use.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* Bump log4js from 6.3.0 to 6.4.1 in /components/crud-web-apps/common/frontend/kubeflow-common-lib (#6309)

Bumps [log4js](https://github.com/log4js-node/log4js-node) from 6.3.0 to 6.4.1.
- [Release notes](https://github.com/log4js-node/log4js-node/releases)
- [Changelog](https://github.com/log4js-node/log4js-node/blob/master/CHANGELOG.md)
- [Commits](https://github.com/log4js-node/log4js-node/compare/v6.3.0...v6.4.1)

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

Signed-off-by: dependabot[bot] <support@github.com>

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

* Bump postcss from 7.0.16 to 7.0.39 in /components/centraldashboard (#6308)

Bumps [postcss](https://github.com/postcss/postcss) from 7.0.16 to 7.0.39.
- [Release notes](https://github.com/postcss/postcss/releases)
- [Changelog](https://github.com/postcss/postcss/blob/7.0.39/CHANGELOG.md)
- [Commits](https://github.com/postcss/postcss/compare/7.0.16...7.0.39)

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

Signed-off-by: dependabot[bot] <support@github.com>

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

* Bump log4js from 6.3.0 to 6.4.1 in /components/crud-web-apps/tensorboards/frontend (#6311)

Bumps [log4js](https://github.com/log4js-node/log4js-node) from 6.3.0 to 6.4.1.
- [Release notes](https://github.com/log4js-node/log4js-node/releases)
- [Changelog](https://github.com/log4js-node/log4js-node/blob/master/CHANGELOG.md)
- [Commits](https://github.com/log4js-node/log4js-node/compare/v6.3.0...v6.4.1)

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

Signed-off-by: dependabot[bot] <support@github.com>

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

* Bump log4js from 6.3.0 to 6.4.1 in /components/crud-web-apps/volumes/frontend (#6310)

Bumps [log4js](https://github.com/log4js-node/log4js-node) from 6.3.0 to 6.4.1.
- [Release notes](https://github.com/log4js-node/log4js-node/releases)
- [Changelog](https://github.com/log4js-node/log4js-node/blob/master/CHANGELOG.md)
- [Commits](https://github.com/log4js-node/log4js-node/compare/v6.3.0...v6.4.1)

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

Signed-off-by: dependabot[bot] <support@github.com>

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

* Bump log4js from 6.3.0 to 6.4.1 in /components/crud-web-apps/jupyter/frontend (#6312)

Bumps [log4js](https://github.com/log4js-node/log4js-node) from 6.3.0 to 6.4.1.
- [Release notes](https://github.com/log4js-node/log4js-node/releases)
- [Changelog](https://github.com/log4js-node/log4js-node/blob/master/CHANGELOG.md)
- [Commits](https://github.com/log4js-node/log4js-node/compare/v6.3.0...v6.4.1)

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

Signed-off-by: dependabot[bot] <support@github.com>

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

* Update images for the 1.5 rc0 release (#6319)

* Update the releasing version tag

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* Run automated script for updating versions

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* Fix(manifests): Upgrade rbac.authorization.k8s.io from v1beta1 to v1 (#6261)

* proposal: Extend Notebook Controller to expose idleness for Jupyter (#6295)

* proposal: Extend Notebook Controller to expose idleness for Jupyter (#6270)

Provide a design doc as a proposal for extending Notebook Controller to
expose idleness for Jupyter. Our proposal is in markdown format and follows
the guidelines of the kubeflow/components/proposal/README.md guide.

You can view the #6270 issue in the following link:
https://github.com/kubeflow/kubeflow/issues/6270

Signed-off-by: Athanasios Markou <athamark@arrikto.com>

* review: change the title of the proposal

Change the title of the proposal to only include the
proposed new feature. The new title of the proposal
will now be "Expose Idleness Information for Jupyter
Notebooks".

* review: rename the proposal markdown file

We want to give a more specific name to the markdown
which contains the proposal. Since this proposal
emphasizes on a feature regarding the Jupyter Notebooks,
the new name will be:

20220121-jupyter-notebook-idleness.md

* Synchronize jupyter-web-application role with clusterrole (#6241)

* Update role.yaml

* Update role.yaml

* Update cluster-role.yaml

* Kubeflow Roadmap update - with 1.5 details (#6266)

* Kubeflow Roadmap update - with 1.5 details

These proposed changes include: identifying that 1.4.1 has been delivered, provides themes for 1.5 and provides details of major features in 1.5 by working group.   This is an initial proposal that needs review by the working group leads.

* correct formatting in KFP features

Moved KFP features under KFP Control Flow doc

* updating KFP section

updating KFP references with updates from KFP team

* Updated the 1.5 release date to March

updated the 1.5 release date to March

* Update ROADMAP.md

change Hyperparameter leader election to Katib leader election

Co-authored-by: Andrey Velichkevich <andrey.velichkevich@gmail.com>

* Update ROADMAP.md

improve description and details of feature for metrics collector

Co-authored-by: Andrey Velichkevich <andrey.velichkevich@gmail.com>

* Update Katib description for Early stopping in 1.5

updating with Andrey's suggestion (but without the world proper).   * Validation for Early Stopping algorithm settings helps users to proper reduce model overfitting

Co-authored-by: Andrey Velichkevich <andrey.velichkevich@gmail.com>

* notebooks: Extend Notebook Controller to expose idleness for Jupyter (#6297)

* notebooks: Update image's tag in make

Modify Makefile to update properly the TAG
based on the git TAG.

Signed-off-by: Athanasios Markou <athamark@arrikto.com>
Reviewed-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* notebooks: Expose last-activity

Extend the notebook-controller to:
* cull idle Notebook Servers based on their new `last-activity`
  annotation
* expose the last activity of each Notebook Server as an annotation
  on the metadata of the corresponding CR object

Modify notebook_controller.go to:
* update the Last Activity of each Notebook Server that has a
  Running pod
* delete the Last Activity Annotation for every Notebook Server
  that does not have a Running pod

Extend culler.go to:
* perform culling based on the new `last-activity` annotation and
  not based on the `/api/status` endpoint.
* update the last activity of a Notebook Server, based on the
  kernels' execution states.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>
Reviewed-by: Athanasios Markou <athamark@arrikto.com>

* notebooks: Introduce a DEV env var

We introduce a DEV ENV var to allow admins
develop and test on their local machine their
custom Notebook Controller.
We provide information and instructions inside
the components/notebook-controller/README.md.

Signed-off-by: Athanasios Markou <athamark@arrikto.com>
Reviewed-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* notebooks: Add unit tests for last-activity

* Introduce new tests for allKernelsAreIdle()
* Extend the tests for NotebookIsIdle() and for
  NotebookNeedsCulling().

Signed-off-by: Athanasios Markou <athamark@arrikto.com>
Reviewed-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* review: UpdateNotebookLastActivityAnnotation()

Ensure that UpdateNotebookLastActivityAnnotation() does not return
"true". This function should not return any value.

Signed-off-by: Athanasios Markou <athamark@arrikto.com>

* jwa: Rework the Storage API of the web app (#6321)

* wa(back): Add helper for deserializing JSON obj

In some cases we might need to construct Python k8s lib objects from the
JSONs that are provided by clients. I.e. the UI will be sending a PVC
object in json format, so the backend will need to create the
corresponding client.V1PersistentVolumeClaim object and submit it.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>
Reviewed-by: Ilias Katsakioris <elikatsis@arrikto.com>

* wa(back): Serialization helper

Add helper function for converting a k8s-client object into a dict that
can be sent as an HTTP response.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>
Reviewed-by: Ilias Katsakioris <elikatsis@arrikto.com>

* wa(back): Add dry run to Notebooks and PVCs

The backend will need to be able to create objects with dry-run, in
order to ensure they are valid. The backend will need to check that both
the Notebook and the PVCs can be created beforehand.

This way we avoid the scenario where we create PVCs but the Notebook
fails to be created, and the PVCs are never garbage collected.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>
Reviewed-by: Ilias Katsakioris <elikatsis@arrikto.com>

* wa(back): Update kubernetes to 0.17

In order to support dry-run we must use the 0.17 version of the Python
k8s client.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>
Reviewed-by: Ilias Katsakioris <elikatsis@arrikto.com>

* wa(back): Extend api module to patch pvcs

The backend will need to be able to PATCH PVCs in order to set the
ownerReference to the Notebook that mounts the PVCs.

Ref: arrikto/dev/issues/386#issuecomment-856700392

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>
Reviewed-by: Ilias Katsakioris <elikatsis@arrikto.com>

* jwa(back): Work with new Volumes API

The backend API should not add any more layers of abstractions on top of
the K8s API. The backend should expect the client/UI to be sending the
entire PVC spec of a new PVC.

Refs: arrikto/dev/issues/386

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>
Reviewed-by: Ilias Katsakioris <elikatsis@arrikto.com>

* jwa(back): Add unittests for new volumes API

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>
Reviewed-by: Ilias Katsakioris <elikatsis@arrikto.com>

* jwa(back): Extend the PVC info returned

We want to show both the access mode and size of the existing PVCs, when
a user clicks on the dropdown to select which PVC to mount.

The backend will need to provide this information to the frontend. We
don't want to send the K8s list of PVCs since this will result in a lot
of unnecessary data to be sent.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>
Reviewed-by: Ilias Katsakioris <elikatsis@arrikto.com>

* jwa(front): Add proxy config for Rok

When developing the Rok flavor locally we will need to be able to open
the Rok chooser. This can be done by using Angular/webpack proxy to
bring the exposed rok service and the app under the same domain.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>
Reviewed-by: Tasos Alexiou <tasos@arrikto.com>

* jwa(front): Remove card from form

The form of the app should not be a big card, but a normal form.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>
Reviewed-by: Tasos Alexiou <tasos@arrikto.com>

* jwa(front): Install AceModule for yaml editing

Install AceModule to allow users to edit yamls of objects.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>
Reviewed-by: Tasos Alexiou <tasos@arrikto.com>

* wa(front): Change the styling of form sections

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>
Reviewed-by: Tasos Alexiou <tasos@arrikto.com>

* jwa(front): Create common volume components

Component for:
* New PVC and configuring its spec
* Attaching an existing PVC in a Notebook

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>
Reviewed-by: Tasos Alexiou <tasos@arrikto.com>

* jwa(front): Update Rok form for new Volume API

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>
Reviewed-by: Tasos Alexiou <tasos@arrikto.com>

* jwa(front): Mark inputs as dirty when restoring Lab

When the UI autofills the form with values from a JupyterLab snapshot
then it should mark the touched fields as dirty. This way if a field has
errors the UI will make that input red.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>
Reviewed-by: Tasos Alexiou <tasos@arrikto.com>

* jwa: Update ConfigMap in manifests

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(front): Fix format errors

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* profiles: Update the permissions for notebook idleness (#6335)

Extend the Profiles Controller to give permissions to Notebooks
controller for making GET requests to notebook's /api/kernels endpoint.

Refs https://github.com/kubeflow/kubeflow/blob/master/components/proposals/20220121-jupyter-notebook-idleness.md

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* notebooks: Graceful handling of events (#6338)

* notebooks: Handle events gracefully

The controller is not exiting the reconciliation loop after it has
re-emitted a Pod/STS Event as a Notebook Event. This results in the
controller to later on try and GET a Notebook with the name of the Event
that triggered the reconciliation loop.

The controller should exit the reconciliation function once it has
emitted the event.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* notebooks: Don't reconcile on deleted events

We don't want to trigger the reconciliation function when an event gets
deleted.

If a Notebook would be deleted then the underlying events would
be deleted as well, which results in the reconcile function to get
triggered and try to GET Events and Notebooks with the name of the
deleted event.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* notebooks: Fix endless restarts (#6341)

* notebooks: Update notebook if timestamp changed

We don't want to be updating the spec of the notebook if the timestamp
hasn't changed, since this will lead to constant updates and
reconciliation loops.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* notebooks: Use a deep-copy of the notebook spec

The controller should use a deep-copy of the notebook spec when
calculating the spec for the StatefulSet. If not then we could
update the notebook object without wanting it, since the spec could have
been changed when calculating the STS spec.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* notebooks: Add prefix env var only if missing

The controller should be setting OR updating the NB_PREFIX env var.
Previously it would always blindly append it to the spec, which could
result in double entries for the same env var.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* releasing: Update tags for v1.5.0-rc.1 (#6343)

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* fix error comparison between pointer and pointer in "CopyStatefulSetFields" (#6195)

error comparison between pointer and pointer in "CopyStatefulSetFields"

* [bugfix]: update unsupported links to latest pages (#6192)

* centraldashboard: Add KServe overlay (#6383)

Configure the dashboard to use the KServe app instead of the KFServing
0.6.1 one.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* notebooks: Don't reconcile on Events deletion (#6391)

The controller should not trigger the reconcile loop when an Event is
deleted. Previously the controller would run the reconciliation loop on
any event deletion.

This commit updates it to not run the loop for ANY event.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* releasing: Create v1.5.0-rc.2 images (#6394)

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* release: Images for the 1.5.0 tag (#6398)

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* added env for app secure cookies (#6395)

* build(deps): bump github.com/gogo/protobuf from 1.1.1 to 1.3.2 in /components/tensorboard-controller (#6424)

Bumps [github.com/gogo/protobuf](https://github.com/gogo/protobuf) from 1.1.1 to 1.3.2.
- [Release notes](https://github.com/gogo/protobuf/releases)
- [Commits](https://github.com/gogo/protobuf/compare/v1.1.1...v1.3.2)

---
updated-dependencies:
- dependency-name: github.com/gogo/protobuf
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <support@github.com>

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

* build(deps): bump notebook from 6.4.0 to 6.4.10 in /components/example-notebook-servers/jupyter (#6431)

Bumps [notebook](http://jupyter.org) from 6.4.0 to 6.4.10.

---
updated-dependencies:
- dependency-name: notebook
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <support@github.com>

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

* build(deps): bump minimist from 1.2.5 to 1.2.6 in /components/crud-web-apps/tensorboards/frontend (#6417)

Bumps [minimist](https://github.com/substack/minimist) from 1.2.5 to 1.2.6.
- [Release notes](https://github.com/substack/minimist/releases)
- [Commits](https://github.com/substack/minimist/compare/1.2.5...1.2.6)

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

Signed-off-by: dependabot[bot] <support@github.com>

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

* build(deps): bump minimist from 1.2.5 to 1.2.6 in /components/crud-web-apps/jupyter/frontend (#6429)

Bumps [minimist](https://github.com/substack/minimist) from 1.2.5 to 1.2.6.
- [Release notes](https://github.com/substack/minimist/releases)
- [Commits](https://github.com/substack/minimist/compare/1.2.5...1.2.6)

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

Signed-off-by: dependabot[bot] <support@github.com>

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

* build(deps): bump follow-redirects from 1.14.1 to 1.14.8 in /components/crud-web-apps/tensorboards/frontend (#6357)

Bumps [follow-redirects](https://github.com/follow-redirects/follow-redirects) from 1.14.1 to 1.14.8.
- [Release notes](https://github.com/follow-redirects/follow-redirects/releases)
- [Commits](https://github.com/follow-redirects/follow-redirects/compare/v1.14.1...v1.14.8)

---
updated-dependencies:
- dependency-name: follow-redirects
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <support@github.com>

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

* build(deps): bump ajv from 6.10.2 to 6.12.6 in /components/crud-web-apps/volumes/frontend (#6358)

Bumps [ajv](https://github.com/ajv-validator/ajv) from 6.10.2 to 6.12.6.
- [Release notes](https://github.com/ajv-validator/ajv/releases)
- [Commits](https://github.com/ajv-validator/ajv/compare/v6.10.2...v6.12.6)

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

Signed-off-by: dependabot[bot] <support@github.com>

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

* build(deps): bump ajv from 6.10.2 to 6.12.6 in /components/crud-web-apps/jupyter/frontend (#6359)

Bumps [ajv](https://github.com/ajv-validator/ajv) from 6.10.2 to 6.12.6.
- [Release notes](https://github.com/ajv-validator/ajv/releases)
- [Commits](https://github.com/ajv-validator/ajv/compare/v6.10.2...v6.12.6)

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

Signed-off-by: dependabot[bot] <support@github.com>

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

* build(deps): bump lodash-es from 4.17.11 to 4.17.21 in /components/crud-web-apps/tensorboards/frontend (#6402)

Bumps [lodash-es](https://github.com/lodash/lodash) from 4.17.11 to 4.17.21.
- [Release notes](https://github.com/lodash/lodash/releases)
- [Commits](https://github.com/lodash/lodash/compare/4.17.11...4.17.21)

---
updated-dependencies:
- dependency-name: lodash-es
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <support@github.com>

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

* build(deps): bump minimist from 1.2.5 to 1.2.6 in /components/crud-web-apps/common/frontend/kubeflow-common-lib (#6416)

Bumps [minimist](https://github.com/substack/minimist) from 1.2.5 to 1.2.6.
- [Release notes](https://github.com/substack/minimist/releases)
- [Commits](https://github.com/substack/minimist/compare/1.2.5...1.2.6)

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

Signed-off-by: dependabot[bot] <support@github.com>

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

* build(deps): bump minimist from 1.2.5 to 1.2.6 in /components/crud-web-apps/volumes/frontend (#6423)

Bumps [minimist](https://github.com/substack/minimist) from 1.2.5 to 1.2.6.
- [Release notes](https://github.com/substack/minimist/releases)
- [Commits](https://github.com/substack/minimist/compare/1.2.5...1.2.6)

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

Signed-off-by: dependabot[bot] <support@github.com>

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

* build(deps): bump undefsafe from 2.0.2 to 2.0.5 in /components/centraldashboard (#6342)

Bumps [undefsafe](https://github.com/remy/undefsafe) from 2.0.2 to 2.0.5.
- [Release notes](https://github.com/remy/undefsafe/releases)
- [Commits](https://github.com/remy/undefsafe/compare/v2.0.2...v2.0.5)

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

Signed-off-by: dependabot[bot] <support@github.com>

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

* build(deps): bump url-parse from 1.5.1 to 1.5.10 in /components/crud-web-apps/common/frontend/kubeflow-common-lib (#6378)

Bumps [url-parse](https://github.com/unshiftio/url-parse) from 1.5.1 to 1.5.10.
- [Release notes](https://github.com/unshiftio/url-parse/releases)
- [Commits](https://github.com/unshiftio/url-parse/compare/1.5.1...1.5.10)

---
updated-dependencies:
- dependency-name: url-parse
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <support@github.com>

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

* build(deps): bump url-parse from 1.5.1 to 1.5.10 in /components/crud-web-apps/tensorboards/frontend (#6379)

Bumps [url-parse](https://github.com/unshiftio/url-parse) from 1.5.1 to 1.5.10.
- [Release notes](https://github.com/unshiftio/url-parse/releases)
- [Commits](https://github.com/unshiftio/url-parse/compare/1.5.1...1.5.10)

---
updated-dependencies:
- dependency-name: url-parse
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <support@github.com>

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

* build(deps-dev): bump karma from 6.3.4 to 6.3.16 in /components/crud-web-apps/common/frontend/kubeflow-common-lib (#6386)

Bumps [karma](https://github.com/karma-runner/karma) from 6.3.4 to 6.3.16.
- [Release notes](https://github.com/karma-runner/karma/releases)
- [Changelog](https://github.com/karma-runner/karma/blob/master/CHANGELOG.md)
- [Commits](https://github.com/karma-runner/karma/compare/v6.3.4...v6.3.16)

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

Signed-off-by: dependabot[bot] <support@github.com>

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

* build(deps-dev): bump karma from 6.3.4 to 6.3.16 in /components/crud-web-apps/tensorboards/frontend (#6387)

Bumps [karma](https://github.com/karma-runner/karma) from 6.3.4 to 6.3.16.
- [Release notes](https://github.com/karma-runner/karma/releases)
- [Changelog](https://github.com/karma-runner/karma/blob/master/CHANGELOG.md)
- [Commits](https://github.com/karma-runner/karma/compare/v6.3.4...v6.3.16)

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

Signed-off-by: dependabot[bot] <support@github.com>

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

* build(deps): bump moment from 2.24.0 to 2.29.2 in /components/centraldashboard (#6432)

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

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

Signed-off-by: dependabot[bot] <support@github.com>

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

* build(deps): bump url-parse from 1.5.1 to 1.5.10 in /components/crud-web-apps/volumes/frontend (#6380)

Bumps [url-parse](https://github.com/unshiftio/url-parse) from 1.5.1 to 1.5.10.
- [Release notes](https://github.com/unshiftio/url-parse/releases)
- [Commits](https://github.com/unshiftio/url-parse/compare/1.5.1...1.5.10)

---
updated-dependencies:
- dependency-name: url-parse
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <support@github.com>

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

* build(deps): bump url-parse from 1.4.7 to 1.5.10 in /components/centraldashboard (#6377)

Bumps [url-parse](https://github.com/unshiftio/url-parse) from 1.4.7 to 1.5.10.
- [Release notes](https://github.com/unshiftio/url-parse/releases)
- [Commits](https://github.com/unshiftio/url-parse/compare/1.4.7...1.5.10)

---
updated-dependencies:
- dependency-name: url-parse
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <support@github.com>

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

* build(deps-dev): bump karma from 6.3.4 to 6.3.16 in /components/crud-web-apps/volumes/frontend (#6388)

Bumps [karma](https://github.com/karma-runner/karma) from 6.3.4 to 6.3.16.
- [Release notes](https://github.com/karma-runner/karma/releases)
- [Changelog](https://github.com/karma-runner/karma/blob/master/CHANGELOG.md)
- [Commits](https://github.com/karma-runner/karma/compare/v6.3.4...v6.3.16)

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

Signed-off-by: dependabot[bot] <support@github.com>

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

* build(deps-dev): bump karma from 6.3.4 to 6.3.16 in /components/crud-web-apps/jupyter/frontend (#6389)

Bumps [karma](https://github.com/karma-runner/karma) from 6.3.4 to 6.3.16.
- [Release notes](https://github.com/karma-runner/karma/releases)
- [Changelog](https://github.com/karma-runner/karma/blob/master/CHANGELOG.md)
- [Commits](https://github.com/karma-runner/karma/compare/v6.3.4...v6.3.16)

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

Signed-off-by: dependabot[bot] <support@github.com>

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

* build(deps): bump follow-redirects from 1.14.1 to 1.14.9 in /components/crud-web-apps/jupyter/frontend (#6441)

Bumps [follow-redirects](https://github.com/follow-redirects/follow-redirects) from 1.14.1 to 1.14.9.
- [Release notes](https://github.com/follow-redirects/follow-redirects/releases)
- [Commits](https://github.com/follow-redirects/follow-redirects/compare/v1.14.1...v1.14.9)

---
updated-dependencies:
- dependency-name: follow-redirects
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <support@github.com>

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

* build(deps): bump nanoid from 3.1.23 to 3.3.2 in /components/crud-web-apps/common/frontend/kubeflow-common-lib (#6437)

Bumps [nanoid](https://github.com/ai/nanoid) from 3.1.23 to 3.3.2.
- [Release notes](https://github.com/ai/nanoid/releases)
- [Changelog](https://github.com/ai/nanoid/blob/main/CHANGELOG.md)
- [Commits](https://github.com/ai/nanoid/compare/3.1.23.…
chuckbelisle pushed a commit to StatCan/kubeflow that referenced this pull request Sep 22, 2022
* releasing: Add WG-Notebooks leads to release owners (#5777)

Signed-off-by: Yannis Zarkadas <yanniszark@arrikto.com>

* Specify commonLabels for tensorboard-controller (#5780)

* Remove jlewi as an approver (#5786)

Removing myself as an owner. This should lead to better auto-assignment of code reviews.

* Update notebook server base images (#5804)

* Update second layer docker images to new tags and some python deps (#5809)

* Update image tags and python packages in pytorch and tensorflow full dockerfiles (#5817)

* Add CI format checks for the Jupyter web app (#5811)

* jwa(front): Add npm rule for checking the format

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* ci: Add common tasks for format checks

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* ci(jwa): Add format check tasks

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(back): Fix formatting

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(front): Fix formatting

Run `npm run format:write` on frontend

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(front): Include prettier in package.json

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(docker): Don't copy node_modules in Dockerfile

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* web-apps: Add global dockerignore file

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa: Remove unused dockerignore file

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(make): Don't include dockerignore and cleanup

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* initial notebook server images README.md (#5818)

* tensorboard-controller: Fix scheduling unbound PVCs (#5819)

When the TB controller attempts to schedule a RWO PVC it checks its
accessModes in the PVC status. The controller panics if the list is
empty.

This commit adds a check to ensure the list is not empty.

Signed-off-by: Ilias Katsakioris <elikatsis@arrikto.com>

* Fix profile-controller CRD pruning issue (#5822)

* api: Add marker for preserving unknown fields in Plugins

Signed-off-by: Yannis Zarkadas <yanniszark@arrikto.com>

* manifests: Regenerate manifests

Signed-off-by: Yannis Zarkadas <yanniszark@arrikto.com>

* Remove Logo Trademarks from the Jupyter Web App and make logos configurable (#5823)

* rebase: Make logos configurable in configmap and remove trademark references

Rebased to remove the changes to the package-lock.json

* review: add suggested changes and add image group section to README

* Make notebook limits configurable with a multiplication factor (#5815)

* Make notebook limits configurable with a multiplication factor

* Make limits configurable under advanced section

* run prettier to format frontend code

* fix formatting and add rounding in backend

* Return error if limit is smaller than request

* Allow disabling limitFactor by setting it to none

* review: remove camelCase in python backend

* fix: update spawner_ui_config.yaml in manifests directory

* review: fix setting limits backend

* review: remove unnecessary check from backend

* notebook-servers: Update JupyterLab and add Git Extension (#5846)

* chore: Add comment to reference RStudio license. (#5884)

* CI/CD: add kustomize build tests (#5919)

* Add CI format checks for the Volumes web app (#5820)

* vwa(front): Add npm script to check the formatting

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* vwa(front): Update the package-lock.json

Run `npm install` to bring the package-lock.json up to date

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* vwa(front): Fix formatting

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* vwa(back): Fix formatting

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* vwa(docker): Remove unused dockerignore file

We have created a global dockerignore file for all the web apps in the
parent dir.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* vwa(docker): Don't copy node_modules in dockerfile

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* vwa(make): Don't include dockerignore

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* ci(vwa): Add format check tasks

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(docker): Copy only necessary files for build

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* review: Use docker context instead of cd ..

Don't use a `cd ..` and copy dockerignore files back and forth. Instead
we should use the Docker context and the global dockerignore file we
have for all the web apps.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* tensorboard-controller: fix binding issue (#5925)

* JWA: Don't override assets with logos ConfigMap (#5942)

* jwa(front): Add static logos in the app

The app does not contain the logos' svgs in its source code/static
files. This results in the icons to not show when developing locally.

This commit adds the svgs found in the logos ConfigMap to the static
files of the app as well.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(front): Change logos fetch url

Change the URLs of the logos from `static/assets/*` to
`static/assets/logos`.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(manifests): Don't override assets with logos

Mount the ConfigMap under the `static/assets/logos` directory to not
override the contents of the entire assets dir.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* Make it possible to add Namespaced menu items (#5871)

* Make it possible to add Namespaced menu items

* Reduce the nesting levels

* FIX The number of gpu must be set as string in Kubernetes/Openshift (#5891)

* Update form.py

* Update form.py

* Update form-default.component.ts

* fix whitespace issue

* feat(jupyter): add fonts as assets to service (#5691)

* cwa(front): Ignore font files in assets

* feat(jupyter): add fonts as assets to service

* CRUD: fonts in common

* CWA: Remove link to css file

* jwa(front): Remove font assets from jupyter

Co-authored-by: Wendy Gaultier <wvgaultier@gmail.com>

* Update base notebook for Renovate compatibility (#5955)

* Update base notebook for Renovate compatibility

* Separate kubectl and S6 arch

* Update jupyter, rstudio and vs code notebook images + allow rstudio in iframe (#5961)

* Update jupyter, rstudio and vs code notebook images

* Remove sed and use substring removal

* Make menu bar scrollable again (#5964)

* [fix]: Make jupyter-web-app parse workspace volume MountPath (#5952)

* [fix]: Make jupyter-web-app parse workspace volume MountPath

- workspace volume path was fixed with "/home/jovyan"
- it should be enable to parse from jupyter-web-app-config's data

* change parsing key correctly

* Remove the f-strings in the Jupyter web app's backend (#5680)

* Notebook servers: Add explanation about kernel not connecting (#5920)

* fix(profile-controller): KNative probes (#5848)

Knative control-plane components need to probe certain paths of Knative
Pods, in order to function correctly. These paths are:
- /healthz
- /metrics
- /ready
- /wait-for-drain

For this reason, we extend the Profile Controller to apply an Istio
AuthorizationPolicy that allows traffic to these HTTP paths for Pods in
user namespaces, as per Knative's instructions:
https://knative.dev/docs/serving/istio-authorization/#allowing-access-from-system-pods-by-paths

This only fixes requests through the public ingress. Cluster-internal
traffic and predictor-transformer use-cases are not supported yet. We
continue discussing these in kubeflow#5965.

Refs kubeflow#5965
Refs kubeflow/kfserving#1558

Signed-off-by: Yurii Komar <subreptivus@gmail.com>

* feat(jupyter): Support hiding image registry/tag (#5681)

* feat(jupyter): Support hiding image registry/tag

* feat(jupyter): Support hiding VSCode/RStudio images registry/tag

* rebasing onto upstream master

* fix: formatting with prettier and set default hideRegistry=true

* fix: move hideRegistry and hideTag from spawnerFormDefaults.image

* fix whitespace

* add tooltip for images

Co-authored-by: JessicaBarh <jessicabarhoma@gmail.com>

* Update dockerfiles and make compatible with Renovate (#5968)

* Update dockerfiles and make compatible with Renovate

* Set memory for jupyter pytorch to the same as jupyter tensorflow

* Update protobuf

* Remove conda version and use substring expansion

* Update SQLAlchemy

* Update dill

* Create OWNERS file in .github folder for Renovate config + workflows (#5983)

* Extend deadline for PyTorch image building (#5991)

* Allow user to add/delete labels to user namespace using ConfigMap. Fix #5712 (#5761)

The profile-controller applies a hardcoded list of labels to Profile
namespaces. Make this list of labels configurable with a ConfigMap. The
new list of steps for the profile-controller is:

1. **(new)** Profile-controller starts and reads the file given by the
   `--namespace-labels-path` CLI flag (defaults to
   `/etc/profile-controller/namespace-labels.yaml`). This file contains
   a YAML dict of label key/values to add to each Profile namespace.
2. Profile-controller creates namespace.
3. Profile-controller merges existing namespace labels with given
   labels. A given label is applied ONLY if it doesn't exist already.
4. **(new)** If a given label value is empty (``), remove the label from
   the namespace if it already exists.

We are using a ConfigMap instead of a plain CLI flag, so that users can
change the list of labels without restarting the profile-controller.

Closes #5712

Signed-off-by: Yannis Zarkadas <yanniszark@arrikto.com>

Co-authored-by: James Liu <jamxl@google.com>

Co-authored-by: Yannis Zarkadas <yanniszark@arrikto.com>

* Update last batch of notebook server Dockerfiles (#5997)

* Update spawner_ui_config.yaml with new tags (#5998)

* Update images in spawner_ui_config.yaml

* Update manifest image tags

* CRUD web apps:  i18n (#5880)

* feat(jupyter): add modules for translations

* feat(jupyter): fix path for translations + add GPU translations

* feat(jupyter): add translations

* feat(jupyter): add translations to form-default

* feat(jupyter): add translations to affinity/tolerations

* feat(jupyter): add translations to Image Pull Policy

* feat(jupyter): remove imports from app.module.ts + fix translations

* web-apps(front): add TranslationModule to common for jupyter

* Implement i18n for jupyter frontend
- Including common components needed
- Refactor translation file

* web-apps(front): add i18n to messages from backend
Status tooltip text
Snackbar message

* web-apps(front): volumes and tensorboards i18n ini
Add the i18n feature for the other 2 projects

* feat(volume): add translations for volume

* feat(jupyter): refactor asset file

* Update asset files for all folders

* feat(tensorboards): add i18n
Update all asset files

* cwa: add new translations

* cwa: Add README documentation

* fix: formatting with prettier

* Fix tests

* Fix python format

* remove comment

Co-authored-by: saffaalvi <alvi118@uwindsor.ca>
Co-authored-by: Jose-Matsuda <tongster789@gmail.com>

* Remove deprecated JWA (#5959)

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* Sync spawner yaml from source to manifests (#5999)

* notebooks: Add additional PGP server for RStudio package validation (#6010)

* Change PGP server for RStudio validation to pgp.surfnet.nl

* Add as secondary keyserver

* Add support for ServiceAccountName and AutomountServiceAccountName to admission-webhook (#5939)

* WA: Use relative paths for fetching translation files (#6034)

* jwa(front): Use relative path for i18n json

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* vwa(front): Use relative path for i18n json

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* twa(front): Use relative path for i18n json

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* Update CRUD web apps from Angular 8 to Angular 12 (#6004)

* common: update angular core and cli to 9

ng update @angular/core@9 @angular/cli@9

* common: update material, cdk and cdk-experimental to 9

ng update @angular/cdk@9 @angular/material@9 @angular/cdk-experimental@9

* common: remove entryComponents and replace TestBed.get with TestBed.inject

Edit files manually

* common: make compatible with Angular 9

Edit files manually

* common: update fontawesome to 0.6.0

npm i @fortawesome/angular-fontawesome@0.6.0

* update kubeflow-common project dependencies

* common: add @angular/localize

ng add @angular/localize

* jupyter: update angular core and cli 8

ng update @angular/core@8 @angular/cli@8

* jupyter: update angular core and cli to 9

ng update @angular/core@9 @angular/cli@9

* jupyter: update material, cdk and cdk-experimental to 9

ng update @angular/cdk@9 @angular/material@9 @angular/cdk-experimental@9

* jupyter: remove entryComponents

Edit file manually

* jupyter: update fontawesome to 0.6.0

npm i @fortawesome/angular-fontawesome@0.6.0

* jupyter: add @angular/localize

ng add @angular/localize

* tensorboards: update angular core and cli to 9

ng update @angular/core@9 @angular/cli@9

* tensorboards: update material, cdk and cdk-experimental to 9

ng update @angular/cdk@9 @angular/material@9 @angular/cdk-experimental@9

* tensorboards: remove entryComponents and replace TestBed.get with TestBed.inje…

Edit files manually

* tensorboards: update fontawesome to 0.6.0

npm i @fortawesome/angular-fontawesome@0.6.0

* tensorboards: add @angular/localize

ng add @angular/localize

* Fix tensorboards-web-app run-dev in makefile

Edit file manually

* volumes: update angular core and cli to 9

ng update @angular/core@9 @angular/cli@9

* volumes: update material, cdk and cdk-experimental to 9

ng update @angular/cdk@9 @angular/material@9 @angular/cdk-experimental@9

* volumes: remove entryComponents and replace TestBed.get with TestBe…

Edit files manually

* volumes: update fontawesome to 0.6.0

npm i @fortawesome/angular-fontawesome@0.6.0

* volumes: add @angular/localize

ng add @angular/localize

* common: update angular core and cli to 10

ng update @angular/core@10 @angular/cli@10 --force

* common: update material, cdk and cdk-experimental to 10

ng update @angular/cdk@10 @angular/material@10 @angular/cdk-experimental@10

* update kubeflow-common project dependencies

Edit file manually

* common: update fontawesome to 0.7.0

npm i @fortawesome/angular-fontawesome@0.7.0

* common: fix font location

Edit file manually

* jupyter: update angular-fontawesome to 0.7.0

npm i @fortawesome/angular-fontawesome@0.7.0

* jupyter: update angular core and cli to 10

ng update @angular/core@10 @angular/cli@10

* jupyter: update material, cdk and cdk-experimental to 10

ng update @angular/cdk@10 @angular/material@10 @angular/cdk-experimental@10

* tensorboards: update angular-fontawesome to 0.7.0

npm i @fortawesome/angular-fontawesome@0.7.0

* tensorboards: update angular core and cli to 10

ng update @angular/core@10 @angular/cli@10

* tensorboards: update material, cdk and cdk-experimental to 10

ng update @angular/cdk@10 @angular/material@10 @angular/cdk-experimental@10

* volumes: update angular-fontawesome to 0.7.0

npm i @fortawesome/angular-fontawesome@0.7.0

* volumes: update angular core and cli to 10

ng update @angular/core@10 @angular/cli@10

* volumes: update material, cdk and cdk-experimental to 10

ng update @angular/cdk@10 @angular/material@10 @angular/cdk-experimental@10

* common: update angular-fontawesome to 0.8.2

npm i @fortawesome/angular-fontawesome@0.8.2

* common: update codelyzer to 6.0.2

npm i codelyzer@6.0.2

* common: install @angular-devkit/core@11

npm install --save-dev @angular-devkit/core@11

* common: update angular core and cli to 11

ng update @angular/core@11 @angular/cli@11

* common: update material, cdk and cdk-experimental to 11

ng update @angular/cdk@11 @angular/material@11 @angular/cdk-experimental@11

* update kubeflow-common project dependencies

Edit file manually

* jupyter: update angular-fontawesome to 0.8.2

npm i @fortawesome/angular-fontawesome@0.8.2

* jupyter: update codelyzer to 6.0.2

npm i codelyzer@6.0.2

* jupyter: update angular core and cli to 11

ng update @angular/core@11 @angular/cli@11

* jupyter: update material, cdk and cdk-experimental to 11

ng update @angular/cdk@11 @angular/material@11 @angular/cdk-experimental@11

* tensorboards: update angular-fontawesome to 0.8.2

npm i @fortawesome/angular-fontawesome@0.8.2

* tensorboards: update codelyzer to 6.0.2

npm i codelyzer@6.0.2

* tensorboards: update angular core and cli to 11

ng update @angular/core@11 @angular/cli@11

* tensorboards: update material, cdk and cdk-experimental to 11

ng update @angular/cdk@11 @angular/material@11 @angular/cdk-experimental@11

* volumes: update angular-fontawesome to 0.8.2

npm i @fortawesome/angular-fontawesome@0.8.2

* volumes: update codelyzer to 6.0.2

npm i codelyzer@6.0.2

* volumes: update angular core and cli to 11

ng update @angular/core@11 @angular/cli@11

* volumes: update material, cdk and cdk-experimental to 11

ng update @angular/cdk@11 @angular/material@11 @angular/cdk-experimental@11

* common: Fix CI failure

* jupyter: npm run format:write

npm run format:write

* volumes: npm run format:write

npm run format:write

* tensorboards: manually run prettier

prettier --write 'src/**/*.{js,ts,html,scss,css}'

* common: Fix MockComponent entryComponents for CI

Manually edit file

* common: update angular-fontawesome to 0.9.0

npm i @fortawesome/angular-fontawesome@0.9.0

* common: update angular core and cli to 12

ng update @angular/core@12 @angular/cli@12

* common: update material, cdk and cdk-experimental to 12

ng update @angular/cdk@12 @angular/material@12 @angular/cdk-experimental@12

* update kubeflow-common project dependencies

Edit file manually

* common: fix Intl.DateTimeFormatOptions

Edit file manually. Type of defaultDateOptions and defaultTimeOptions needs to explicitly be set to Intl.DateTimeFormatOptions.

* jupyter: update angular-fontawesome to 0.9.0

npm i @fortawesome/angular-fontawesome@0.9.0

* jupyter: update codelyzer to 6.0.2

npm i codelyzer@6.0.2

* jupyter: update angular core and cli to 12

ng update @angular/core@12 @angular/cli@12

* jupyter: update material, cdk and cdk-experimental to 12

ng update @angular/cdk@12 @angular/material@12 @angular/cdk-experimental@12

* tensorboards: update angular-fontawesome to 0.9.0

npm i @fortawesome/angular-fontawesome@0.9.0

* tensorboards: update angular core and cli to 12

ng update @angular/core@12 @angular/cli@12

* tensorboards: update material, cdk and cdk-experimental to 12

ng update @angular/cdk@12 @angular/material@12 @angular/cdk-experimental@12

* volumes: update angular-fontawesome to 0.9.0

npm i @fortawesome/angular-fontawesome@0.9.0

* volumes: update angular core and cli to 12

ng update @angular/core@12 @angular/cli@12

* volumes: update material, cdk and cdk-experimental to 12

ng update @angular/cdk@12 @angular/material@12 @angular/cdk-experimental@12

* common: update and audit packages

npm update && npm audit fix && npm i lodash-es@4.17.21 && npm i material-icons@0.7.3

* jupyter: update and audit packages

npm update && npm audit fix && npm i material-icons@0.7.3

* tensorboards: update and audit packages

npm update && npm audit fix && npm i material-icons@0.7.3

* volumes: update and audit packages

npm update && npm audit fix && npm i material-icons@0.7.3

* jupyter: npm run format:write

npm run format:write

* lists roadmap in reverse chronological order (#6055)

Signed-off-by: Malini Bhandaru <mbhandaru@vmware.com>

* Correct missing predicates in controller watches. Fixes #5326 (#5873)

Co-authored-by: Filinto Duran <fduran@d2iq.com>

* fix(web-apps): Use Angular's i18n proposed implementation (#6065)

* jwa(front): Add i18n for french in angular.json

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(front): Add i18n rules in package.json

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(front): i18n FormImage

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(front): i18n FormAdvancedOptions

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(front): i18n FormAffinityTolerations

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(front): i18n FormConfigurations

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(front): i18n FormCpuRam

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(front): i18n FormDataVolumes

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(front): i18n FormGpus

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(front): i18n FromName

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(front): i18n FormWorkspaceVolume

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(front): i18n Volume

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* web-apps(front): i18n BackendService

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* web-apps(front): Add localize to peerDependencies

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* web-apps(front): Import $localize in library

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* web-apps(front): i18n RokService

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(front): i18n Form

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(front): i18n BackendService

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* web-apps(front): i18n ConfirmDialog

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(front): i18n Index

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* web-apps(front): i18n ResourceTable

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* web-apps(front): i18n SnackBar

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* web-apps(front): i18n Form

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* web-app(front): i18n NameNamespaceSelector

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* web-apps(front): Remove ngx-translate

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(front): i18n App

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(front): Remove ngx-translate from package.json

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(back): Don't parse keys in status

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* vwa(back): Revert backend messages

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(front): Update translation language files

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* web-apps(back): Don't parse keys in status

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* vwa(front): Add i18n for french in angular.json

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* vwa(front): Add i18n rules in package.json

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* vwa(front): i18n Form

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* vwa(front): i18n Index

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* vwa(front): i18n BackendService

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* vwa(front): i18n App

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* vwa(front): Remove ngx-translate from package.json

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* vwa(front): i18n Form

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* vwa(front): Update translation language files

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* twa(back): Revert backend messages

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* twa(front): Add i18n rules in package.json

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* twa(front): Add i18n for french in angular.json

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* twa(front): i18n Index

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* twa(front): i18n BackendService

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* twa(front): i18n App

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* twa(front): Update translation language files

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* twa(front): Remove ngx-translate from package.json

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* wa: Update READMEs

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* web-apps(front): Add localize when testing

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* [CentralDashboard] Add entry for Models web app (#6085)

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* Interact with the namespace selector (#5995)

* Interact with the namespace selector

* Use namespace given as an argument

* Keep the index of selected item

* fix(jwa): Fix limits calculation when limitFactor is none (#6058)

* jwa(front): Don't allow NaN values in limits

The UI should always catch a NaN value and don't add it in the form.
Currently this is the case for the cpu/memory limits.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(front): Limits should not be changed if dirty

If the user has manually edited the limits fields then the UI should not
try to automatically calculate them again, using the limitFactors.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* Fix NaN in Kubeflow Notebooks (#6092)

* Fix NaN in Kubeflow Notebooks

* add a validation check for NaN values

* fix: validation check if value is None

* lint: fix line length

* add section for 1.4; small updates for 1.3 section (#6082)

* add section for 1.4; small updates for 1.3 section

* refer to training operators consistently; add link to Notebooks roadmap.

* address @Bobgy feedback regarding Kubeflow Pipelines portions of 1.4 roadmap

* Remove virtualservice timeout to prevent websocket disconnect (#6126)

In the existing version, the 'timeout: 300s' added to the notebook's virtual service would cause websockets to disconnect at the 5 minute mark, causing the Jupyter Notebook web terminal function to hang. This is described in https://github.com/kubeflow/kubeflow/issues/6124.

* fix(admission-webhook): attach namespace to pod request if pod does not have it (#6052)

* fix(admission-webhook): attach namespace to pod request if pod does not have it

* Apply suggestions from code review

Add more explicit logs

Co-authored-by: Erhan Kesken <ekesken@users.noreply.github.com>

* Add more explicit log

Co-authored-by: Erhan Kesken <ekesken@users.noreply.github.com>

Co-authored-by: Erhan Kesken <ekesken@users.noreply.github.com>

* Fix notebook base image gpg keys (#6138)

* update links for new testing infra (#6144)

* Add more root OWNERS (#6146)

* Improve README for example-notebook-servers (#6165)

* jwa(front): Don't disable vendors with no GPUs (#6171)

JWA should not block users from selecting GPUs if the current cluster
nodes do not have any GPUs attached to them. We've seen users that have
autoscaled nodegroups for GPUs, so a GPU node will be added to the
cluster once a Pod has requested it.

Refs: arrikto/dev#1484

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* Bump Golang version in PodDefaults, TensorBoard Controller and KFAM to 1.17 (#6180)

* kfam: Upgrade go to 1.17

Update to a more recent docker image that has a newer version of
openssl.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* poddefaults: Upgrade go to 1.17

Update to a more recent docker image that has a newer version of
openssl.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* tensorboards: Upgrade go to 1.17

Update to a more recent docker image that has a newer version of
openssl.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* Update Notebooks README regarding custom packages (#6176)

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* Fix reference to PodPreset as PodDefault (#6184)

* fix: tensorboard-controller is killed due to out of memory (#6148)

* Update manager.yaml

* Update manager.yaml

* fix(web-apps): Use relative path for --deploy-url build. #5566 (#6206)

* fix(web-apps): Use relative path for --base-href build

* volume web app should use volume base-href

* Document for cutting Kubeflow website release. (#6198)

* Document for cutting Kubeflow website release.

* Update docs_dev/releasing.md

Co-authored-by: Shannon Bradshaw <shannon.bradshaw@arrikto.com>

* Update docs_dev/releasing.md

Co-authored-by: Shannon Bradshaw <shannon.bradshaw@arrikto.com>

* address comment

* canonical name

Co-authored-by: Shannon Bradshaw <shannon.bradshaw@arrikto.com>

* Change namespace label for Katib metrics collector injection (#6225)

* fix(backend): tensorboard-controller does not work because of missing permissions (#6216)

* add(web-apps): i18n Fr translations (#6122)

* chore(docs): It's better to refer to the cli documents, not repo's readme (#6239)

* fix(docs): Fix the 404 K8s docs link (#6236)

* nit: typo in README.md (#6211)

* web-apps(front) : change error msg for min (#6193)

* Centraldashboard/add uncaughtexception handler (#6203)

* Central dashboard - Add global uncaught exception handler

* fix comments

* Fix(frontend): Upgrade ts node to v4 (#6242)

* Upgrade ts-node to v10

* Update lock

* admission-webhook-manifests: Update the cert manager from v1alpha2 to v1 (#6230)

* Short term updates to Published Kubeflow Roadmap (#6248)

3 changes - Add info on Kubeflow 1.5 and 1.4.1.   Update roadmap to show that Kubeflow 1.4 was delivered.

* Upgrade the google cloud monitoring node.js client (#6210)

* Upgrade the google cloud monitoring node.js client

* Update package-lock.json

* Upgrade TypeScript to v4

* Update package-lock.json

* run tslint

* fix dashboard sending malformed headers to kfam (#6186)

* correct typo - track to tracked (#6264)

corrected typo "track" to "tracked"

* centraldashboard: Update node and use latest-stable (#6260)

Change the tests to NOT fetch Chromium from the Edge branch, which is develop,
but instead use latest-stable. We saw that edge can have problems from times to
times.

In the same commit we also update the node version to fix CVEs with the current
v12.18.3 version that we had.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* Tell user to select a namespace to access namespaced menu item (#6181)

* Fix unnecessary updates of istio AuthorizationPolicy (#6269)

* Introduce a basic proposals flow for more advanced features (#6272)

* Introduce a proposals dir for Notebooks

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* Move the proposals dir under components

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* Secure access to KFAM (#6077)

* fix(notebooks) make culling work with multi-user (#5128) (#5980)

* JWA: Very basic E2E tests to run locally (#6005)

* web-apps(front): Add test attrs selectors

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(front): Update gitignore for cypress files

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(front): Install cypress & npm scripts for e2e

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(e2e): Add first cypress tests

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa: Update the README for e2e instructions

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* chore: remove Bobgy from OWNERS

* frontend: Use responsive tables instead of cards (#6316)

* jwa(front): Make ng lint work by ignoring e2e/tsconfig.json

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>
Reviewed-by: Tasos Alexiou <tasos@arrikto.com>

* web-apps(front): Make the table responsive

We introduced all of our changes in the underlying TableComponent. This
component can then be either used independently or inside a material
card.

The changes we did in that component are:
1. Create Output() emitters, since this component can be used directly
2. The config object for a Table row now supports a `style` prop for
   defining the list of CSS styles to be applied
2. Remove the truncate classes (small, medium, large) and only have
   a boolean value. The user can define the width directly now via the
   `style` property in the row's config
3. Modify the classes for aligning contents right and left

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>
Reviewed-by: Tasos Alexiou <tasos@arrikto.com>

* web-apps(front): Add table paginator

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>
Reviewed-by: Tasos Alexiou <tasos@arrikto.com>

* web-apps(front): Add padding to titlebar text

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>
Reviewed-by: Tasos Alexiou <tasos@arrikto.com>

* web-apps(front): Use bigger font for toolbar title

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>
Reviewed-by: Tasos Alexiou <tasos@arrikto.com>

* jwa(front): Use responsive table and toolbar

We refactor both the index and form pages of the app to:
1. Add a top row toolbar with the title of the app and the button to
   create a new Notebook
2. Replace the card with a responsive table that shows the items. The
   component also has a paginator

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>
Reviewed-by: Tasos Alexiou <tasos@arrikto.com>

* vwa(front): Use responsive table and toolbar

This commit:
1. Adds a toolbar at the top of the index page with the title of the app
   and the button to create a new volume
2. Replaces the card with a responsive table

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>
Reviewed-by: Tasos Alexiou <tasos@arrikto.com>

* twa(front): Use responsive table and toolbar

This commit:
1. Adds a toolbar at the top of the index page with the title of the app
   and a button to create a new TensorBoard instance
2. Replaces the card with a resopnsive table

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>
Reviewed-by: Tasos Alexiou <tasos@arrikto.com>

* fix the format

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* wa(front): Add npm script for running unit tests in docker

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* fix unit tests failing

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* Don't use asterisk in Dockerfiles (#6318)

* wa(docker): Don't copy node_modules

To ensure Kaniko is not copying stale node_moduels folders, even though
we have a dockerignore file, we are explicitly only copying the source
code.

We have seen the build system with Kaniko to fail, due to NFS stale
instances with files in node_modules and expect that this is the root
cause.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* twa: Update makefile to use global dockerignore

The dockerfile for TWA was copying over the local dockerignore. This was
overriding the global one we had for all the web apps.

This commit updates the Makefile of the app to use the global
dockerignore that all the apps should use.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* Bump log4js from 6.3.0 to 6.4.1 in /components/crud-web-apps/common/frontend/kubeflow-common-lib (#6309)

Bumps [log4js](https://github.com/log4js-node/log4js-node) from 6.3.0 to 6.4.1.
- [Release notes](https://github.com/log4js-node/log4js-node/releases)
- [Changelog](https://github.com/log4js-node/log4js-node/blob/master/CHANGELOG.md)
- [Commits](https://github.com/log4js-node/log4js-node/compare/v6.3.0...v6.4.1)

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

Signed-off-by: dependabot[bot] <support@github.com>

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

* Bump postcss from 7.0.16 to 7.0.39 in /components/centraldashboard (#6308)

Bumps [postcss](https://github.com/postcss/postcss) from 7.0.16 to 7.0.39.
- [Release notes](https://github.com/postcss/postcss/releases)
- [Changelog](https://github.com/postcss/postcss/blob/7.0.39/CHANGELOG.md)
- [Commits](https://github.com/postcss/postcss/compare/7.0.16...7.0.39)

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

Signed-off-by: dependabot[bot] <support@github.com>

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

* Bump log4js from 6.3.0 to 6.4.1 in /components/crud-web-apps/tensorboards/frontend (#6311)

Bumps [log4js](https://github.com/log4js-node/log4js-node) from 6.3.0 to 6.4.1.
- [Release notes](https://github.com/log4js-node/log4js-node/releases)
- [Changelog](https://github.com/log4js-node/log4js-node/blob/master/CHANGELOG.md)
- [Commits](https://github.com/log4js-node/log4js-node/compare/v6.3.0...v6.4.1)

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

Signed-off-by: dependabot[bot] <support@github.com>

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

* Bump log4js from 6.3.0 to 6.4.1 in /components/crud-web-apps/volumes/frontend (#6310)

Bumps [log4js](https://github.com/log4js-node/log4js-node) from 6.3.0 to 6.4.1.
- [Release notes](https://github.com/log4js-node/log4js-node/releases)
- [Changelog](https://github.com/log4js-node/log4js-node/blob/master/CHANGELOG.md)
- [Commits](https://github.com/log4js-node/log4js-node/compare/v6.3.0...v6.4.1)

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

Signed-off-by: dependabot[bot] <support@github.com>

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

* Bump log4js from 6.3.0 to 6.4.1 in /components/crud-web-apps/jupyter/frontend (#6312)

Bumps [log4js](https://github.com/log4js-node/log4js-node) from 6.3.0 to 6.4.1.
- [Release notes](https://github.com/log4js-node/log4js-node/releases)
- [Changelog](https://github.com/log4js-node/log4js-node/blob/master/CHANGELOG.md)
- [Commits](https://github.com/log4js-node/log4js-node/compare/v6.3.0...v6.4.1)

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

Signed-off-by: dependabot[bot] <support@github.com>

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

* Update images for the 1.5 rc0 release (#6319)

* Update the releasing version tag

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* Run automated script for updating versions

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* Fix(manifests): Upgrade rbac.authorization.k8s.io from v1beta1 to v1 (#6261)

* proposal: Extend Notebook Controller to expose idleness for Jupyter (#6295)

* proposal: Extend Notebook Controller to expose idleness for Jupyter (#6270)

Provide a design doc as a proposal for extending Notebook Controller to
expose idleness for Jupyter. Our proposal is in markdown format and follows
the guidelines of the kubeflow/components/proposal/README.md guide.

You can view the #6270 issue in the following link:
https://github.com/kubeflow/kubeflow/issues/6270

Signed-off-by: Athanasios Markou <athamark@arrikto.com>

* review: change the title of the proposal

Change the title of the proposal to only include the
proposed new feature. The new title of the proposal
will now be "Expose Idleness Information for Jupyter
Notebooks".

* review: rename the proposal markdown file

We want to give a more specific name to the markdown
which contains the proposal. Since this proposal
emphasizes on a feature regarding the Jupyter Notebooks,
the new name will be:

20220121-jupyter-notebook-idleness.md

* Synchronize jupyter-web-application role with clusterrole (#6241)

* Update role.yaml

* Update role.yaml

* Update cluster-role.yaml

* Kubeflow Roadmap update - with 1.5 details (#6266)

* Kubeflow Roadmap update - with 1.5 details

These proposed changes include: identifying that 1.4.1 has been delivered, provides themes for 1.5 and provides details of major features in 1.5 by working group.   This is an initial proposal that needs review by the working group leads.

* correct formatting in KFP features

Moved KFP features under KFP Control Flow doc

* updating KFP section

updating KFP references with updates from KFP team

* Updated the 1.5 release date to March

updated the 1.5 release date to March

* Update ROADMAP.md

change Hyperparameter leader election to Katib leader election

Co-authored-by: Andrey Velichkevich <andrey.velichkevich@gmail.com>

* Update ROADMAP.md

improve description and details of feature for metrics collector

Co-authored-by: Andrey Velichkevich <andrey.velichkevich@gmail.com>

* Update Katib description for Early stopping in 1.5

updating with Andrey's suggestion (but without the world proper).   * Validation for Early Stopping algorithm settings helps users to proper reduce model overfitting

Co-authored-by: Andrey Velichkevich <andrey.velichkevich@gmail.com>

* notebooks: Extend Notebook Controller to expose idleness for Jupyter (#6297)

* notebooks: Update image's tag in make

Modify Makefile to update properly the TAG
based on the git TAG.

Signed-off-by: Athanasios Markou <athamark@arrikto.com>
Reviewed-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* notebooks: Expose last-activity

Extend the notebook-controller to:
* cull idle Notebook Servers based on their new `last-activity`
  annotation
* expose the last activity of each Notebook Server as an annotation
  on the metadata of the corresponding CR object

Modify notebook_controller.go to:
* update the Last Activity of each Notebook Server that has a
  Running pod
* delete the Last Activity Annotation for every Notebook Server
  that does not have a Running pod

Extend culler.go to:
* perform culling based on the new `last-activity` annotation and
  not based on the `/api/status` endpoint.
* update the last activity of a Notebook Server, based on the
  kernels' execution states.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>
Reviewed-by: Athanasios Markou <athamark@arrikto.com>

* notebooks: Introduce a DEV env var

We introduce a DEV ENV var to allow admins
develop and test on their local machine their
custom Notebook Controller.
We provide information and instructions inside
the components/notebook-controller/README.md.

Signed-off-by: Athanasios Markou <athamark@arrikto.com>
Reviewed-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* notebooks: Add unit tests for last-activity

* Introduce new tests for allKernelsAreIdle()
* Extend the tests for NotebookIsIdle() and for
  NotebookNeedsCulling().

Signed-off-by: Athanasios Markou <athamark@arrikto.com>
Reviewed-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* review: UpdateNotebookLastActivityAnnotation()

Ensure that UpdateNotebookLastActivityAnnotation() does not return
"true". This function should not return any value.

Signed-off-by: Athanasios Markou <athamark@arrikto.com>

* jwa: Rework the Storage API of the web app (#6321)

* wa(back): Add helper for deserializing JSON obj

In some cases we might need to construct Python k8s lib objects from the
JSONs that are provided by clients. I.e. the UI will be sending a PVC
object in json format, so the backend will need to create the
corresponding client.V1PersistentVolumeClaim object and submit it.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>
Reviewed-by: Ilias Katsakioris <elikatsis@arrikto.com>

* wa(back): Serialization helper

Add helper function for converting a k8s-client object into a dict that
can be sent as an HTTP response.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>
Reviewed-by: Ilias Katsakioris <elikatsis@arrikto.com>

* wa(back): Add dry run to Notebooks and PVCs

The backend will need to be able to create objects with dry-run, in
order to ensure they are valid. The backend will need to check that both
the Notebook and the PVCs can be created beforehand.

This way we avoid the scenario where we create PVCs but the Notebook
fails to be created, and the PVCs are never garbage collected.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>
Reviewed-by: Ilias Katsakioris <elikatsis@arrikto.com>

* wa(back): Update kubernetes to 0.17

In order to support dry-run we must use the 0.17 version of the Python
k8s client.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>
Reviewed-by: Ilias Katsakioris <elikatsis@arrikto.com>

* wa(back): Extend api module to patch pvcs

The backend will need to be able to PATCH PVCs in order to set the
ownerReference to the Notebook that mounts the PVCs.

Ref: arrikto/dev/issues/386#issuecomment-856700392

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>
Reviewed-by: Ilias Katsakioris <elikatsis@arrikto.com>

* jwa(back): Work with new Volumes API

The backend API should not add any more layers of abstractions on top of
the K8s API. The backend should expect the client/UI to be sending the
entire PVC spec of a new PVC.

Refs: arrikto/dev/issues/386

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>
Reviewed-by: Ilias Katsakioris <elikatsis@arrikto.com>

* jwa(back): Add unittests for new volumes API

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>
Reviewed-by: Ilias Katsakioris <elikatsis@arrikto.com>

* jwa(back): Extend the PVC info returned

We want to show both the access mode and size of the existing PVCs, when
a user clicks on the dropdown to select which PVC to mount.

The backend will need to provide this information to the frontend. We
don't want to send the K8s list of PVCs since this will result in a lot
of unnecessary data to be sent.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>
Reviewed-by: Ilias Katsakioris <elikatsis@arrikto.com>

* jwa(front): Add proxy config for Rok

When developing the Rok flavor locally we will need to be able to open
the Rok chooser. This can be done by using Angular/webpack proxy to
bring the exposed rok service and the app under the same domain.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>
Reviewed-by: Tasos Alexiou <tasos@arrikto.com>

* jwa(front): Remove card from form

The form of the app should not be a big card, but a normal form.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>
Reviewed-by: Tasos Alexiou <tasos@arrikto.com>

* jwa(front): Install AceModule for yaml editing

Install AceModule to allow users to edit yamls of objects.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>
Reviewed-by: Tasos Alexiou <tasos@arrikto.com>

* wa(front): Change the styling of form sections

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>
Reviewed-by: Tasos Alexiou <tasos@arrikto.com>

* jwa(front): Create common volume components

Component for:
* New PVC and configuring its spec
* Attaching an existing PVC in a Notebook

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>
Reviewed-by: Tasos Alexiou <tasos@arrikto.com>

* jwa(front): Update Rok form for new Volume API

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>
Reviewed-by: Tasos Alexiou <tasos@arrikto.com>

* jwa(front): Mark inputs as dirty when restoring Lab

When the UI autofills the form with values from a JupyterLab snapshot
then it should mark the touched fields as dirty. This way if a field has
errors the UI will make that input red.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>
Reviewed-by: Tasos Alexiou <tasos@arrikto.com>

* jwa: Update ConfigMap in manifests

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(front): Fix format errors

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* profiles: Update the permissions for notebook idleness (#6335)

Extend the Profiles Controller to give permissions to Notebooks
controller for making GET requests to notebook's /api/kernels endpoint.

Refs https://github.com/kubeflow/kubeflow/blob/master/components/proposals/20220121-jupyter-notebook-idleness.md

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* notebooks: Graceful handling of events (#6338)

* notebooks: Handle events gracefully

The controller is not exiting the reconciliation loop after it has
re-emitted a Pod/STS Event as a Notebook Event. This results in the
controller to later on try and GET a Notebook with the name of the Event
that triggered the reconciliation loop.

The controller should exit the reconciliation function once it has
emitted the event.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* notebooks: Don't reconcile on deleted events

We don't want to trigger the reconciliation function when an event gets
deleted.

If a Notebook would be deleted then the underlying events would
be deleted as well, which results in the reconcile function to get
triggered and try to GET Events and Notebooks with the name of the
deleted event.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* notebooks: Fix endless restarts (#6341)

* notebooks: Update notebook if timestamp changed

We don't want to be updating the spec of the notebook if the timestamp
hasn't changed, since this will lead to constant updates and
reconciliation loops.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* notebooks: Use a deep-copy of the notebook spec

The controller should use a deep-copy of the notebook spec when
calculating the spec for the StatefulSet. If not then we could
update the notebook object without wanting it, since the spec could have
been changed when calculating the STS spec.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* notebooks: Add prefix env var only if missing

The controller should be setting OR updating the NB_PREFIX env var.
Previously it would always blindly append it to the spec, which could
result in double entries for the same env var.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* releasing: Update tags for v1.5.0-rc.1 (#6343)

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* fix error comparison between pointer and pointer in "CopyStatefulSetFields" (#6195)

error comparison between pointer and pointer in "CopyStatefulSetFields"

* [bugfix]: update unsupported links to latest pages (#6192)

* centraldashboard: Add KServe overlay (#6383)

Configure the dashboard to use the KServe app instead of the KFServing
0.6.1 one.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* notebooks: Don't reconcile on Events deletion (#6391)

The controller should not trigger the reconcile loop when an Event is
deleted. Previously the controller would run the reconciliation loop on
any event deletion.

This commit updates it to not run the loop for ANY event.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* releasing: Create v1.5.0-rc.2 images (#6394)

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* release: Images for the 1.5.0 tag (#6398)

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* added env for app secure cookies (#6395)

* build(deps): bump github.com/gogo/protobuf from 1.1.1 to 1.3.2 in /components/tensorboard-controller (#6424)

Bumps [github.com/gogo/protobuf](https://github.com/gogo/protobuf) from 1.1.1 to 1.3.2.
- [Release notes](https://github.com/gogo/protobuf/releases)
- [Commits](https://github.com/gogo/protobuf/compare/v1.1.1...v1.3.2)

---
updated-dependencies:
- dependency-name: github.com/gogo/protobuf
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <support@github.com>

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

* build(deps): bump notebook from 6.4.0 to 6.4.10 in /components/example-notebook-servers/jupyter (#6431)

Bumps [notebook](http://jupyter.org) from 6.4.0 to 6.4.10.

---
updated-dependencies:
- dependency-name: notebook
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <support@github.com>

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

* build(deps): bump minimist from 1.2.5 to 1.2.6 in /components/crud-web-apps/tensorboards/frontend (#6417)

Bumps [minimist](https://github.com/substack/minimist) from 1.2.5 to 1.2.6.
- [Release notes](https://github.com/substack/minimist/releases)
- [Commits](https://github.com/substack/minimist/compare/1.2.5...1.2.6)

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

Signed-off-by: dependabot[bot] <support@github.com>

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

* build(deps): bump minimist from 1.2.5 to 1.2.6 in /components/crud-web-apps/jupyter/frontend (#6429)

Bumps [minimist](https://github.com/substack/minimist) from 1.2.5 to 1.2.6.
- [Release notes](https://github.com/substack/minimist/releases)
- [Commits](https://github.com/substack/minimist/compare/1.2.5...1.2.6)

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

Signed-off-by: dependabot[bot] <support@github.com>

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

* build(deps): bump follow-redirects from 1.14.1 to 1.14.8 in /components/crud-web-apps/tensorboards/frontend (#6357)

Bumps [follow-redirects](https://github.com/follow-redirects/follow-redirects) from 1.14.1 to 1.14.8.
- [Release notes](https://github.com/follow-redirects/follow-redirects/releases)
- [Commits](https://github.com/follow-redirects/follow-redirects/compare/v1.14.1...v1.14.8)

---
updated-dependencies:
- dependency-name: follow-redirects
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <support@github.com>

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

* build(deps): bump ajv from 6.10.2 to 6.12.6 in /components/crud-web-apps/volumes/frontend (#6358)

Bumps [ajv](https://github.com/ajv-validator/ajv) from 6.10.2 to 6.12.6.
- [Release notes](https://github.com/ajv-validator/ajv/releases)
- [Commits](https://github.com/ajv-validator/ajv/compare/v6.10.2...v6.12.6)

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

Signed-off-by: dependabot[bot] <support@github.com>

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

* build(deps): bump ajv from 6.10.2 to 6.12.6 in /components/crud-web-apps/jupyter/frontend (#6359)

Bumps [ajv](https://github.com/ajv-validator/ajv) from 6.10.2 to 6.12.6.
- [Release notes](https://github.com/ajv-validator/ajv/releases)
- [Commits](https://github.com/ajv-validator/ajv/compare/v6.10.2...v6.12.6)

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

Signed-off-by: dependabot[bot] <support@github.com>

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

* build(deps): bump lodash-es from 4.17.11 to 4.17.21 in /components/crud-web-apps/tensorboards/frontend (#6402)

Bumps [lodash-es](https://github.com/lodash/lodash) from 4.17.11 to 4.17.21.
- [Release notes](https://github.com/lodash/lodash/releases)
- [Commits](https://github.com/lodash/lodash/compare/4.17.11...4.17.21)

---
updated-dependencies:
- dependency-name: lodash-es
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <support@github.com>

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

* build(deps): bump minimist from 1.2.5 to 1.2.6 in /components/crud-web-apps/common/frontend/kubeflow-common-lib (#6416)

Bumps [minimist](https://github.com/substack/minimist) from 1.2.5 to 1.2.6.
- [Release notes](https://github.com/substack/minimist/releases)
- [Commits](https://github.com/substack/minimist/compare/1.2.5...1.2.6)

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

Signed-off-by: dependabot[bot] <support@github.com>

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

* build(deps): bump minimist from 1.2.5 to 1.2.6 in /components/crud-web-apps/volumes/frontend (#6423)

Bumps [minimist](https://github.com/substack/minimist) from 1.2.5 to 1.2.6.
- [Release notes](https://github.com/substack/minimist/releases)
- [Commits](https://github.com/substack/minimist/compare/1.2.5...1.2.6)

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

Signed-off-by: dependabot[bot] <support@github.com>

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

* build(deps): bump undefsafe from 2.0.2 to 2.0.5 in /components/centraldashboard (#6342)

Bumps [undefsafe](https://github.com/remy/undefsafe) from 2.0.2 to 2.0.5.
- [Release notes](https://github.com/remy/undefsafe/releases)
- [Commits](https://github.com/remy/undefsafe/compare/v2.0.2...v2.0.5)

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

Signed-off-by: dependabot[bot] <support@github.com>

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

* build(deps): bump url-parse from 1.5.1 to 1.5.10 in /components/crud-web-apps/common/frontend/kubeflow-common-lib (#6378)

Bumps [url-parse](https://github.com/unshiftio/url-parse) from 1.5.1 to 1.5.10.
- [Release notes](https://github.com/unshiftio/url-parse/releases)
- [Commits](https://github.com/unshiftio/url-parse/compare/1.5.1...1.5.10)

---
updated-dependencies:
- dependency-name: url-parse
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <support@github.com>

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

* build(deps): bump url-parse from 1.5.1 to 1.5.10 in /components/crud-web-apps/tensorboards/frontend (#6379)

Bumps [url-parse](https://github.com/unshiftio/url-parse) from 1.5.1 to 1.5.10.
- [Release notes](https://github.com/unshiftio/url-parse/releases)
- [Commits](https://github.com/unshiftio/url-parse/compare/1.5.1...1.5.10)

---
updated-dependencies:
- dependency-name: url-parse
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <support@github.com>

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

* build(deps-dev): bump karma from 6.3.4 to 6.3.16 in /components/crud-web-apps/common/frontend/kubeflow-common-lib (#6386)

Bumps [karma](https://github.com/karma-runner/karma) from 6.3.4 to 6.3.16.
- [Release notes](https://github.com/karma-runner/karma/releases)
- [Changelog](https://github.com/karma-runner/karma/blob/master/CHANGELOG.md)
- [Commits](https://github.com/karma-runner/karma/compare/v6.3.4...v6.3.16)

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

Signed-off-by: dependabot[bot] <support@github.com>

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

* build(deps-dev): bump karma from 6.3.4 to 6.3.16 in /components/crud-web-apps/tensorboards/frontend (#6387)

Bumps [karma](https://github.com/karma-runner/karma) from 6.3.4 to 6.3.16.
- [Release notes](https://github.com/karma-runner/karma/releases)
- [Changelog](https://github.com/karma-runner/karma/blob/master/CHANGELOG.md)
- [Commits](https://github.com/karma-runner/karma/compare/v6.3.4...v6.3.16)

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

Signed-off-by: dependabot[bot] <support@github.com>

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

* build(deps): bump moment from 2.24.0 to 2.29.2 in /components/centraldashboard (#6432)

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

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

Signed-off-by: dependabot[bot] <support@github.com>

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

* build(deps): bump url-parse from 1.5.1 to 1.5.10 in /components/crud-web-apps/volumes/frontend (#6380)

Bumps [url-parse](https://github.com/unshiftio/url-parse) from 1.5.1 to 1.5.10.
- [Release notes](https://github.com/unshiftio/url-parse/releases)
- [Commits](https://github.com/unshiftio/url-parse/compare/1.5.1...1.5.10)

---
updated-dependencies:
- dependency-name: url-parse
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <support@github.com>

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

* build(deps): bump url-parse from 1.4.7 to 1.5.10 in /components/centraldashboard (#6377)

Bumps [url-parse](https://github.com/unshiftio/url-parse) from 1.4.7 to 1.5.10.
- [Release notes](https://github.com/unshiftio/url-parse/releases)
- [Commits](https://github.com/unshiftio/url-parse/compare/1.4.7...1.5.10)

---
updated-dependencies:
- dependency-name: url-parse
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <support@github.com>

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

* build(deps-dev): bump karma from 6.3.4 to 6.3.16 in /components/crud-web-apps/volumes/frontend (#6388)

Bumps [karma](https://github.com/karma-runner/karma) from 6.3.4 to 6.3.16.
- [Release notes](https://github.com/karma-runner/karma/releases)
- [Changelog](https://github.com/karma-runner/karma/blob/master/CHANGELOG.md)
- [Commits](https://github.com/karma-runner/karma/compare/v6.3.4...v6.3.16)

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

Signed-off-by: dependabot[bot] <support@github.com>

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

* build(deps-dev): bump karma from 6.3.4 to 6.3.16 in /components/crud-web-apps/jupyter/frontend (#6389)

Bumps [karma](https://github.com/karma-runner/karma) from 6.3.4 to 6.3.16.
- [Release notes](https://github.com/karma-runner/karma/releases)
- [Changelog](https://github.com/karma-runner/karma/blob/master/CHANGELOG.md)
- [Commits](https://github.com/karma-runner/karma/compare/v6.3.4...v6.3.16)

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

Signed-off-by: dependabot[bot] <support@github.com>

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

* build(deps): bump follow-redirects from 1.14.1 to 1.14.9 in /components/crud-web-apps/jupyter/frontend (#6441)

Bumps [follow-redirects](https://github.com/follow-redirects/follow-redirects) from 1.14.1 to 1.14.9.
- [Release notes](https://github.com/follow-redirects/follow-redirects/releases)
- [Commits](https://github.com/follow-redirects/follow-redirects/compare/v1.14.1...v1.14.9)

---
updated-dependencies:
- dependency-name: follow-redirects
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <support@github.com>

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

* build(deps): bump nanoid from 3.1.23 to 3.3.2 in /components/crud-web-apps/common/frontend/kubeflow-common-lib (#6437)

Bumps [nanoid](https://github.com/ai/nanoid) from 3.1.23 to 3.3.2.
- [Release notes](https://github.com/ai/nanoid/releases)
- [Changelog](https://github.com/ai/nanoid/blob/main/CHANGELOG.md)
- [Commits](https://github.com/ai/nanoid/compare/3…
chuckbelisle pushed a commit to StatCan/kubeflow that referenced this pull request Oct 11, 2022
* releasing: Add WG-Notebooks leads to release owners (#5777)

Signed-off-by: Yannis Zarkadas <yanniszark@arrikto.com>

* Specify commonLabels for tensorboard-controller (#5780)

* Remove jlewi as an approver (#5786)

Removing myself as an owner. This should lead to better auto-assignment of code reviews.

* Update notebook server base images (#5804)

* Update second layer docker images to new tags and some python deps (#5809)

* Update image tags and python packages in pytorch and tensorflow full dockerfiles (#5817)

* Add CI format checks for the Jupyter web app (#5811)

* jwa(front): Add npm rule for checking the format

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* ci: Add common tasks for format checks

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* ci(jwa): Add format check tasks

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(back): Fix formatting

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(front): Fix formatting

Run `npm run format:write` on frontend

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(front): Include prettier in package.json

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(docker): Don't copy node_modules in Dockerfile

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* web-apps: Add global dockerignore file

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa: Remove unused dockerignore file

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(make): Don't include dockerignore and cleanup

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* initial notebook server images README.md (#5818)

* tensorboard-controller: Fix scheduling unbound PVCs (#5819)

When the TB controller attempts to schedule a RWO PVC it checks its
accessModes in the PVC status. The controller panics if the list is
empty.

This commit adds a check to ensure the list is not empty.

Signed-off-by: Ilias Katsakioris <elikatsis@arrikto.com>

* Fix profile-controller CRD pruning issue (#5822)

* api: Add marker for preserving unknown fields in Plugins

Signed-off-by: Yannis Zarkadas <yanniszark@arrikto.com>

* manifests: Regenerate manifests

Signed-off-by: Yannis Zarkadas <yanniszark@arrikto.com>

* Remove Logo Trademarks from the Jupyter Web App and make logos configurable (#5823)

* rebase: Make logos configurable in configmap and remove trademark references

Rebased to remove the changes to the package-lock.json

* review: add suggested changes and add image group section to README

* Make notebook limits configurable with a multiplication factor (#5815)

* Make notebook limits configurable with a multiplication factor

* Make limits configurable under advanced section

* run prettier to format frontend code

* fix formatting and add rounding in backend

* Return error if limit is smaller than request

* Allow disabling limitFactor by setting it to none

* review: remove camelCase in python backend

* fix: update spawner_ui_config.yaml in manifests directory

* review: fix setting limits backend

* review: remove unnecessary check from backend

* notebook-servers: Update JupyterLab and add Git Extension (#5846)

* chore: Add comment to reference RStudio license. (#5884)

* CI/CD: add kustomize build tests (#5919)

* Add CI format checks for the Volumes web app (#5820)

* vwa(front): Add npm script to check the formatting

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* vwa(front): Update the package-lock.json

Run `npm install` to bring the package-lock.json up to date

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* vwa(front): Fix formatting

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* vwa(back): Fix formatting

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* vwa(docker): Remove unused dockerignore file

We have created a global dockerignore file for all the web apps in the
parent dir.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* vwa(docker): Don't copy node_modules in dockerfile

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* vwa(make): Don't include dockerignore

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* ci(vwa): Add format check tasks

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(docker): Copy only necessary files for build

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* review: Use docker context instead of cd ..

Don't use a `cd ..` and copy dockerignore files back and forth. Instead
we should use the Docker context and the global dockerignore file we
have for all the web apps.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* tensorboard-controller: fix binding issue (#5925)

* JWA: Don't override assets with logos ConfigMap (#5942)

* jwa(front): Add static logos in the app

The app does not contain the logos' svgs in its source code/static
files. This results in the icons to not show when developing locally.

This commit adds the svgs found in the logos ConfigMap to the static
files of the app as well.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(front): Change logos fetch url

Change the URLs of the logos from `static/assets/*` to
`static/assets/logos`.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(manifests): Don't override assets with logos

Mount the ConfigMap under the `static/assets/logos` directory to not
override the contents of the entire assets dir.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* Make it possible to add Namespaced menu items (#5871)

* Make it possible to add Namespaced menu items

* Reduce the nesting levels

* FIX The number of gpu must be set as string in Kubernetes/Openshift (#5891)

* Update form.py

* Update form.py

* Update form-default.component.ts

* fix whitespace issue

* feat(jupyter): add fonts as assets to service (#5691)

* cwa(front): Ignore font files in assets

* feat(jupyter): add fonts as assets to service

* CRUD: fonts in common

* CWA: Remove link to css file

* jwa(front): Remove font assets from jupyter

Co-authored-by: Wendy Gaultier <wvgaultier@gmail.com>

* Update base notebook for Renovate compatibility (#5955)

* Update base notebook for Renovate compatibility

* Separate kubectl and S6 arch

* Update jupyter, rstudio and vs code notebook images + allow rstudio in iframe (#5961)

* Update jupyter, rstudio and vs code notebook images

* Remove sed and use substring removal

* Make menu bar scrollable again (#5964)

* [fix]: Make jupyter-web-app parse workspace volume MountPath (#5952)

* [fix]: Make jupyter-web-app parse workspace volume MountPath

- workspace volume path was fixed with "/home/jovyan"
- it should be enable to parse from jupyter-web-app-config's data

* change parsing key correctly

* Remove the f-strings in the Jupyter web app's backend (#5680)

* Notebook servers: Add explanation about kernel not connecting (#5920)

* fix(profile-controller): KNative probes (#5848)

Knative control-plane components need to probe certain paths of Knative
Pods, in order to function correctly. These paths are:
- /healthz
- /metrics
- /ready
- /wait-for-drain

For this reason, we extend the Profile Controller to apply an Istio
AuthorizationPolicy that allows traffic to these HTTP paths for Pods in
user namespaces, as per Knative's instructions:
https://knative.dev/docs/serving/istio-authorization/#allowing-access-from-system-pods-by-paths

This only fixes requests through the public ingress. Cluster-internal
traffic and predictor-transformer use-cases are not supported yet. We
continue discussing these in kubeflow#5965.

Refs kubeflow#5965
Refs kubeflow/kfserving#1558

Signed-off-by: Yurii Komar <subreptivus@gmail.com>

* feat(jupyter): Support hiding image registry/tag (#5681)

* feat(jupyter): Support hiding image registry/tag

* feat(jupyter): Support hiding VSCode/RStudio images registry/tag

* rebasing onto upstream master

* fix: formatting with prettier and set default hideRegistry=true

* fix: move hideRegistry and hideTag from spawnerFormDefaults.image

* fix whitespace

* add tooltip for images

Co-authored-by: JessicaBarh <jessicabarhoma@gmail.com>

* Update dockerfiles and make compatible with Renovate (#5968)

* Update dockerfiles and make compatible with Renovate

* Set memory for jupyter pytorch to the same as jupyter tensorflow

* Update protobuf

* Remove conda version and use substring expansion

* Update SQLAlchemy

* Update dill

* Create OWNERS file in .github folder for Renovate config + workflows (#5983)

* Extend deadline for PyTorch image building (#5991)

* Allow user to add/delete labels to user namespace using ConfigMap. Fix #5712 (#5761)

The profile-controller applies a hardcoded list of labels to Profile
namespaces. Make this list of labels configurable with a ConfigMap. The
new list of steps for the profile-controller is:

1. **(new)** Profile-controller starts and reads the file given by the
   `--namespace-labels-path` CLI flag (defaults to
   `/etc/profile-controller/namespace-labels.yaml`). This file contains
   a YAML dict of label key/values to add to each Profile namespace.
2. Profile-controller creates namespace.
3. Profile-controller merges existing namespace labels with given
   labels. A given label is applied ONLY if it doesn't exist already.
4. **(new)** If a given label value is empty (``), remove the label from
   the namespace if it already exists.

We are using a ConfigMap instead of a plain CLI flag, so that users can
change the list of labels without restarting the profile-controller.

Closes #5712

Signed-off-by: Yannis Zarkadas <yanniszark@arrikto.com>

Co-authored-by: James Liu <jamxl@google.com>

Co-authored-by: Yannis Zarkadas <yanniszark@arrikto.com>

* Update last batch of notebook server Dockerfiles (#5997)

* Update spawner_ui_config.yaml with new tags (#5998)

* Update images in spawner_ui_config.yaml

* Update manifest image tags

* CRUD web apps:  i18n (#5880)

* feat(jupyter): add modules for translations

* feat(jupyter): fix path for translations + add GPU translations

* feat(jupyter): add translations

* feat(jupyter): add translations to form-default

* feat(jupyter): add translations to affinity/tolerations

* feat(jupyter): add translations to Image Pull Policy

* feat(jupyter): remove imports from app.module.ts + fix translations

* web-apps(front): add TranslationModule to common for jupyter

* Implement i18n for jupyter frontend
- Including common components needed
- Refactor translation file

* web-apps(front): add i18n to messages from backend
Status tooltip text
Snackbar message

* web-apps(front): volumes and tensorboards i18n ini
Add the i18n feature for the other 2 projects

* feat(volume): add translations for volume

* feat(jupyter): refactor asset file

* Update asset files for all folders

* feat(tensorboards): add i18n
Update all asset files

* cwa: add new translations

* cwa: Add README documentation

* fix: formatting with prettier

* Fix tests

* Fix python format

* remove comment

Co-authored-by: saffaalvi <alvi118@uwindsor.ca>
Co-authored-by: Jose-Matsuda <tongster789@gmail.com>

* Remove deprecated JWA (#5959)

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* Sync spawner yaml from source to manifests (#5999)

* notebooks: Add additional PGP server for RStudio package validation (#6010)

* Change PGP server for RStudio validation to pgp.surfnet.nl

* Add as secondary keyserver

* Add support for ServiceAccountName and AutomountServiceAccountName to admission-webhook (#5939)

* WA: Use relative paths for fetching translation files (#6034)

* jwa(front): Use relative path for i18n json

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* vwa(front): Use relative path for i18n json

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* twa(front): Use relative path for i18n json

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* Update CRUD web apps from Angular 8 to Angular 12 (#6004)

* common: update angular core and cli to 9

ng update @angular/core@9 @angular/cli@9

* common: update material, cdk and cdk-experimental to 9

ng update @angular/cdk@9 @angular/material@9 @angular/cdk-experimental@9

* common: remove entryComponents and replace TestBed.get with TestBed.inject

Edit files manually

* common: make compatible with Angular 9

Edit files manually

* common: update fontawesome to 0.6.0

npm i @fortawesome/angular-fontawesome@0.6.0

* update kubeflow-common project dependencies

* common: add @angular/localize

ng add @angular/localize

* jupyter: update angular core and cli 8

ng update @angular/core@8 @angular/cli@8

* jupyter: update angular core and cli to 9

ng update @angular/core@9 @angular/cli@9

* jupyter: update material, cdk and cdk-experimental to 9

ng update @angular/cdk@9 @angular/material@9 @angular/cdk-experimental@9

* jupyter: remove entryComponents

Edit file manually

* jupyter: update fontawesome to 0.6.0

npm i @fortawesome/angular-fontawesome@0.6.0

* jupyter: add @angular/localize

ng add @angular/localize

* tensorboards: update angular core and cli to 9

ng update @angular/core@9 @angular/cli@9

* tensorboards: update material, cdk and cdk-experimental to 9

ng update @angular/cdk@9 @angular/material@9 @angular/cdk-experimental@9

* tensorboards: remove entryComponents and replace TestBed.get with TestBed.inje…

Edit files manually

* tensorboards: update fontawesome to 0.6.0

npm i @fortawesome/angular-fontawesome@0.6.0

* tensorboards: add @angular/localize

ng add @angular/localize

* Fix tensorboards-web-app run-dev in makefile

Edit file manually

* volumes: update angular core and cli to 9

ng update @angular/core@9 @angular/cli@9

* volumes: update material, cdk and cdk-experimental to 9

ng update @angular/cdk@9 @angular/material@9 @angular/cdk-experimental@9

* volumes: remove entryComponents and replace TestBed.get with TestBe…

Edit files manually

* volumes: update fontawesome to 0.6.0

npm i @fortawesome/angular-fontawesome@0.6.0

* volumes: add @angular/localize

ng add @angular/localize

* common: update angular core and cli to 10

ng update @angular/core@10 @angular/cli@10 --force

* common: update material, cdk and cdk-experimental to 10

ng update @angular/cdk@10 @angular/material@10 @angular/cdk-experimental@10

* update kubeflow-common project dependencies

Edit file manually

* common: update fontawesome to 0.7.0

npm i @fortawesome/angular-fontawesome@0.7.0

* common: fix font location

Edit file manually

* jupyter: update angular-fontawesome to 0.7.0

npm i @fortawesome/angular-fontawesome@0.7.0

* jupyter: update angular core and cli to 10

ng update @angular/core@10 @angular/cli@10

* jupyter: update material, cdk and cdk-experimental to 10

ng update @angular/cdk@10 @angular/material@10 @angular/cdk-experimental@10

* tensorboards: update angular-fontawesome to 0.7.0

npm i @fortawesome/angular-fontawesome@0.7.0

* tensorboards: update angular core and cli to 10

ng update @angular/core@10 @angular/cli@10

* tensorboards: update material, cdk and cdk-experimental to 10

ng update @angular/cdk@10 @angular/material@10 @angular/cdk-experimental@10

* volumes: update angular-fontawesome to 0.7.0

npm i @fortawesome/angular-fontawesome@0.7.0

* volumes: update angular core and cli to 10

ng update @angular/core@10 @angular/cli@10

* volumes: update material, cdk and cdk-experimental to 10

ng update @angular/cdk@10 @angular/material@10 @angular/cdk-experimental@10

* common: update angular-fontawesome to 0.8.2

npm i @fortawesome/angular-fontawesome@0.8.2

* common: update codelyzer to 6.0.2

npm i codelyzer@6.0.2

* common: install @angular-devkit/core@11

npm install --save-dev @angular-devkit/core@11

* common: update angular core and cli to 11

ng update @angular/core@11 @angular/cli@11

* common: update material, cdk and cdk-experimental to 11

ng update @angular/cdk@11 @angular/material@11 @angular/cdk-experimental@11

* update kubeflow-common project dependencies

Edit file manually

* jupyter: update angular-fontawesome to 0.8.2

npm i @fortawesome/angular-fontawesome@0.8.2

* jupyter: update codelyzer to 6.0.2

npm i codelyzer@6.0.2

* jupyter: update angular core and cli to 11

ng update @angular/core@11 @angular/cli@11

* jupyter: update material, cdk and cdk-experimental to 11

ng update @angular/cdk@11 @angular/material@11 @angular/cdk-experimental@11

* tensorboards: update angular-fontawesome to 0.8.2

npm i @fortawesome/angular-fontawesome@0.8.2

* tensorboards: update codelyzer to 6.0.2

npm i codelyzer@6.0.2

* tensorboards: update angular core and cli to 11

ng update @angular/core@11 @angular/cli@11

* tensorboards: update material, cdk and cdk-experimental to 11

ng update @angular/cdk@11 @angular/material@11 @angular/cdk-experimental@11

* volumes: update angular-fontawesome to 0.8.2

npm i @fortawesome/angular-fontawesome@0.8.2

* volumes: update codelyzer to 6.0.2

npm i codelyzer@6.0.2

* volumes: update angular core and cli to 11

ng update @angular/core@11 @angular/cli@11

* volumes: update material, cdk and cdk-experimental to 11

ng update @angular/cdk@11 @angular/material@11 @angular/cdk-experimental@11

* common: Fix CI failure

* jupyter: npm run format:write

npm run format:write

* volumes: npm run format:write

npm run format:write

* tensorboards: manually run prettier

prettier --write 'src/**/*.{js,ts,html,scss,css}'

* common: Fix MockComponent entryComponents for CI

Manually edit file

* common: update angular-fontawesome to 0.9.0

npm i @fortawesome/angular-fontawesome@0.9.0

* common: update angular core and cli to 12

ng update @angular/core@12 @angular/cli@12

* common: update material, cdk and cdk-experimental to 12

ng update @angular/cdk@12 @angular/material@12 @angular/cdk-experimental@12

* update kubeflow-common project dependencies

Edit file manually

* common: fix Intl.DateTimeFormatOptions

Edit file manually. Type of defaultDateOptions and defaultTimeOptions needs to explicitly be set to Intl.DateTimeFormatOptions.

* jupyter: update angular-fontawesome to 0.9.0

npm i @fortawesome/angular-fontawesome@0.9.0

* jupyter: update codelyzer to 6.0.2

npm i codelyzer@6.0.2

* jupyter: update angular core and cli to 12

ng update @angular/core@12 @angular/cli@12

* jupyter: update material, cdk and cdk-experimental to 12

ng update @angular/cdk@12 @angular/material@12 @angular/cdk-experimental@12

* tensorboards: update angular-fontawesome to 0.9.0

npm i @fortawesome/angular-fontawesome@0.9.0

* tensorboards: update angular core and cli to 12

ng update @angular/core@12 @angular/cli@12

* tensorboards: update material, cdk and cdk-experimental to 12

ng update @angular/cdk@12 @angular/material@12 @angular/cdk-experimental@12

* volumes: update angular-fontawesome to 0.9.0

npm i @fortawesome/angular-fontawesome@0.9.0

* volumes: update angular core and cli to 12

ng update @angular/core@12 @angular/cli@12

* volumes: update material, cdk and cdk-experimental to 12

ng update @angular/cdk@12 @angular/material@12 @angular/cdk-experimental@12

* common: update and audit packages

npm update && npm audit fix && npm i lodash-es@4.17.21 && npm i material-icons@0.7.3

* jupyter: update and audit packages

npm update && npm audit fix && npm i material-icons@0.7.3

* tensorboards: update and audit packages

npm update && npm audit fix && npm i material-icons@0.7.3

* volumes: update and audit packages

npm update && npm audit fix && npm i material-icons@0.7.3

* jupyter: npm run format:write

npm run format:write

* lists roadmap in reverse chronological order (#6055)

Signed-off-by: Malini Bhandaru <mbhandaru@vmware.com>

* Correct missing predicates in controller watches. Fixes #5326 (#5873)

Co-authored-by: Filinto Duran <fduran@d2iq.com>

* fix(web-apps): Use Angular's i18n proposed implementation (#6065)

* jwa(front): Add i18n for french in angular.json

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(front): Add i18n rules in package.json

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(front): i18n FormImage

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(front): i18n FormAdvancedOptions

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(front): i18n FormAffinityTolerations

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(front): i18n FormConfigurations

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(front): i18n FormCpuRam

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(front): i18n FormDataVolumes

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(front): i18n FormGpus

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(front): i18n FromName

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(front): i18n FormWorkspaceVolume

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(front): i18n Volume

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* web-apps(front): i18n BackendService

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* web-apps(front): Add localize to peerDependencies

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* web-apps(front): Import $localize in library

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* web-apps(front): i18n RokService

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(front): i18n Form

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(front): i18n BackendService

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* web-apps(front): i18n ConfirmDialog

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(front): i18n Index

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* web-apps(front): i18n ResourceTable

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* web-apps(front): i18n SnackBar

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* web-apps(front): i18n Form

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* web-app(front): i18n NameNamespaceSelector

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* web-apps(front): Remove ngx-translate

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(front): i18n App

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(front): Remove ngx-translate from package.json

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(back): Don't parse keys in status

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* vwa(back): Revert backend messages

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(front): Update translation language files

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* web-apps(back): Don't parse keys in status

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* vwa(front): Add i18n for french in angular.json

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* vwa(front): Add i18n rules in package.json

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* vwa(front): i18n Form

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* vwa(front): i18n Index

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* vwa(front): i18n BackendService

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* vwa(front): i18n App

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* vwa(front): Remove ngx-translate from package.json

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* vwa(front): i18n Form

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* vwa(front): Update translation language files

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* twa(back): Revert backend messages

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* twa(front): Add i18n rules in package.json

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* twa(front): Add i18n for french in angular.json

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* twa(front): i18n Index

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* twa(front): i18n BackendService

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* twa(front): i18n App

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* twa(front): Update translation language files

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* twa(front): Remove ngx-translate from package.json

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* wa: Update READMEs

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* web-apps(front): Add localize when testing

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* [CentralDashboard] Add entry for Models web app (#6085)

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* Interact with the namespace selector (#5995)

* Interact with the namespace selector

* Use namespace given as an argument

* Keep the index of selected item

* fix(jwa): Fix limits calculation when limitFactor is none (#6058)

* jwa(front): Don't allow NaN values in limits

The UI should always catch a NaN value and don't add it in the form.
Currently this is the case for the cpu/memory limits.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(front): Limits should not be changed if dirty

If the user has manually edited the limits fields then the UI should not
try to automatically calculate them again, using the limitFactors.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* Fix NaN in Kubeflow Notebooks (#6092)

* Fix NaN in Kubeflow Notebooks

* add a validation check for NaN values

* fix: validation check if value is None

* lint: fix line length

* add section for 1.4; small updates for 1.3 section (#6082)

* add section for 1.4; small updates for 1.3 section

* refer to training operators consistently; add link to Notebooks roadmap.

* address @Bobgy feedback regarding Kubeflow Pipelines portions of 1.4 roadmap

* Remove virtualservice timeout to prevent websocket disconnect (#6126)

In the existing version, the 'timeout: 300s' added to the notebook's virtual service would cause websockets to disconnect at the 5 minute mark, causing the Jupyter Notebook web terminal function to hang. This is described in https://github.com/kubeflow/kubeflow/issues/6124.

* fix(admission-webhook): attach namespace to pod request if pod does not have it (#6052)

* fix(admission-webhook): attach namespace to pod request if pod does not have it

* Apply suggestions from code review

Add more explicit logs

Co-authored-by: Erhan Kesken <ekesken@users.noreply.github.com>

* Add more explicit log

Co-authored-by: Erhan Kesken <ekesken@users.noreply.github.com>

Co-authored-by: Erhan Kesken <ekesken@users.noreply.github.com>

* Fix notebook base image gpg keys (#6138)

* update links for new testing infra (#6144)

* Add more root OWNERS (#6146)

* Improve README for example-notebook-servers (#6165)

* jwa(front): Don't disable vendors with no GPUs (#6171)

JWA should not block users from selecting GPUs if the current cluster
nodes do not have any GPUs attached to them. We've seen users that have
autoscaled nodegroups for GPUs, so a GPU node will be added to the
cluster once a Pod has requested it.

Refs: arrikto/dev#1484

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* Bump Golang version in PodDefaults, TensorBoard Controller and KFAM to 1.17 (#6180)

* kfam: Upgrade go to 1.17

Update to a more recent docker image that has a newer version of
openssl.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* poddefaults: Upgrade go to 1.17

Update to a more recent docker image that has a newer version of
openssl.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* tensorboards: Upgrade go to 1.17

Update to a more recent docker image that has a newer version of
openssl.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* Update Notebooks README regarding custom packages (#6176)

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* Fix reference to PodPreset as PodDefault (#6184)

* fix: tensorboard-controller is killed due to out of memory (#6148)

* Update manager.yaml

* Update manager.yaml

* fix(web-apps): Use relative path for --deploy-url build. #5566 (#6206)

* fix(web-apps): Use relative path for --base-href build

* volume web app should use volume base-href

* Document for cutting Kubeflow website release. (#6198)

* Document for cutting Kubeflow website release.

* Update docs_dev/releasing.md

Co-authored-by: Shannon Bradshaw <shannon.bradshaw@arrikto.com>

* Update docs_dev/releasing.md

Co-authored-by: Shannon Bradshaw <shannon.bradshaw@arrikto.com>

* address comment

* canonical name

Co-authored-by: Shannon Bradshaw <shannon.bradshaw@arrikto.com>

* Change namespace label for Katib metrics collector injection (#6225)

* fix(backend): tensorboard-controller does not work because of missing permissions (#6216)

* add(web-apps): i18n Fr translations (#6122)

* chore(docs): It's better to refer to the cli documents, not repo's readme (#6239)

* fix(docs): Fix the 404 K8s docs link (#6236)

* nit: typo in README.md (#6211)

* web-apps(front) : change error msg for min (#6193)

* Centraldashboard/add uncaughtexception handler (#6203)

* Central dashboard - Add global uncaught exception handler

* fix comments

* Fix(frontend): Upgrade ts node to v4 (#6242)

* Upgrade ts-node to v10

* Update lock

* admission-webhook-manifests: Update the cert manager from v1alpha2 to v1 (#6230)

* Short term updates to Published Kubeflow Roadmap (#6248)

3 changes - Add info on Kubeflow 1.5 and 1.4.1.   Update roadmap to show that Kubeflow 1.4 was delivered.

* Upgrade the google cloud monitoring node.js client (#6210)

* Upgrade the google cloud monitoring node.js client

* Update package-lock.json

* Upgrade TypeScript to v4

* Update package-lock.json

* run tslint

* fix dashboard sending malformed headers to kfam (#6186)

* correct typo - track to tracked (#6264)

corrected typo "track" to "tracked"

* centraldashboard: Update node and use latest-stable (#6260)

Change the tests to NOT fetch Chromium from the Edge branch, which is develop,
but instead use latest-stable. We saw that edge can have problems from times to
times.

In the same commit we also update the node version to fix CVEs with the current
v12.18.3 version that we had.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* Tell user to select a namespace to access namespaced menu item (#6181)

* Fix unnecessary updates of istio AuthorizationPolicy (#6269)

* Introduce a basic proposals flow for more advanced features (#6272)

* Introduce a proposals dir for Notebooks

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* Move the proposals dir under components

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* Secure access to KFAM (#6077)

* fix(notebooks) make culling work with multi-user (#5128) (#5980)

* JWA: Very basic E2E tests to run locally (#6005)

* web-apps(front): Add test attrs selectors

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(front): Update gitignore for cypress files

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(front): Install cypress & npm scripts for e2e

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(e2e): Add first cypress tests

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa: Update the README for e2e instructions

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* chore: remove Bobgy from OWNERS

* frontend: Use responsive tables instead of cards (#6316)

* jwa(front): Make ng lint work by ignoring e2e/tsconfig.json

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>
Reviewed-by: Tasos Alexiou <tasos@arrikto.com>

* web-apps(front): Make the table responsive

We introduced all of our changes in the underlying TableComponent. This
component can then be either used independently or inside a material
card.

The changes we did in that component are:
1. Create Output() emitters, since this component can be used directly
2. The config object for a Table row now supports a `style` prop for
   defining the list of CSS styles to be applied
2. Remove the truncate classes (small, medium, large) and only have
   a boolean value. The user can define the width directly now via the
   `style` property in the row's config
3. Modify the classes for aligning contents right and left

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>
Reviewed-by: Tasos Alexiou <tasos@arrikto.com>

* web-apps(front): Add table paginator

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>
Reviewed-by: Tasos Alexiou <tasos@arrikto.com>

* web-apps(front): Add padding to titlebar text

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>
Reviewed-by: Tasos Alexiou <tasos@arrikto.com>

* web-apps(front): Use bigger font for toolbar title

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>
Reviewed-by: Tasos Alexiou <tasos@arrikto.com>

* jwa(front): Use responsive table and toolbar

We refactor both the index and form pages of the app to:
1. Add a top row toolbar with the title of the app and the button to
   create a new Notebook
2. Replace the card with a responsive table that shows the items. The
   component also has a paginator

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>
Reviewed-by: Tasos Alexiou <tasos@arrikto.com>

* vwa(front): Use responsive table and toolbar

This commit:
1. Adds a toolbar at the top of the index page with the title of the app
   and the button to create a new volume
2. Replaces the card with a responsive table

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>
Reviewed-by: Tasos Alexiou <tasos@arrikto.com>

* twa(front): Use responsive table and toolbar

This commit:
1. Adds a toolbar at the top of the index page with the title of the app
   and a button to create a new TensorBoard instance
2. Replaces the card with a resopnsive table

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>
Reviewed-by: Tasos Alexiou <tasos@arrikto.com>

* fix the format

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* wa(front): Add npm script for running unit tests in docker

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* fix unit tests failing

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* Don't use asterisk in Dockerfiles (#6318)

* wa(docker): Don't copy node_modules

To ensure Kaniko is not copying stale node_moduels folders, even though
we have a dockerignore file, we are explicitly only copying the source
code.

We have seen the build system with Kaniko to fail, due to NFS stale
instances with files in node_modules and expect that this is the root
cause.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* twa: Update makefile to use global dockerignore

The dockerfile for TWA was copying over the local dockerignore. This was
overriding the global one we had for all the web apps.

This commit updates the Makefile of the app to use the global
dockerignore that all the apps should use.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* Bump log4js from 6.3.0 to 6.4.1 in /components/crud-web-apps/common/frontend/kubeflow-common-lib (#6309)

Bumps [log4js](https://github.com/log4js-node/log4js-node) from 6.3.0 to 6.4.1.
- [Release notes](https://github.com/log4js-node/log4js-node/releases)
- [Changelog](https://github.com/log4js-node/log4js-node/blob/master/CHANGELOG.md)
- [Commits](https://github.com/log4js-node/log4js-node/compare/v6.3.0...v6.4.1)

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

Signed-off-by: dependabot[bot] <support@github.com>

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

* Bump postcss from 7.0.16 to 7.0.39 in /components/centraldashboard (#6308)

Bumps [postcss](https://github.com/postcss/postcss) from 7.0.16 to 7.0.39.
- [Release notes](https://github.com/postcss/postcss/releases)
- [Changelog](https://github.com/postcss/postcss/blob/7.0.39/CHANGELOG.md)
- [Commits](https://github.com/postcss/postcss/compare/7.0.16...7.0.39)

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

Signed-off-by: dependabot[bot] <support@github.com>

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

* Bump log4js from 6.3.0 to 6.4.1 in /components/crud-web-apps/tensorboards/frontend (#6311)

Bumps [log4js](https://github.com/log4js-node/log4js-node) from 6.3.0 to 6.4.1.
- [Release notes](https://github.com/log4js-node/log4js-node/releases)
- [Changelog](https://github.com/log4js-node/log4js-node/blob/master/CHANGELOG.md)
- [Commits](https://github.com/log4js-node/log4js-node/compare/v6.3.0...v6.4.1)

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

Signed-off-by: dependabot[bot] <support@github.com>

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

* Bump log4js from 6.3.0 to 6.4.1 in /components/crud-web-apps/volumes/frontend (#6310)

Bumps [log4js](https://github.com/log4js-node/log4js-node) from 6.3.0 to 6.4.1.
- [Release notes](https://github.com/log4js-node/log4js-node/releases)
- [Changelog](https://github.com/log4js-node/log4js-node/blob/master/CHANGELOG.md)
- [Commits](https://github.com/log4js-node/log4js-node/compare/v6.3.0...v6.4.1)

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

Signed-off-by: dependabot[bot] <support@github.com>

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

* Bump log4js from 6.3.0 to 6.4.1 in /components/crud-web-apps/jupyter/frontend (#6312)

Bumps [log4js](https://github.com/log4js-node/log4js-node) from 6.3.0 to 6.4.1.
- [Release notes](https://github.com/log4js-node/log4js-node/releases)
- [Changelog](https://github.com/log4js-node/log4js-node/blob/master/CHANGELOG.md)
- [Commits](https://github.com/log4js-node/log4js-node/compare/v6.3.0...v6.4.1)

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

Signed-off-by: dependabot[bot] <support@github.com>

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

* Update images for the 1.5 rc0 release (#6319)

* Update the releasing version tag

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* Run automated script for updating versions

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* Fix(manifests): Upgrade rbac.authorization.k8s.io from v1beta1 to v1 (#6261)

* proposal: Extend Notebook Controller to expose idleness for Jupyter (#6295)

* proposal: Extend Notebook Controller to expose idleness for Jupyter (#6270)

Provide a design doc as a proposal for extending Notebook Controller to
expose idleness for Jupyter. Our proposal is in markdown format and follows
the guidelines of the kubeflow/components/proposal/README.md guide.

You can view the #6270 issue in the following link:
https://github.com/kubeflow/kubeflow/issues/6270

Signed-off-by: Athanasios Markou <athamark@arrikto.com>

* review: change the title of the proposal

Change the title of the proposal to only include the
proposed new feature. The new title of the proposal
will now be "Expose Idleness Information for Jupyter
Notebooks".

* review: rename the proposal markdown file

We want to give a more specific name to the markdown
which contains the proposal. Since this proposal
emphasizes on a feature regarding the Jupyter Notebooks,
the new name will be:

20220121-jupyter-notebook-idleness.md

* Synchronize jupyter-web-application role with clusterrole (#6241)

* Update role.yaml

* Update role.yaml

* Update cluster-role.yaml

* Kubeflow Roadmap update - with 1.5 details (#6266)

* Kubeflow Roadmap update - with 1.5 details

These proposed changes include: identifying that 1.4.1 has been delivered, provides themes for 1.5 and provides details of major features in 1.5 by working group.   This is an initial proposal that needs review by the working group leads.

* correct formatting in KFP features

Moved KFP features under KFP Control Flow doc

* updating KFP section

updating KFP references with updates from KFP team

* Updated the 1.5 release date to March

updated the 1.5 release date to March

* Update ROADMAP.md

change Hyperparameter leader election to Katib leader election

Co-authored-by: Andrey Velichkevich <andrey.velichkevich@gmail.com>

* Update ROADMAP.md

improve description and details of feature for metrics collector

Co-authored-by: Andrey Velichkevich <andrey.velichkevich@gmail.com>

* Update Katib description for Early stopping in 1.5

updating with Andrey's suggestion (but without the world proper).   * Validation for Early Stopping algorithm settings helps users to proper reduce model overfitting

Co-authored-by: Andrey Velichkevich <andrey.velichkevich@gmail.com>

* notebooks: Extend Notebook Controller to expose idleness for Jupyter (#6297)

* notebooks: Update image's tag in make

Modify Makefile to update properly the TAG
based on the git TAG.

Signed-off-by: Athanasios Markou <athamark@arrikto.com>
Reviewed-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* notebooks: Expose last-activity

Extend the notebook-controller to:
* cull idle Notebook Servers based on their new `last-activity`
  annotation
* expose the last activity of each Notebook Server as an annotation
  on the metadata of the corresponding CR object

Modify notebook_controller.go to:
* update the Last Activity of each Notebook Server that has a
  Running pod
* delete the Last Activity Annotation for every Notebook Server
  that does not have a Running pod

Extend culler.go to:
* perform culling based on the new `last-activity` annotation and
  not based on the `/api/status` endpoint.
* update the last activity of a Notebook Server, based on the
  kernels' execution states.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>
Reviewed-by: Athanasios Markou <athamark@arrikto.com>

* notebooks: Introduce a DEV env var

We introduce a DEV ENV var to allow admins
develop and test on their local machine their
custom Notebook Controller.
We provide information and instructions inside
the components/notebook-controller/README.md.

Signed-off-by: Athanasios Markou <athamark@arrikto.com>
Reviewed-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* notebooks: Add unit tests for last-activity

* Introduce new tests for allKernelsAreIdle()
* Extend the tests for NotebookIsIdle() and for
  NotebookNeedsCulling().

Signed-off-by: Athanasios Markou <athamark@arrikto.com>
Reviewed-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* review: UpdateNotebookLastActivityAnnotation()

Ensure that UpdateNotebookLastActivityAnnotation() does not return
"true". This function should not return any value.

Signed-off-by: Athanasios Markou <athamark@arrikto.com>

* jwa: Rework the Storage API of the web app (#6321)

* wa(back): Add helper for deserializing JSON obj

In some cases we might need to construct Python k8s lib objects from the
JSONs that are provided by clients. I.e. the UI will be sending a PVC
object in json format, so the backend will need to create the
corresponding client.V1PersistentVolumeClaim object and submit it.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>
Reviewed-by: Ilias Katsakioris <elikatsis@arrikto.com>

* wa(back): Serialization helper

Add helper function for converting a k8s-client object into a dict that
can be sent as an HTTP response.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>
Reviewed-by: Ilias Katsakioris <elikatsis@arrikto.com>

* wa(back): Add dry run to Notebooks and PVCs

The backend will need to be able to create objects with dry-run, in
order to ensure they are valid. The backend will need to check that both
the Notebook and the PVCs can be created beforehand.

This way we avoid the scenario where we create PVCs but the Notebook
fails to be created, and the PVCs are never garbage collected.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>
Reviewed-by: Ilias Katsakioris <elikatsis@arrikto.com>

* wa(back): Update kubernetes to 0.17

In order to support dry-run we must use the 0.17 version of the Python
k8s client.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>
Reviewed-by: Ilias Katsakioris <elikatsis@arrikto.com>

* wa(back): Extend api module to patch pvcs

The backend will need to be able to PATCH PVCs in order to set the
ownerReference to the Notebook that mounts the PVCs.

Ref: arrikto/dev/issues/386#issuecomment-856700392

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>
Reviewed-by: Ilias Katsakioris <elikatsis@arrikto.com>

* jwa(back): Work with new Volumes API

The backend API should not add any more layers of abstractions on top of
the K8s API. The backend should expect the client/UI to be sending the
entire PVC spec of a new PVC.

Refs: arrikto/dev/issues/386

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>
Reviewed-by: Ilias Katsakioris <elikatsis@arrikto.com>

* jwa(back): Add unittests for new volumes API

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>
Reviewed-by: Ilias Katsakioris <elikatsis@arrikto.com>

* jwa(back): Extend the PVC info returned

We want to show both the access mode and size of the existing PVCs, when
a user clicks on the dropdown to select which PVC to mount.

The backend will need to provide this information to the frontend. We
don't want to send the K8s list of PVCs since this will result in a lot
of unnecessary data to be sent.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>
Reviewed-by: Ilias Katsakioris <elikatsis@arrikto.com>

* jwa(front): Add proxy config for Rok

When developing the Rok flavor locally we will need to be able to open
the Rok chooser. This can be done by using Angular/webpack proxy to
bring the exposed rok service and the app under the same domain.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>
Reviewed-by: Tasos Alexiou <tasos@arrikto.com>

* jwa(front): Remove card from form

The form of the app should not be a big card, but a normal form.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>
Reviewed-by: Tasos Alexiou <tasos@arrikto.com>

* jwa(front): Install AceModule for yaml editing

Install AceModule to allow users to edit yamls of objects.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>
Reviewed-by: Tasos Alexiou <tasos@arrikto.com>

* wa(front): Change the styling of form sections

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>
Reviewed-by: Tasos Alexiou <tasos@arrikto.com>

* jwa(front): Create common volume components

Component for:
* New PVC and configuring its spec
* Attaching an existing PVC in a Notebook

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>
Reviewed-by: Tasos Alexiou <tasos@arrikto.com>

* jwa(front): Update Rok form for new Volume API

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>
Reviewed-by: Tasos Alexiou <tasos@arrikto.com>

* jwa(front): Mark inputs as dirty when restoring Lab

When the UI autofills the form with values from a JupyterLab snapshot
then it should mark the touched fields as dirty. This way if a field has
errors the UI will make that input red.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>
Reviewed-by: Tasos Alexiou <tasos@arrikto.com>

* jwa: Update ConfigMap in manifests

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* jwa(front): Fix format errors

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* profiles: Update the permissions for notebook idleness (#6335)

Extend the Profiles Controller to give permissions to Notebooks
controller for making GET requests to notebook's /api/kernels endpoint.

Refs https://github.com/kubeflow/kubeflow/blob/master/components/proposals/20220121-jupyter-notebook-idleness.md

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* notebooks: Graceful handling of events (#6338)

* notebooks: Handle events gracefully

The controller is not exiting the reconciliation loop after it has
re-emitted a Pod/STS Event as a Notebook Event. This results in the
controller to later on try and GET a Notebook with the name of the Event
that triggered the reconciliation loop.

The controller should exit the reconciliation function once it has
emitted the event.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* notebooks: Don't reconcile on deleted events

We don't want to trigger the reconciliation function when an event gets
deleted.

If a Notebook would be deleted then the underlying events would
be deleted as well, which results in the reconcile function to get
triggered and try to GET Events and Notebooks with the name of the
deleted event.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* notebooks: Fix endless restarts (#6341)

* notebooks: Update notebook if timestamp changed

We don't want to be updating the spec of the notebook if the timestamp
hasn't changed, since this will lead to constant updates and
reconciliation loops.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* notebooks: Use a deep-copy of the notebook spec

The controller should use a deep-copy of the notebook spec when
calculating the spec for the StatefulSet. If not then we could
update the notebook object without wanting it, since the spec could have
been changed when calculating the STS spec.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* notebooks: Add prefix env var only if missing

The controller should be setting OR updating the NB_PREFIX env var.
Previously it would always blindly append it to the spec, which could
result in double entries for the same env var.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* releasing: Update tags for v1.5.0-rc.1 (#6343)

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* fix error comparison between pointer and pointer in "CopyStatefulSetFields" (#6195)

error comparison between pointer and pointer in "CopyStatefulSetFields"

* [bugfix]: update unsupported links to latest pages (#6192)

* centraldashboard: Add KServe overlay (#6383)

Configure the dashboard to use the KServe app instead of the KFServing
0.6.1 one.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* notebooks: Don't reconcile on Events deletion (#6391)

The controller should not trigger the reconcile loop when an Event is
deleted. Previously the controller would run the reconciliation loop on
any event deletion.

This commit updates it to not run the loop for ANY event.

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* releasing: Create v1.5.0-rc.2 images (#6394)

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* release: Images for the 1.5.0 tag (#6398)

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* added env for app secure cookies (#6395)

* build(deps): bump github.com/gogo/protobuf from 1.1.1 to 1.3.2 in /components/tensorboard-controller (#6424)

Bumps [github.com/gogo/protobuf](https://github.com/gogo/protobuf) from 1.1.1 to 1.3.2.
- [Release notes](https://github.com/gogo/protobuf/releases)
- [Commits](https://github.com/gogo/protobuf/compare/v1.1.1...v1.3.2)

---
updated-dependencies:
- dependency-name: github.com/gogo/protobuf
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <support@github.com>

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

* build(deps): bump notebook from 6.4.0 to 6.4.10 in /components/example-notebook-servers/jupyter (#6431)

Bumps [notebook](http://jupyter.org) from 6.4.0 to 6.4.10.

---
updated-dependencies:
- dependency-name: notebook
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <support@github.com>

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

* build(deps): bump minimist from 1.2.5 to 1.2.6 in /components/crud-web-apps/tensorboards/frontend (#6417)

Bumps [minimist](https://github.com/substack/minimist) from 1.2.5 to 1.2.6.
- [Release notes](https://github.com/substack/minimist/releases)
- [Commits](https://github.com/substack/minimist/compare/1.2.5...1.2.6)

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

Signed-off-by: dependabot[bot] <support@github.com>

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

* build(deps): bump minimist from 1.2.5 to 1.2.6 in /components/crud-web-apps/jupyter/frontend (#6429)

Bumps [minimist](https://github.com/substack/minimist) from 1.2.5 to 1.2.6.
- [Release notes](https://github.com/substack/minimist/releases)
- [Commits](https://github.com/substack/minimist/compare/1.2.5...1.2.6)

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

Signed-off-by: dependabot[bot] <support@github.com>

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

* build(deps): bump follow-redirects from 1.14.1 to 1.14.8 in /components/crud-web-apps/tensorboards/frontend (#6357)

Bumps [follow-redirects](https://github.com/follow-redirects/follow-redirects) from 1.14.1 to 1.14.8.
- [Release notes](https://github.com/follow-redirects/follow-redirects/releases)
- [Commits](https://github.com/follow-redirects/follow-redirects/compare/v1.14.1...v1.14.8)

---
updated-dependencies:
- dependency-name: follow-redirects
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <support@github.com>

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

* build(deps): bump ajv from 6.10.2 to 6.12.6 in /components/crud-web-apps/volumes/frontend (#6358)

Bumps [ajv](https://github.com/ajv-validator/ajv) from 6.10.2 to 6.12.6.
- [Release notes](https://github.com/ajv-validator/ajv/releases)
- [Commits](https://github.com/ajv-validator/ajv/compare/v6.10.2...v6.12.6)

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

Signed-off-by: dependabot[bot] <support@github.com>

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

* build(deps): bump ajv from 6.10.2 to 6.12.6 in /components/crud-web-apps/jupyter/frontend (#6359)

Bumps [ajv](https://github.com/ajv-validator/ajv) from 6.10.2 to 6.12.6.
- [Release notes](https://github.com/ajv-validator/ajv/releases)
- [Commits](https://github.com/ajv-validator/ajv/compare/v6.10.2...v6.12.6)

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

Signed-off-by: dependabot[bot] <support@github.com>

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

* build(deps): bump lodash-es from 4.17.11 to 4.17.21 in /components/crud-web-apps/tensorboards/frontend (#6402)

Bumps [lodash-es](https://github.com/lodash/lodash) from 4.17.11 to 4.17.21.
- [Release notes](https://github.com/lodash/lodash/releases)
- [Commits](https://github.com/lodash/lodash/compare/4.17.11...4.17.21)

---
updated-dependencies:
- dependency-name: lodash-es
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <support@github.com>

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

* build(deps): bump minimist from 1.2.5 to 1.2.6 in /components/crud-web-apps/common/frontend/kubeflow-common-lib (#6416)

Bumps [minimist](https://github.com/substack/minimist) from 1.2.5 to 1.2.6.
- [Release notes](https://github.com/substack/minimist/releases)
- [Commits](https://github.com/substack/minimist/compare/1.2.5...1.2.6)

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

Signed-off-by: dependabot[bot] <support@github.com>

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

* build(deps): bump minimist from 1.2.5 to 1.2.6 in /components/crud-web-apps/volumes/frontend (#6423)

Bumps [minimist](https://github.com/substack/minimist) from 1.2.5 to 1.2.6.
- [Release notes](https://github.com/substack/minimist/releases)
- [Commits](https://github.com/substack/minimist/compare/1.2.5...1.2.6)

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

Signed-off-by: dependabot[bot] <support@github.com>

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

* build(deps): bump undefsafe from 2.0.2 to 2.0.5 in /components/centraldashboard (#6342)

Bumps [undefsafe](https://github.com/remy/undefsafe) from 2.0.2 to 2.0.5.
- [Release notes](https://github.com/remy/undefsafe/releases)
- [Commits](https://github.com/remy/undefsafe/compare/v2.0.2...v2.0.5)

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

Signed-off-by: dependabot[bot] <support@github.com>

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

* build(deps): bump url-parse from 1.5.1 to 1.5.10 in /components/crud-web-apps/common/frontend/kubeflow-common-lib (#6378)

Bumps [url-parse](https://github.com/unshiftio/url-parse) from 1.5.1 to 1.5.10.
- [Release notes](https://github.com/unshiftio/url-parse/releases)
- [Commits](https://github.com/unshiftio/url-parse/compare/1.5.1...1.5.10)

---
updated-dependencies:
- dependency-name: url-parse
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <support@github.com>

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

* build(deps): bump url-parse from 1.5.1 to 1.5.10 in /components/crud-web-apps/tensorboards/frontend (#6379)

Bumps [url-parse](https://github.com/unshiftio/url-parse) from 1.5.1 to 1.5.10.
- [Release notes](https://github.com/unshiftio/url-parse/releases)
- [Commits](https://github.com/unshiftio/url-parse/compare/1.5.1...1.5.10)

---
updated-dependencies:
- dependency-name: url-parse
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <support@github.com>

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

* build(deps-dev): bump karma from 6.3.4 to 6.3.16 in /components/crud-web-apps/common/frontend/kubeflow-common-lib (#6386)

Bumps [karma](https://github.com/karma-runner/karma) from 6.3.4 to 6.3.16.
- [Release notes](https://github.com/karma-runner/karma/releases)
- [Changelog](https://github.com/karma-runner/karma/blob/master/CHANGELOG.md)
- [Commits](https://github.com/karma-runner/karma/compare/v6.3.4...v6.3.16)

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

Signed-off-by: dependabot[bot] <support@github.com>

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

* build(deps-dev): bump karma from 6.3.4 to 6.3.16 in /components/crud-web-apps/tensorboards/frontend (#6387)

Bumps [karma](https://github.com/karma-runner/karma) from 6.3.4 to 6.3.16.
- [Release notes](https://github.com/karma-runner/karma/releases)
- [Changelog](https://github.com/karma-runner/karma/blob/master/CHANGELOG.md)
- [Commits](https://github.com/karma-runner/karma/compare/v6.3.4...v6.3.16)

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

Signed-off-by: dependabot[bot] <support@github.com>

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

* build(deps): bump moment from 2.24.0 to 2.29.2 in /components/centraldashboard (#6432)

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

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

Signed-off-by: dependabot[bot] <support@github.com>

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

* build(deps): bump url-parse from 1.5.1 to 1.5.10 in /components/crud-web-apps/volumes/frontend (#6380)

Bumps [url-parse](https://github.com/unshiftio/url-parse) from 1.5.1 to 1.5.10.
- [Release notes](https://github.com/unshiftio/url-parse/releases)
- [Commits](https://github.com/unshiftio/url-parse/compare/1.5.1...1.5.10)

---
updated-dependencies:
- dependency-name: url-parse
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <support@github.com>

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

* build(deps): bump url-parse from 1.4.7 to 1.5.10 in /components/centraldashboard (#6377)

Bumps [url-parse](https://github.com/unshiftio/url-parse) from 1.4.7 to 1.5.10.
- [Release notes](https://github.com/unshiftio/url-parse/releases)
- [Commits](https://github.com/unshiftio/url-parse/compare/1.4.7...1.5.10)

---
updated-dependencies:
- dependency-name: url-parse
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <support@github.com>

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

* build(deps-dev): bump karma from 6.3.4 to 6.3.16 in /components/crud-web-apps/volumes/frontend (#6388)

Bumps [karma](https://github.com/karma-runner/karma) from 6.3.4 to 6.3.16.
- [Release notes](https://github.com/karma-runner/karma/releases)
- [Changelog](https://github.com/karma-runner/karma/blob/master/CHANGELOG.md)
- [Commits](https://github.com/karma-runner/karma/compare/v6.3.4...v6.3.16)

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

Signed-off-by: dependabot[bot] <support@github.com>

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

* build(deps-dev): bump karma from 6.3.4 to 6.3.16 in /components/crud-web-apps/jupyter/frontend (#6389)

Bumps [karma](https://github.com/karma-runner/karma) from 6.3.4 to 6.3.16.
- [Release notes](https://github.com/karma-runner/karma/releases)
- [Changelog](https://github.com/karma-runner/karma/blob/master/CHANGELOG.md)
- [Commits](https://github.com/karma-runner/karma/compare/v6.3.4...v6.3.16)

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

Signed-off-by: dependabot[bot] <support@github.com>

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

* build(deps): bump follow-redirects from 1.14.1 to 1.14.9 in /components/crud-web-apps/jupyter/frontend (#6441)

Bumps [follow-redirects](https://github.com/follow-redirects/follow-redirects) from 1.14.1 to 1.14.9.
- [Release notes](https://github.com/follow-redirects/follow-redirects/releases)
- [Commits](https://github.com/follow-redirects/follow-redirects/compare/v1.14.1...v1.14.9)

---
updated-dependencies:
- dependency-name: follow-redirects
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <support@github.com>

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

* build(deps): bump nanoid from 3.1.23 to 3.3.2 in /components/crud-web-apps/common/frontend/kubeflow-common-lib (#6437)

Bumps [nanoid](https://github.com/ai/nanoid) from 3.1.23 to 3.3.2.
- [Release notes](https://github.com/ai/nanoid/releases)
- [Changelog](https://github.com/ai/nanoid/blob/main/CHANGELOG.md)
- [Commits](https://github.com/ai/nanoid/c…
@wg102 wg102 deleted the angular-ngx-translate branch December 13, 2022 14:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Implement OL in Crud-Web-Apps Upstream
7 participants