Skip to content
This repository was archived by the owner on May 12, 2025. It is now read-only.

WIP: Add bash to alpine image#109

Closed
1084565 wants to merge 2 commits intomasterfrom
pr/bash-for-neo
Closed

WIP: Add bash to alpine image#109
1084565 wants to merge 2 commits intomasterfrom
pr/bash-for-neo

Conversation

@1084565
Copy link
Copy Markdown
Contributor

@1084565 1084565 commented Jun 27, 2019

Without bash the container won't run on Azure Devops.
cf. Azure doc

@1084565 1084565 requested review from IKirilova and fwilhe June 27, 2019 17:16
Copy link
Copy Markdown

@IKirilova IKirilova left a comment

Choose a reason for hiding this comment

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

lgtm

@1084565
Copy link
Copy Markdown
Contributor Author

1084565 commented Jul 4, 2019

The job exceeded the maximum time limit for jobs, and has been terminated.

Does anybody @SAP/jenkins-team2 have an idea why?

@fwilhe
Copy link
Copy Markdown
Contributor

fwilhe commented Jul 5, 2019

The job exceeded the maximum time limit for jobs, and has been terminated.

Does anybody @SAP/jenkins-team2 have an idea why?

Did not have a close look, but I've had this happen before. Will improve when we have the repo split because less load will be on travis. Would try to rerun.

@1084565 1084565 changed the title Add bash to alpine image WIP: Add bash to alpine image Jul 8, 2019
Copy link
Copy Markdown
Contributor Author

@1084565 1084565 left a comment

Choose a reason for hiding this comment

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

Please dont merge for now

@fwilhe
Copy link
Copy Markdown
Contributor

fwilhe commented Jul 12, 2019

Is this required? If so, please open in the new repo.

@1084565
Copy link
Copy Markdown
Contributor Author

1084565 commented Jul 12, 2019

Yes, I will, just give me a couple of days.

@fwilhe
Copy link
Copy Markdown
Contributor

fwilhe commented Jul 16, 2019

Merged in new repo

@fwilhe fwilhe closed this Jul 16, 2019
@1084565 1084565 deleted the pr/bash-for-neo branch June 2, 2022 15:59
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants