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

Fixes #26686 - Pulp3 changed contents #8097

Closed
wants to merge 1 commit into from

Conversation

sjha4
Copy link
Member

@sjha4 sjha4 commented May 1, 2019

After a pulp3 sync, we get a new version href from pulp3. We want to check in the new version created if any new content was added. Based on that we save the version, create new publication and refresh distribution.
The same contents_changed is passed to index action to either run or skip indexing.

To do:

  • Re-record VCRs

(cherry picked from commit 9fe5a36757e60017ff376aad52ae550203a6a76f)
@theforeman-bot
Copy link

Issues: #26686

@@ -14,7 +14,6 @@ def invoke_external_task
end

def external_task=(tasks)
output[:contents_changed] = false
output[:create_version] = true
super
end
Copy link
Member Author

Choose a reason for hiding this comment

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

We don't need this anymore. Will remove after tests are run.

create_distribution(prefix, path) if repo.root.unprotected
else
create_distribution(prefix, path)
end
end
Copy link
Member Author

Choose a reason for hiding this comment

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

Copied from @jjeffers update PR since there were issues with the distribution creation.

@sjha4
Copy link
Member Author

sjha4 commented May 28, 2019

Cherry-picked in #8123. Closing..

@sjha4 sjha4 closed this May 28, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants