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.

Sign up for GitHub

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

Jump to bottom

Image refresh for fedora-coreos #921

Merged
merged 3 commits into from
Jun 4, 2020
Merged

Image refresh for fedora-coreos #921

merged 3 commits into from
Jun 4, 2020

Conversation

cockpituous
Copy link
Contributor

@cockpituous cockpituous commented Jun 3, 2020 •
edited
Loading

Image refresh for fedora-coreos

  • image-refresh fedora-coreos

@cockpituous cockpituous added the bot label Jun 1, 2020
@cockpituous
Copy link
Contributor Author

image-refresh in progress on 1-ci-srv-05.
Log: https://logs.cockpit-project.org/logs/image-refresh-921-20200601-233030/

@cockpituous cockpituous changed the title Image refresh for fedora-coreos WIP: 1-ci-srv-05: [no-test] Image refresh for fedora-coreos Jun 1, 2020
@cockpituous cockpituous changed the title WIP: 1-ci-srv-05: [no-test] Image refresh for fedora-coreos WIP: centosci-tasks-7w9qz: [no-test] Image refresh for fedora-coreos Jun 1, 2020
@cockpituous
Copy link
Contributor Author

image-refresh in progress on centosci-tasks-7w9qz.
Log: https://logs-https-cockpit.apps.ci.centos.org/logs/image-refresh-921-20200601-233030/

@cockpituous
Copy link
Contributor Author

Task failed: https://logs-https-cockpit.apps.ci.centos.org/logs/image-refresh-921-20200601-233030/

@cockpituous cockpituous changed the title WIP: centosci-tasks-7w9qz: [no-test] Image refresh for fedora-coreos Image refresh for fedora-coreos Jun 1, 2020
@cockpituous
Copy link
Contributor Author

image-refresh in progress on 2-cockpit-8.
Log: https://logs.cockpit-project.org/logs/image-refresh-921-20200603-164242/

@cockpituous cockpituous changed the title Image refresh for fedora-coreos WIP: 2-cockpit-8: [no-test] Image refresh for fedora-coreos Jun 3, 2020
@cockpituous
Copy link
Contributor Author

image-refresh fedora-coreos done: https://github.com/cockpituous/bots/commits/image-refresh-fedora-coreos-20200603-165050

@cockpituous cockpituous changed the title WIP: 2-cockpit-8: [no-test] Image refresh for fedora-coreos Image refresh for fedora-coreos Jun 3, 2020
@martinpitt
Copy link
Member

All tests fail with

fatal: The server refused to authenticate 'admin' using password authentication, and no other supported authentication methods are available.

It seems ssh disabled password auth by default now?

@martinpitt martinpitt self-assigned this Jun 3, 2020
@martinpitt martinpitt mentioned this pull request Jun 3, 2020
Closed
@martinpitt
Copy link
Member

Apparently this was a deliberate change, see coreos/fedora-coreos-tracker#138 . At some point we'll be able to change this in Ignition, but it seems that currently there's no official knob for that. So let's re-enable it in sshd for now and adjust documentation later once there is an official method.

@martinpitt
Copy link
Member

Ah, my hack works for cockpit, but not for c-ostree.

@martinpitt
images: Drop obsolete hack in fedora-coreos.install
d1e668a 
This got fixed more generally in Cockpit itself.
@martinpitt
images: Adjust enabling SSH password authentication
580d0ef 
Current image changed the default to create a separate file. Move the
hack to the setup script, so that cockpit-ostree and other projects
benefit from it as well.

Also adjust the UseDNS config change to use the new sshd_config.d/
directory.

Closes #921
@cockpituous cockpituous changed the title Image refresh for fedora-coreos WIP: 1-ci-srv-03: [no-test] Image refresh for fedora-coreos Jun 3, 2020
@cockpituous
Copy link
Contributor Author

image-refresh in progress on 1-ci-srv-03.
Log: https://logs.cockpit-project.org/logs/image-refresh-921-20200603-193333/

@cockpituous
Copy link
Contributor Author

image-refresh fedora-coreos done: https://github.com/cockpituous/bots/commits/image-refresh-fedora-coreos-20200603-194444

@cockpituous cockpituous changed the title WIP: 1-ci-srv-03: [no-test] Image refresh for fedora-coreos Image refresh for fedora-coreos Jun 3, 2020
@martinpitt martinpitt requested a review from marusak June 4, 2020 04:50
marusak
marusak approved these changes Jun 4, 2020
View reviewed changes
Copy link
Member

@marusak marusak left a comment

Choose a reason for hiding this comment

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

Thanks!

@marusak marusak merged commit f7c8c3c into cockpit-project:master Jun 4, 2020
marusak pushed a commit that referenced this pull request Jun 4, 2020
@martinpitt @marusak
images: Adjust enabling SSH password authentication
938e2f6 
Current image changed the default to create a separate file. Move the
hack to the setup script, so that cockpit-ostree and other projects
benefit from it as well.

Also adjust the UseDNS config change to use the new sshd_config.d/
directory.

Closes #921
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@marusak marusak marusak approved these changes

Assignees

@martinpitt martinpitt

Labels
bot no-test
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@cockpituous @martinpitt @marusak