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

(FM-7971) backwards compatibility with PE 2019.0 #82

Merged
merged 5 commits into from
Apr 25, 2019

Conversation

Thomas-Franklin
Copy link
Contributor

@Thomas-Franklin Thomas-Franklin commented Apr 23, 2019

This PR will add support to the module to work with RSAPI 1.6.x that Puppet 6.0.x is shipped with. This PR could be reverted once STS of PE 2019.0 is over.

@codecov
Copy link

codecov bot commented Apr 23, 2019

Codecov Report

Merging #82 into master will increase coverage by 0.1%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff            @@
##           master      #82     +/-   ##
=========================================
+ Coverage   99.15%   99.26%   +0.1%     
=========================================
  Files          39       40      +1     
  Lines         948     1084    +136     
=========================================
+ Hits          940     1076    +136     
  Misses          8        8
Impacted Files Coverage Δ
lib/puppet/util/task_helper.rb 100% <100%> (ø) ⬆️
lib/puppet_x/puppetlabs/panos/transport_shim.rb 100% <100%> (ø)
lib/puppet/util/network_device/panos/device.rb 100% <100%> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 6c278c3...7d38a29. Read the comment docs.

@Thomas-Franklin Thomas-Franklin changed the title (FM-7971) updating the travis testing to cover 6.0.x (FM-7971) backwards compatibility with RSAPI < 1.8.x Apr 23, 2019
@Thomas-Franklin Thomas-Franklin changed the title (FM-7971) backwards compatibility with RSAPI < 1.8.x WIP (FM-7971) backwards compatibility with RSAPI < 1.8.x Apr 23, 2019
@Thomas-Franklin
Copy link
Contributor Author

This is blocked for timebeing on a release of 1.6.4 RSAPI to rubygems

Thomas-Franklin added 4 commits April 23, 2019 16:50
This code is a horrible workaround for the transport work that this module was converted to support but meant that current STS puppet versions would not work.

This code can be reverted in August 2019 once 2019.0 STS ends.
This is to satisfy the condition that the version of bolt on a 6.0.x PE instalation may use the 1.6.x RSAPI version.

Example of a success with 1.6.x RSAPI and bolt:

```
Started on cgnodswd2r9rm9e.delivery.puppetlabs.net...
Finished on cgnodswd2r9rm9e.delivery.puppetlabs.net:
  {
    "apikey": "xxx"
  }
Successful on 1 node: cgnodswd2r9rm9e.delivery.puppetlabs.net
Ran on 1 node in 2.85 seconds
```
@DavidS DavidS changed the title WIP (FM-7971) backwards compatibility with RSAPI < 1.8.x (FM-7971) backwards compatibility with PE 2019.0 Apr 24, 2019
@DavidS DavidS added bugfix Signify a bug fix and removed invalid labels Apr 24, 2019
Copy link
Contributor

@DavidS DavidS left a comment

Choose a reason for hiding this comment

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

Looks good to me.

@DavidS DavidS merged commit 749bc6e into puppetlabs:master Apr 25, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bugfix Signify a bug fix
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants