Commit Graph

5153 Commits

Author SHA1 Message Date
Wenkai Yin
6704828f19
Merge pull request #5411 from wy65701436/fix-notary
Add Notary server endpoint as a parameter for the CI test cases
2018-07-30 16:31:27 +08:00
Wenkai Yin
48489bf7ae
Merge pull request #5424 from steven-zou/add_more_ut_cases
Add more UT cases for chart repo API controller
2018-07-30 15:20:22 +08:00
Mia ZHOU
52634dcec4 Fix remove button not clickable
Problem description:
In member tab, Action->remove is not clickable in production mode.

Root cause analysis:

Some style for clarity checkbox impact dropdown menu.

Fix:
We do a quick fix in member.component.scss. Set checkbox in this component
position:inherit;

This will overide the clarity original style
2018-07-30 14:01:19 +08:00
Steven Zou
9f37a66f21
Merge pull request #5417 from ywk253100/180716_redis_cross_slot
Fix cross slot issue of redis
2018-07-30 13:50:41 +08:00
Steven Zou
34db0d9c8a Add more UT cases for chart repo API controller
case for testing require access
case for testing require namespace
case for testing multipart checking
case for testing URL rewriting
2018-07-30 13:25:11 +08:00
wangyan
c3c56ada9a Add Notary server endpoint as a parameter for the CI test cases 2018-07-29 19:45:28 -07:00
Daniel Jiang
5f83562a12
Merge pull request #5415 from reasonerjt/cfg-migrator-1.6
Enable cfg migrator to 1.6.0
2018-07-30 09:20:07 +08:00
Daniel Jiang
46de1432f2 Enable cfg migrator to 1.6.0
In 1.6, there will be only one DB process in the default deployment.
The migrator will try to handle the setting by "guessing" whether Harbor
was pointed to external DB.

Verified 1.5->1.6 and 1.4->1.6 migration.
2018-07-27 17:11:45 +08:00
Daniel Jiang
30fe6c8595
Merge pull request #5413 from ywk253100/180727_chart_bugfix
Fix misc issues of Harbor chart
2018-07-27 17:09:24 +08:00
Wenkai Yin
e9f8db79d6
Merge pull request #5412 from steven-zou/supporting_prov_status
Provide digital signature info of the related chart version
2018-07-27 16:07:31 +08:00
Wenkai Yin
5772a61e5d Fix misc issues of Harbor chart
1. Read value of storage driver from values.yaml for adminserver
2. Set Redis as UI cache
3. Set Redis as registry cache
2018-07-27 15:28:22 +08:00
Steven Zou
4b1f0470ca Provide digital signature info of the related chart version 2018-07-27 15:07:22 +08:00
Wenkai Yin
2a40068d75 Fix cross slot issue of redis
This commit upgrdes the github.com/gocraft/work package to 0.5.1 and updates the namespace value
to fix the redis cross slot issue mentioned in #4633
2018-07-27 14:23:39 +08:00
Wenkai Yin
c3106fc447
Merge pull request #5401 from reasonerjt/config-clair-interval
Enable configuring the interval of clair updaters
2018-07-26 18:18:16 +08:00
Wenkai Yin
b8979b09e8
Merge pull request #5400 from reasonerjt/update-no-proxy
Add registry to default no_proxy hosts for Clair
2018-07-26 16:36:55 +08:00
Daniel Jiang
733a89dea2 Enable configuring the interval of clair updaters
To mitigate the impact we saw in the updater issues in clair, this
commit enable configuring the interval, include disabling the updaters
of clair.
2018-07-26 16:27:23 +08:00
Steven Zou
cfefcb7450
Merge pull request #5399 from steven-zou/fix_access_control_issue
Fix the access control checking issue
2018-07-26 15:27:30 +08:00
Qian Deng
0f7eab38b7
Merge pull request #5384 from wy65701436/update-migrator-aj
Update migrator for admin job
2018-07-26 03:19:31 -04:00
Qian Deng
508145019e
Merge pull request #5397 from zhoumeina/project_error_msg
Replicate project error message is not correct
2018-07-26 03:18:31 -04:00
Daniel Jiang
ad0c0eba36 Add registry to default no_proxy hosts for Clair
When proxy is set for Clair, there may be issue when Clair pulls image
from the registryif the `no_proxy` attribute is not updated.  This
commit adds `registry` to the default setting.
2018-07-26 14:41:03 +08:00
Mia ZHOU
4ff45b0624 Use capitalize in configuration dropdown menu
Problem Description:
configuration dropdown menu use all uppercase letters.

Fix:
Modify all of them in capitalize letters.

Note:Try to use css text-transform:Caplitalize but not working, if anyone know the reason, please kindly tell me.
2018-07-26 14:40:12 +08:00
Steven Zou
01fb95062c Fix the access control checking issue
should check access first, then check authentication
return 401/403 accroding to the checking results properly
2018-07-26 14:28:16 +08:00
Mia ZHOU
397af582a2 Replicate project error message is not correct
Problem description:
when user try to new a replication rule, he need to input source project name.
In this case, if user input a project name not exist. The error message will be:"Please add a project name",
this is not accurate.

Fix:
Modify the error message to "This project is not exist".

