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

Test coverage is not reported on each pr #1557

Closed
amitkrout opened this issue Apr 4, 2019 · 9 comments
Closed

Test coverage is not reported on each pr #1557

amitkrout opened this issue Apr 4, 2019 · 9 comments
Assignees
Labels
area/testing Issues or PRs related to testing, Quality Assurance or Quality Engineering kind/feature Categorizes issue as a feature request. For PRs, that means that the PR is the implementation priority/Medium Nice to have issue. Getting it done before priority changes would be great.

Comments

@amitkrout
Copy link
Contributor

[kind/bug]

What versions of software are you using?

  • Operating System: CI Setup
  • Output of odo version: master

How did you find it?

Check coverage report on each pr

Actual behavior

No coverage report

Expected behavior

Test coverage should be reported on each pr

Any logs, error output, etc?

@amitkrout amitkrout self-assigned this Apr 4, 2019
@amitkrout amitkrout added kind/bug Categorizes issue or PR as related to a bug. priority/High Important issue; should be worked on before any other issues (except priority/Critical issue(s)). labels Apr 4, 2019
@kadel
Copy link
Member

kadel commented Apr 5, 2019

/kind testing
/priority medium

@openshift-ci-robot openshift-ci-robot added area/testing Issues or PRs related to testing, Quality Assurance or Quality Engineering priority/Medium Nice to have issue. Getting it done before priority changes would be great. labels Apr 5, 2019
@kadel kadel removed the priority/High Important issue; should be worked on before any other issues (except priority/Critical issue(s)). label Apr 5, 2019
@kanchwala-yusuf
Copy link
Contributor

@amitkrout , Can you please provide more details as to what is expected here? Can you please elaborate? Thanks

@amitkrout
Copy link
Contributor Author

@kanchwala-yusuf Early days odo repo was being maintained in github.com/redhat-developer/odo org, later it moved to OpenShift org which we currently working on. During the repo migration it seem certain make target does not work for us. For example, each pr run executes code coverage target [1] but the report does not reflected in pr like it was reflecting before [2]. You can refer [3] for more understanding.

[1] https://github.com/openshift/odo/blob/master/.travis.yml#L26
[2] #818 (comment)
[3] https://docs.codecov.io/docs/pull-request-comments

NOTE: you may need write access to make the changes in odo repo setting. @kadel can you please grant write access to @kanchwala-yusuf

@kanchwala-yusuf
Copy link
Contributor

kanchwala-yusuf commented Nov 27, 2019

@kanchwala-yusuf Early days odo repo was being maintained in github.com/redhat-developer/odo org

Is this is the link you are referring to:
https://github.com/redhat-developer/odo-fork

@amitkrout
Copy link
Contributor Author

@kanchwala-yusuf Early days odo repo was being maintained in github.com/redhat-developer/odo org

Is this is the link you are referring to:
https://github.com/redhat-developer/odo-fork

yes

@kadel kadel added the points/3 label Nov 27, 2019
kanchwala-yusuf added a commit to kanchwala-yusuf/odo that referenced this issue Nov 28, 2019
2. Changes to generate-coverage.sh for fixing errors

Fixes redhat-developer#1557
kanchwala-yusuf added a commit to kanchwala-yusuf/odo that referenced this issue Nov 29, 2019
2. Changes to generate-coverage.sh for fixing errors

Fixes redhat-developer#1557
kanchwala-yusuf added a commit to kanchwala-yusuf/odo that referenced this issue Nov 29, 2019
2. Changes to generate-coverage.sh for fixing errors

Fixes redhat-developer#1557
@kanchwala-yusuf
Copy link
Contributor

Codecov provides code coverage reports.

Codecov has two types of settings to be done:

  1. For all the repositories in the organization.

  2. For local repository

  3. For all the repositories, codecov requires:

  1. For local repository

I observe that the local repository requirements are fulfilled by the openshift/odo repository.

We may need some help in setting up requirements for all repositories as specified in section #1.

@kadel
Copy link
Member

kadel commented Dec 2, 2019

Why all repositories in org? It is not possible to enable it for just one single repo?

@kadel kadel removed the kind/bug Categorizes issue or PR as related to a bug. label Dec 2, 2019
@kanchwala-yusuf
Copy link
Contributor

kanchwala-yusuf commented Dec 3, 2019

Why all repositories in org? It is not possible to enable it for just one single repo?

So CodeCov has two types of settings:

  1. Global (I made up this word) settings which are common to all the repositories. These are configured in a file called "Team Yaml"
  2. Local settings which override globally settings. These are configured in ".codecov.yml" which is already present in odo repo.

Not sure if there is a way to enable/disable a particular repo in the Team Yaml file.

@girishramnani girishramnani added the kind/feature Categorizes issue as a feature request. For PRs, that means that the PR is the implementation label Jan 8, 2020
kanchwala-yusuf added a commit to kanchwala-yusuf/odo that referenced this issue Feb 5, 2020
2. Changes to generate-coverage.sh for fixing errors

Fixes redhat-developer#1557
kanchwala-yusuf added a commit to kanchwala-yusuf/odo that referenced this issue Feb 5, 2020
kanchwala-yusuf added a commit to kanchwala-yusuf/odo that referenced this issue Feb 5, 2020
kanchwala-yusuf added a commit to kanchwala-yusuf/odo that referenced this issue Feb 5, 2020
kanchwala-yusuf added a commit to kanchwala-yusuf/odo that referenced this issue Feb 5, 2020
kanchwala-yusuf added a commit to kanchwala-yusuf/odo that referenced this issue Feb 5, 2020
kanchwala-yusuf added a commit to kanchwala-yusuf/odo that referenced this issue Feb 5, 2020
@kanchwala-yusuf
Copy link
Contributor

Thanks to @amitkrout, codecov comments are being posted on PRs after the access to codecov was granted at the openshift org level.

Now that we can see the codecov reports on each PR, closing this issue.

Please reopen if necessary.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/testing Issues or PRs related to testing, Quality Assurance or Quality Engineering kind/feature Categorizes issue as a feature request. For PRs, that means that the PR is the implementation priority/Medium Nice to have issue. Getting it done before priority changes would be great.
Projects
None yet
5 participants