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

Remove artifacts folder for performance #78

Closed
wants to merge 75 commits into from
Closed

Remove artifacts folder for performance #78

wants to merge 75 commits into from

Conversation

JohnNiang
Copy link
Member

@JohnNiang JohnNiang commented Apr 19, 2021

I removed artifacts folder according to this instruction: https://docs.github.com/en/github/authenticating-to-github/removing-sensitive-data-from-a-repository.

Before:

--- /home/johnniang/Workspaces/johnniang/devops-java-sample ---------------------------------------------
  114.9 MiB [##########] /artifacts                                                                      
   69.2 MiB [######    ] /.git
   64.0 KiB [          ] /deploy
   56.0 KiB [          ] /src
   12.0 KiB [          ]  LICENSE
    8.0 KiB [          ]  README.md
    8.0 KiB [          ] /configuration
    8.0 KiB [          ] /.s2i
    8.0 KiB [          ]  pom.xml
    4.0 KiB [          ]  Jenkinsfile-on-prem
    4.0 KiB [          ]  Jenkinsfile-online
    4.0 KiB [          ]  Dockerfile-on-prem
    4.0 KiB [          ]  Dockerfile-online
    4.0 KiB [          ]  OWNERS
    4.0 KiB [          ]  .gitignore

The result:

--- /home/johnniang/workspace/kubesphere/devops-java-sample ------------------------------------------------------------------------------------------------------------------------------------------------------------------------
                         /..                                                                                                                                                                                                        
  272.0 KiB [##########] /.git
   64.0 KiB [##        ] /deploy
   56.0 KiB [##        ] /src
   12.0 KiB [          ]  LICENSE
    8.0 KiB [          ]  README.md
    8.0 KiB [          ] /configuration
    8.0 KiB [          ] /.s2i
    8.0 KiB [          ]  pom.xml
    4.0 KiB [          ]  Jenkinsfile-on-prem
    4.0 KiB [          ]  Jenkinsfile-online
    4.0 KiB [          ]  Dockerfile-on-prem
    4.0 KiB [          ]  OWNERS
    4.0 KiB [          ]  Dockerfile-online
    4.0 KiB [          ]  .gitignore

Close #76
Close #74

/kind optimization

shaowenchen and others added 15 commits November 6, 2020 21:59
Signed-off-by: shaowenchen <[email protected]>
* Update Jenkinsfile-online

项目中maven 编译使用 mvn -o为离线编译。一般fork项目的都为新安装环境,此处删除-o,有maven 在线下载依赖

* Update README.md

增加项目介绍,和修改使用说明,

* Update README.md

* Update pom.xml

原始的spring-boot-starter-parent 为快照版本,更新为发行版,原始的仓库地址为spring.io官网地址,更新为阿里云国内镜像地址,

* Update README.md

添加 DCO singoff

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

* Update pom.xml

add DCO singoff

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

* Update Jenkinsfile-online

add DCO singoff

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

* Update pom.xml
Signed-off-by: PeiHongbing1223 <[email protected]>
Signed-off-by: PeiHongbing1223 <[email protected]>
Signed-off-by: PeiHongbing1223 <[email protected]>
Correct the wrong order number in the readme file
Signed-off-by: Felixnoo <[email protected]>
Signed-off-by: Felixnoo <[email protected]>
Provide an English version of the README file
@ks-ci-bot
Copy link
Collaborator

The following users are mentioned in OWNERS file(s) but are not members of the kubesphere org.

Once all users have been added as members of the org, you can trigger verification by writing /verify-owners in a comment.

  • runzexia
    • OWNERS

@ks-ci-bot
Copy link
Collaborator

Thanks for your pull request. Before we can look at it, you'll need to add a 'DCO signoff' to your commits.

📝 Please follow instructions in the contributing guide to update your commits with the DCO

Full details of the Developer Certificate of Origin can be found at developercertificate.org.

The list of commits missing DCO signoff:

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. I understand the commands that are listed here.

@ks-ci-bot
Copy link
Collaborator

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: JohnNiang

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

@JohnNiang JohnNiang deleted the perf/remove-artifacts-folder branch April 22, 2021 02:36
@JohnNiang JohnNiang restored the perf/remove-artifacts-folder branch April 22, 2021 02:36
@JohnNiang
Copy link
Member Author

For no common history commits, this pull request was close automatically by github and could not reopen again.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
8 participants