Plus:Modify the error popup using standard tooltip component.
2018-07-26 13:37:45 +08:00
Steven Zou
c1e1779ac0
Merge pull request #5396 from reasonerjt/scan-all-no-clair
No scan all policy if Clair is not installed.
2018-07-26 12:33:23 +08:00
Yan
9524c70f5e
Fix error in package offline installe (#5395)
It has a redundant slash in the end of docker save command,
that causes the command with merge the following lines into
save cmd, and then to fail package offline installer.
2018-07-26 10:09:22 +08:00
Daniel Jiang
97fb67f835 No scan all policy if Clair is not installed.
This commit fixes #5283 by not registering scan all policy when Harbor
is not deployed with Clair.
2018-07-25 19:52:16 +08:00
Mia ZHOU
9fba539bd4
Merge pull request #5392 from zhoumeina/new_label_fix
Fix configuration->label->new sometimes not enable.
2018-07-25 18:32:16 +08:00
Mia ZHOU
60f83843e3 Fix configuration->label->new sometimes not enable.
Problem description:
1 label enable sometimes not correct. User input good label names, submit button still disabled.
2 Some times click submit, not working.

Fix:

1 After review the new label code. It trying to use a global variable to control the progress of validation. In the progress of validation, it will disabled the submit button.

This will cause user bad experience. Just remove this logic.

2 Backend use fuzzy matching, the frontend logic did not change before. So when user already have a user name called "abcd",and then input another name called "abc" it will report duplicated name error.
This logic still need change.

3 This is the first version of refine. Can be refine again if find some other logic not correct.
2018-07-25 17:47:22 +08:00
Yan
602bcdfa5b
remove publish npm package step in the ci flow and set the (#5391)
env for admin and pwd for ci, make it same as nightly test.
2018-07-25 17:20:40 +08:00
Steven Zou
b8c7a8cb68
Merge pull request #5393 from steven-zou/fix_content_length_issue
Set the content length to -1 to let the reader read all the uploading…
2018-07-25 17:08:32 +08:00
Steven Zou
850af80762 Set the content length to -1 to let the reader read all the uploading contents in the request 2018-07-25 16:25:44 +08:00
Daniel Jiang
2fa0cf032c
Merge pull request #5335 from ywk253100/180717_chart
Support the configuration of registry storage in helm chart
2018-07-25 16:20:31 +08:00
Daniel Jiang
b76b982ded
Merge pull request #5390 from ywk253100/180725_doc
Update the user guide
2018-07-25 15:53:43 +08:00
Wenkai Yin
4ee6f6abdf Update the user guide
This commit updates the user guide according to the comments from Louis
2018-07-25 13:04:19 +08:00
Qian Deng
e597910c6f
Merge pull request #5389 from zhoumeina/notification_ui_refine
Notification should not hide content according to ux
2018-07-25 00:35:45 -04:00
wangyan
5b492ee321 Update migrator for support new table admin_job 2018-07-24 20:29:56 -07:00
Mia ZHOU
50678523c4 Notification should not hide content according to ux
Issue description:
Notification will hide some content when it popup.

According to ux requirement, we should make it not extend to the whole widh of UI, and should not hide the content.

Fix:
Use clarity alert-app-level style,it will show as ux design,and text in center too.
2018-07-25 11:29:08 +08:00
Daniel Jiang
c6c5745183 Remove vmware/photon image from offline package (#5347)
We used to package vmware/photon:1.0 in the offline package for product
integration.  Currently this is no longer needed by downstream product.
This commit removes it.
2018-07-25 11:03:00 +08:00
Wenkai Yin
0a8602709a
Merge pull request #5387 from steven-zou/fix_content_length_issue
Update content length of request when rewriting the content
2018-07-25 10:53:50 +08:00
Steven Zou
07bfad968a Update content length of request when rewriting the content 2018-07-24 23:10:42 +08:00
Qian Deng
7ee2e1053e
Merge pull request #5383 from zhoumeina/notification_ui_refine
Notification should not hide content according to ux
2018-07-24 10:00:33 -04:00
Daniel Jiang
3a2c58394b
Merge pull request #5382 from stonezdj/ldap_admin_role_rest
Ldap_group_admin_dn can not updated via rest
2018-07-24 19:46:10 +08:00
Daniel Jiang
5426cfa687
Merge pull request #5385 from wy65701436/bump-1.6.0
Bump harbor release version to 1.6.0
2018-07-24 19:43:35 +08:00
wangyan
6ece052824 Bump harbor release version to 1.6.0 2018-07-24 03:46:05 -07:00
Mia ZHOU
2240693a1d Notification should not hide content according to ux
Issue description:
Notification will hide some content when it popup.

According to ux requirement, we should make it not extend to the whole widh of UI, and should not hide the content.

Fix:
Modify the position from 'absolute' to 'relative', so that it will not hide the content.
2018-07-24 17:56:46 +08:00
stonezdj
9b209858f4 Ldap_group_admin_dn can not updated via rest 2018-07-24 17:47:57 +08:00
Qian Deng
eafee964c1
Merge pull request #5372 from zhoumeina/fix_configuration_button_validation
Fix issue install clair the first time use scan will have undefined e…
2018-07-24 02:07:59 -04:00
Mia ZHOU
19e2a9ad1b Fix issue install clair the first time use scan will have undefined error
Description:
Reproduce Steps:
1 User install Clair
2 User use scan to scan the image
2 There will be a notification pop with undefined error
Root cause:
In result-bar-chart-component.ts  copyValue function
  this.summary.scan_status = newVal.scan_status;
It try to set scan status to summary object.
Somehow, the first time summary is undefined. It will cause this error.

Fix:
1 Do a empty check before assign value to a object.
2 Set a default value to summary.
2018-07-24 13:36:39 +08:00
silenceshell
7745b79b2e var name should not be error (#5332)
Rename the variable names from "error" to "err"
2018-07-24 11:33:21 +08:00
Daniel Jiang
e9d5913d63
Merge pull request #5368 from ywk253100/180723_chart_redis
Add support for external Redis configuration in helm chart
2018-07-24 11:25:56 +08:00