Project

General

Profile

Actions

action #57050

closed

Turn off travis

Added by coolo over 4 years ago. Updated about 4 years ago.

Status:
Resolved
Priority:
High
Assignee:
Category:
Organisational
Target version:
Start date:
2019-09-30
Due date:
% Done:

100%

Estimated time:
(Total: 0.00 h)

Description

Most obvious lack will be documentation generation, possibly more


Subtasks 1 (0 open1 closed)

action #57548: Move building and publishing of docs to CircleCIResolvedandriinikitin2019-09-30

Actions

Related issues 6 (1 open5 closed)

Related to openQA Project - action #53915: switch "travis_test" aka. "openqa_dev" container to a supported base since Leap 42.3 is EOLResolvedokurz2019-09-07

Actions
Related to openQA Project - action #55478: Evaluate circleci for openQAResolvedandriinikitin2019-08-14

Actions
Related to openQA Project - coordination #55364: [epic] Let's make codecov reports reliableResolvedokurz2020-09-24

Actions
Related to openQA Project - action #60056: badge in https://github.com/os-autoinst/openQA/blob/master/README.asciidoc references travis CI, should be circle CIResolvedandriinikitin2019-11-19

Actions
Related to openQA Project - action #57047: Mark .circleci/dependencies.txt as generated fileRejectedokurz2019-09-18

Actions
Related to openQA Project - action #73309: every time a direct dependency is updated in Factory our CI jobs fail until the package is updatedWorkable2020-10-13

Actions
Actions #1

Updated by mkittler over 4 years ago

Not sure about the coverage data. Likely the Circle CI job doesn't upload it to codecov yet.

Actions #2

Updated by andriinikitin over 4 years ago

  • Related to action #53915: switch "travis_test" aka. "openqa_dev" container to a supported base since Leap 42.3 is EOL added
Actions #3

Updated by andriinikitin over 4 years ago

At the moment circleci runs tests without codecov, so coverage must be implemented / tested as part of this ticket

Actions #4

Updated by andriinikitin over 4 years ago

Actions #5

Updated by andriinikitin over 4 years ago

  • Status changed from New to In Progress
  • Assignee set to andriinikitin
Actions #6

Updated by okurz over 4 years ago

  • Blocks action #57047: Mark .circleci/dependencies.txt as generated file added
Actions #7

Updated by andriinikitin over 4 years ago

Now I think dependencies.txt must be renamed to something like ci-setup.txt or testing-layout.txt or packages-used.txt
Because its role is to set up consistent testing environment, and current name may make impression that it is for managing dependencies.

Actions #9

Updated by livdywan over 4 years ago

  • Status changed from In Progress to Blocked

Blocked by #57548

Actions #10

Updated by okurz over 4 years ago

Actions #11

Updated by okurz over 4 years ago

  • Status changed from Blocked to Resolved

With also the docs generation done properly in circle CI we can either set this ticket to "Resolved" or we can keep it open to cover os-autoinst potentially. @andriinikitin I set it to "Resolved" now as we decided in the weekly QA tools meeting that os-autoinst is not important or urgent. Please feel welcome to reopen with a comment if you think differently.

Actions #12

Updated by okurz over 4 years ago

  • Status changed from Resolved to Workable

Ok, I was wrong. We are not really done, e.g. see

$ git grep -i travis
.gitignore:.openqa-travis
Makefile:ifeq ($(TRAVIS),true)
Makefile:       env | grep -E 'CHECKSTYLE|FULLSTACK|UITEST|GH|TRAVIS|CPAN|DEBUG|ZYPPER' > $(docker_env_file)
README.asciidoc::travis-ci: image:https://api.travis-ci.org/os-autoinst/openQA.svg?branch=master[link=https://travis-ci.org/os-autoinst/openQA]
README.asciidoc:{codecov} {travis-ci}
cpanfile:feature 'coverage', 'coverage for travis' => sub {
docs/Contributing.asciidoc:* Every commit is checked by https://travis-ci.org/travis[Travis CI] as soon as
docs/Contributing.asciidoc:  which is made using +docker/travis_test/Dockerfile+ within the openQA repository.
docs/Contributing.asciidoc:tests are executed in the same environment as on the Travis CI. This allows to reproduce issues
docs/Contributing.asciidoc:and based on the `Dockerfile` within the `docker/travis_tests` sub directory of the openQA repository.
docs/Contributing.asciidoc:Logs are redirected to a logfile when running tests within Travis. The output can therefore not be asserted using +Test::Output+. This
script/generate-documentation:            echo "" >> last.commit # somehow travis does not like \n
script/run-tests-within-container:if test -n "$TRAVIS"; then
t/ui/21-admin-needles.t:    # the following should work, but apparently doesn't - at least when executing tests in Travis:
Actions #13

Updated by okurz over 4 years ago

  • Related to action #60056: badge in https://github.com/os-autoinst/openQA/blob/master/README.asciidoc references travis CI, should be circle CI added
Actions #14

Updated by okurz over 4 years ago

  • Blocks deleted (action #57047: Mark .circleci/dependencies.txt as generated file)
Actions #15

Updated by okurz over 4 years ago

  • Related to action #57047: Mark .circleci/dependencies.txt as generated file added
Actions #16

Updated by okurz over 4 years ago

  • Status changed from Workable to In Progress
  • Assignee changed from andriinikitin to okurz
Actions #17

Updated by okurz over 4 years ago

  • Status changed from In Progress to Feedback

merged but still not there yet: https://github.com/os-autoinst/openQA/pull/2579

Actions #18

Updated by okurz over 4 years ago

  • Status changed from Feedback to Workable
  • Assignee deleted (okurz)

Also merged. But there are still references to travis in the github repo.

Actions #19

Updated by okurz over 4 years ago

  • Status changed from Workable to In Progress
  • Assignee set to okurz

Actually I still have https://github.com/os-autoinst/openQA/pull/2587 open and just updated the branch. Waiting for circle CI test results.

It's hard. Fails in https://app.circleci.com/jobs/github/os-autoinst/openQA/12157 and I did not yet understand why. Did I miss another reference? Will apply https://github.com/os-autoinst/openQA/pull/2698 first. This should really be trivial.

Actions #20

Updated by okurz over 4 years ago

  • Status changed from In Progress to Feedback
Actions #21

Updated by livdywan about 4 years ago

gh#2698 got merged

Actions #22

Updated by okurz about 4 years ago

rebased https://github.com/os-autoinst/openQA/pull/2587/files . I assume circle CI also sets the environment variable "CI" which then goes into "run-tests-within-container". I guess the right approach would be to delete that part in the Makefile as well as anything related to "run-tests-wihin-container" unless we find another use case for these. But we should really bring circle CI tests a bit closer again to what developers would do for manual testing.

Actions #23

Updated by okurz about 4 years ago

  • Status changed from Feedback to Resolved

This is as much as we can do. There is unfortunately more duplication introduced by andriinikitin than we had in before. We should try to solve that in #53546

Actions #24

Updated by okurz over 3 years ago

  • Related to action #73309: every time a direct dependency is updated in Factory our CI jobs fail until the package is updated added
Actions

Also available in: Atom PDF