Skip to content

[WIP] travis reorganisation - #3267

Closed
DaanHoogland wants to merge 4 commits into
apache:masterfrom
DaanHoogland:slimDownTravis
Closed

[WIP] travis reorganisation#3267
DaanHoogland wants to merge 4 commits into
apache:masterfrom
DaanHoogland:slimDownTravis

Conversation

@DaanHoogland

@DaanHooglandDaanHoogland commented Apr 8, 2019

Copy link
Copy Markdown
Contributor

Description

Due to tests randomly failing with timeout....

@DaanHoogland
DaanHoogland requested review from borisstoyanov, rafaelweingartner and yadvr and removed request for borisstoyanov and yadvrApril 8, 2019 07:43
borisstoyanov
borisstoyanov reviewed Apr 11, 2019
View reviewed changes

@borisstoyanovborisstoyanov left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

#14 seems to timeout, can you split it as well?

@DaanHoogland

DaanHoogland commented Apr 12, 2019

Copy link
Copy Markdown
ContributorAuthor

done @borisstoyanov , though there are only three smaller tests in there. doubt if size is really the problem, in this case!???

GabrielBrascher
GabrielBrascher approved these changes Apr 23, 2019
View reviewed changes

@GabrielBrascherGabrielBrascher left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks @DaanHoogland!

@GabrielBrascherGabrielBrascher mentioned this pull request Apr 23, 2019
Merged
5 tasks
nvazquez
nvazquez approved these changes Apr 24, 2019
View reviewed changes

@nvazqueznvazquez left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @DaanHoogland!

@yadvr

yadvr commented Apr 24, 2019

Copy link
Copy Markdown
Member

Looks good @DaanHoogland I've rekicked Travis; perhaps some of the tests may be clubbed? Ideally if a job passes within 50mins travis does not consider it failed.

@yadvr

yadvr commented May 23, 2019

Copy link
Copy Markdown
Member

@DaanHoogland can you test latest master, I've tried to re-arrange and fix the failing travis tests due to timeout on 4.11 and forward merged them on 4.12 and master.

@DaanHoogland

DaanHoogland commented May 23, 2019

Copy link
Copy Markdown
ContributorAuthor

ok @rhtyd , i will rebase and force push to looksee

@yadvr

yadvr commented Jul 10, 2019•
edited
Loading

Copy link
Copy Markdown
Member

This is old, there is a new PR the continues the effort here: #3476

@yadvryadvr closed this Jul 10, 2019
@DaanHoogland
DaanHoogland deleted the slimDownTravis branch August 16, 2023 11:31
winterhazel pushed a commit that referenced this pull request Jan 28, 2026
SadiJr
Merge branch 'fix-backup-veeam-dummy' into '4.20.0.0-scclouds'
95868f2
Correção na nomeação de _backups_ para Veeam e Dummy
Closes#3267
See merge request scclouds/scclouds!1355
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@GabrielBrascherGabrielBrascherGabrielBrascher approved these changes
@nvazqueznvazqueznvazquez approved these changes
@borisstoyanovborisstoyanovborisstoyanov approved these changes
@yadvryadvrAwaiting requested review from yadvr
@rafaelweingartnerrafaelweingartnerAwaiting requested review from rafaelweingartner

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@DaanHoogland@yadvr@GabrielBrascher@nvazquez@borisstoyanov