Commit Graph

5266 Commits

Author SHA1 Message Date
Daniel Jiang
bda0a92ea9
Merge pull request #5427 from ywk253100/180730_redirect
Remove the URL rewrite for docker registry request
2018-07-31 13:34:15 +08:00
Daniel Jiang
49894457ca
Merge pull request #5403 from ywk253100/180726_chart_cert
Remove the hardcoded certificate and private key in value.yaml
2018-07-31 13:09:09 +08:00
Daniel Jiang
6062bf279b Set default creation_time and update_time at model
This commit set the default value of creation_time and update_time to
data objects by adding `orm:add_now` annotations.
2018-07-31 12:56:14 +08:00
Mia ZHOU
2d9d6202d1
Merge pull request #5437 from zhoumeina/pr/fix_cardview_title
Fix Repositories card view title displayed wrong
2018-07-31 12:32:51 +08:00
simone morellato
451a51174e
Update README.md
when you checkout harbor, it checks it out to the harbor directory.
2018-07-30 20:46:38 -07:00
Mia ZHOU
1af3bcd93d Fix Repositories card view title displayed wrong
Problem Description:

In the card view of repositories, if the title is long, it can not show all the text.

Fix:

Add css

    overflow: hidden;
    text-overflow: ellipsis;
    word-wrap:break-word;
    white-space: nowrap

in mixin files, and tooltip to show all the text.
2018-07-31 10:51:42 +08:00
Daniel Jiang
4dadff070b
Merge pull request #5430 from wy65701436/fix-pgsql-creation-time
Fix pgsql creation column bug
2018-07-30 21:13:31 +08:00
wangyan
22411cf6b2 Fix pgsql creation column bug
Root cause: Use default 'now'::timestamp will not generate timestamp for each transaction,
PG will convert now to a timestamp as soon as the constant is parsed. To fix it, update it
to defult CURRENT_TIMESTAMP, thie setting is the same as default now(), which returns the
start time of current transaction because ther are fuction calls, hey will give the desired
behavior of defaulting to the time of row insertion.

Reference: https://www.postgresql.org/docs/9.6/static/functions-datetime.html#FUNCTIONS-DATETIME-CURRENT
PG version: 9.6.9
2018-07-30 04:58:44 -07:00
wangyan
5edd408d07 Add scripts to verify Harbor DB migration 2018-07-30 04:54:33 -07:00
Qian Deng
43774df6f3
Merge pull request #5426 from zhoumeina/pr/fix_remove_button_not_clickable
Fix remove button not clickable
2018-07-30 05:27:04 -04:00
Wenkai Yin
7e6a13915b Remove the URL rewrite for docker registry request
This commit redirects the request to UI directory without URL rewrite
2018-07-30 17:24:15 +08:00
Wenkai Yin
cde00b35a8 Update the icon used in Harbor chart
This commit updates the URL of the icon used in Harbor helm chart
2018-07-30 16:46:02 +08:00
Wenkai Yin
cd7044dae1 Remove the hardcoded certificate and private key in value.yaml
The certificate and private key used by token service, registry and notoary server will be generated automatically
2018-07-30 16:41:42 +08:00
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