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

Builder role kickstart ISO inject fails from duplicate error #123

Closed
zjpeterson opened this issue Mar 15, 2023 · 1 comment · Fixed by #131
Closed

Builder role kickstart ISO inject fails from duplicate error #123

zjpeterson opened this issue Mar 15, 2023 · 1 comment · Fixed by #131
Assignees
Labels
approved pm aprroved to work bug Something isn't working

Comments

@zjpeterson
Copy link

When using builder_compose_type=edge-installer, I get a failure from mkksiso that the ISO already exists:

ERROR: Command 'mkksiso /var/www/html/ansible_edge/kickstart.ks /tmp/ansible_edge/ansible_edge-0.0.1.iso /tmp/ansible_edge/ansible_edge-0.0.1.iso' failed with return code: 1 and error message, 'ERROR:/tmp/ansible_edge/ansible_edge-0.0.1.iso already exists\nERROR:Problems running /bin/mkksiso\n'"

If I change the role code here:

- name: Inject kickstart into iso
infra.osbuild.inject_ks:
kickstart: "/var/www/html/{{ builder_blueprint_name }}/kickstart.ks"
src_iso: "/tmp/{{ builder_blueprint_name }}/{{ builder_blueprint_name }}-{{ blueprint_output['current_version'] }}.{{ compose_start_out['result']['output_type'] }}" # noqa yaml[line-length]
dest_iso: "/tmp/{{ builder_blueprint_name }}/{{ builder_blueprint_name }}-{{ blueprint_output['current_version'] }}.{{ compose_start_out['result']['output_type'] }}" # noqa yaml[line-length]
- name: Copy installer to web dir
ansible.builtin.copy:
src: "/tmp/{{ builder_blueprint_name }}/{{ builder_blueprint_name }}-{{ blueprint_output['current_version'] }}.{{ compose_start_out['result']['output_type'] }}" # noqa yaml[line-length]
dest: "/var/www/html/{{ builder_blueprint_name }}/images/{{ builder_blueprint_name }}_{{ _builder_compose_type }}.{{ compose_start_out['result']['output_type'] }}"

using a different temporary filename on lines 215 and 219 (I just put a _ks at the end) then it succeeds. Should lines 214 and 215 really be identical?

@matoval matoval self-assigned this Mar 16, 2023
@chadmf chadmf added bug Something isn't working approved pm aprroved to work labels Mar 17, 2023
@matoval matoval removed their assignment Mar 21, 2023
@matoval
Copy link
Collaborator

matoval commented Mar 28, 2023

@zjpeterson We are having trouble reproducing this issue. Do you have anymore information on how to reproduce this issue?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved pm aprroved to work bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants