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

Delete registries, add registries, edit, view registries test. #1027

Merged
merged 2 commits into from Oct 12, 2021

Conversation

MilanPospisil
Copy link
Contributor

@MilanPospisil MilanPospisil commented Oct 4, 2021

Test for Execution enviroments -> Remote Registries. It tests addition of new and view of table of registries. It deletes registries before and after to maintain db clean and allow repetition of tests.

@MilanPospisil MilanPospisil changed the title delete registries, add registries, view registries test. [WIP] Delete registries, add registries, redit, view registries test. Oct 4, 2021
@MilanPospisil MilanPospisil changed the title [WIP] Delete registries, add registries, redit, view registries test. Delete registries, add registries, redit, view registries test. Oct 4, 2021
@MilanPospisil MilanPospisil changed the title Delete registries, add registries, redit, view registries test. Delete registries, add registries, edit, view registries test. Oct 4, 2021
Copy link
Collaborator

@himdel himdel left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@himdel himdel merged commit c331653 into ansible:master Oct 12, 2021
@himdel himdel added the backport-4.4 This PR should be backported to stable-4.4 (2.1) label Oct 29, 2021
@patchback
Copy link

patchback bot commented Oct 29, 2021

Backport to stable-4.4: 💚 backport PR created

✅ Backport PR branch: patchback/backports/stable-4.4/c3316537e792851c9304482e9c2f6b6a206f1e8c/pr-1027

Backported as #1145

🤖 @patchback
I'm built with octomachinery and
my source is open — https://github.com/sanitizers/patchback-github-app.

patchback bot pushed a commit that referenced this pull request Oct 29, 2021
* delete registries, add registries, view registries test.

* Edit test

(cherry picked from commit c331653)
@himdel
Copy link
Collaborator

himdel commented Oct 29, 2021

(backporting because #1116 depens on this)

himdel pushed a commit that referenced this pull request Oct 29, 2021
#1145)

* delete registries, add registries, view registries test.

* Edit test

(cherry picked from commit c331653)

Co-authored-by: MilanPospisil <arkanus@seznam.cz>
@github-actions github-actions bot added the backported-4.4 This PR has been backported to stable-4.4 (2.1) label Oct 29, 2021
@MilanPospisil
Copy link
Contributor Author

MilanPospisil commented Nov 3, 2021 via email

@himdel
Copy link
Collaborator

himdel commented Nov 3, 2021

@MilanPospisil use English please, in any public communication :)
And do read up on escaping code in markdown, the error message is barely readable this way. https://guides.github.com/features/mastering-markdown/
(Also, is this related to this PR at all?)


As for what you're doing wrong, the command looks good.

Try running it manually to see what happens, any output?
Also try pip show galaxykit | grep Version to see which version you're using, you'll need 0.5.2 or newer.

(And make sure that the UI is actually running on localhost:8002.)

If it's none of those, you can try debugging galaxykit, either by adding logging in galaxykit itself, or by watching the responses with something like strace -e sendto,recvfrom -s 200 galaxykit -u admin -p admin collection upload

EDIT: you can also try which -a galaxykit to see if multiple versions are installed .. could be an old version in /usr/local/bin/galaxykit (installed via sudo pip...) winning over a locally installed .local/bin/galaxykit (installed with pip --user). ; Or it could be that a pip --user installed version is not in PATH.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport-4.4 This PR should be backported to stable-4.4 (2.1) backported-4.4 This PR has been backported to stable-4.4 (2.1)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants